Prudent way of 'declaring' and 'setting' variables in a Userform?

[Irmaxx]

Hi fellow forum members,

Introduction
I have a file in which I register all the people to whom I send a (birthday)card.
Also I register to whom, when and which text and/or poem I have sent.
To do all of that, the Excel file has 4 sheets, of which 1 is hidden (not really important now), and these are:
- Card Text (columns: year, name, country, card text, poem text)
- Poem Text (columns: poem number, poem text, country code)
- Personal credentials (columns: name, country, deceased, columns with years and poem numbers)

The Userform is my main input conduit, I can:
- select a surname
- select a first name
- select a country
- select a poem number, which gives the corresponding poem text
- add/change/delete personal credentials

Userform summary
-------------------------------------------------------------------------------------------------
► At the very start, below "Option Explicit" (total of: 31x Module-variables)
-------------------------------------------------------------------------------------------------
6x Module STRING-variables -------> (e.g. "Private m_sSurname As String")
5x Module INTEGER-variables ------> (e.g. "Private m_iFirstSurnameRow As Integer")
10x Module BOOLEAN-variables ----> (e.g. "Private m_bStartDate As Boolean")
1x Module NEW CLASS-variables ---> (e.g. "Private m_oColor As New clsColoring")
5x Module LISTOBJECT-variables ---> (e.g. "Private m_oTbl1 As ListObject")
4x Module WORKSHEET-variables --> (e.g. "Private m_WS1 as Worksheet")

-------------------------------------------------------------------------------------------------
► In the Sub "Userform_Initialize" (total of: 9x "settings")
-------------------------------------------------------------------------------------------------
4x WORKSHEET settings ------------> (e.g. "Set WS1 = cnCardsNL")
5x TABLE NAME settings ------------> (e.g. "Set m_oTbl0 = WS1.ListObjects(1)")

-------------------------------------------------------------------------------------------------
► Other (meaningless?) totals in the Userform
-------------------------------------------------------------------------------------------------
36x Private Subs
98x variables within the 36x SUBs

Question
"Is this way of programming prudent or just bad programming ?" (see: attachments)
In other words: do I 'declare' and 'set' as many variables as possible as Module variables or is it more prudent to 'declare' and 'set' them in the appropriate Subs?

Note: it seems the Userform loads (reacts) quicker when as many variables as possible are 'declared' and 'set' as Module variables....!

It would be much appreciated if someone could shed some light on programming protocols / rules / etiquette.
Thanks in advance and.... I do apologies for this rather elaborate post, sorry.