jwcolby
jwcolby at colbyconsulting.com
Wed Aug 8 14:37:54 CDT 2012
A.D.
>Coming back to the original question, it would seem that for such a form, if both sets of events
(built-in as well as custom) are to be utilized in a wrapper class, setting up of two separate
pointers would be necessary.
It certainly does appear that way. If my class is going to sink such events I usually create
WithEvent pointers in the form wrapper for the control that will be raising the event and then
handle it in that form wrapper class. What you are essentially saying is that you want the wrapper
class to handle the event in the end so why not just sink the event directly in the form wrapper class?
Basically I remove all of that stuff from the form itself and place it in the form wrapper. Now I
either pass in the control in my wrapper mInit() method or I reach up into the form and get a
pointer to those controls. I always pass a pointer to the form to my wrapper class and store it so
getting stuff from the form is trivial.
In fact if the control itself has any other functionality it needs to handle I will create a wrapper
for the control itself as well. Combos and text boxes are examples of such controls.
As an example in my framework I have a mFindControls method which literally iterates the form's
controls collection and attempts to create a wrapper for each control found, the type of which is
called out in a big case statement. I have prewritten wrapper classes for some but hardly all of
the various types of controls. However if down the road I need a new wrapper class for a control
type not yet covered, I write it and hook it into this case statement. This is how my framework
evolved over time.
I use naming conventions to trigger loading variations of control wrappers as needed. As an example
I will have a txtPKID text box bound to my PKID (always a long int right!) and when found I plug
that into a dimensioned variable in the class header (no wrapper at all). I use that control to
tell the form wrapper what PKID the form is currently on in case I need to sync a combo box or whatever.
OTOH I have a class which I load for text boxes not matching my naming conventions. This allows me
to (for example) look at the data type of the field that the text box is bound to (this is possible
with DAO) and set a standardized date/time format in the text box's format property if the control
is bound to a date/time data type.
For Each ctl In mfrm.Controls 'Find each control in the form's control collection
With ctl
On Error Resume Next
If blnNoRecordset = False Then
intCtlDatatype = mrst.Fields(ctl.ControlSource).Type
End If
On Error GoTo Err_mFindControls
Select Case .ControlType 'Determine it's type
Case acTextBox 'Find all text boxes and load class to change backcolor
'
'There can be two text boxes used for specific purposes.
'If they exist they will have specific names.
'
Select Case .name
'
'txtPKID is the control bound to the PK of the recordset
'and is used for finding records, the recordselector combo etc.
'
Case "txtPKID"
Set mtxtPKID = ctl
'
'txtParentPKID is bound to the parent form's PK in a subform
'and allows the framework to discover when the parent record changed
'so that objects in the subform can be requeried when the parent form
'changes from record to record
'
Case "txtFKParentID"
Set mtxtFKParentID = ctl
mstrFKFldName = mtxtFKParentID.ControlSource
On Error Resume Next
mlngParentPK = mtxtFKParentID.Value
Case Else
mcolClasses.Add New dclsCtlTxtBox, .name
mcolClasses(.name).mInit Me, mfrm, ctl, intCtlDatatype
End Select
Case acSubform
mcolSubForms.Add New dclsCtlSFrm, .name
mcolSubForms(.name).mInit Me, mfrm, ctl
Case acTabCtl 'tab pages are handled in the tab control
mcolTabs.Add New dclsCtlTab, .name
mcolTabs(.name).mInit Me, mfrm, ctl
Case acOptionGroup
Case acCheckBox
Case acOptionButton
Case acCommandButton
Case acToggleButton
Case acListBox
Case acComboBox
If .name = "cboRecSel" Then
Set mdclsCtlCboRecSel = New dclsCtlCboRecSel
'mcolClasses.Add New dclsCtlCboRecSel, .Name
mdclsCtlCboRecSel.mInit Me, mfrm, ctl, mfrm!txtPKID, mblnUsesADO
'mdclsCtlCboRecSel.cGI.pIsADOFrm = mblnUsesADO
Else
Dim lclsCtlCbo As dclsCtlCbo
Set lclsCtlCbo = New dclsCtlCbo
lclsCtlCbo.mInit Me, mfrm, ctl, intCtlDatatype
lclsCtlCbo.cGI.pIsADOFrm = mblnUsesADO
mcolClasses.Add lclsCtlCbo, .name
' mcolClasses(.name).mInit Me, mfrm, ctl, intCtlDatatype
End If
Case Else
End Select
John W. Colby
Colby Consulting
Reality is what refuses to go away
when you do not believe in it
On 8/8/2012 2:44 PM, A.D. Tejpal wrote:
> JC,
>
> It so happens that hooking of events is not the issue here. In the class module, setting required event properties to "[Event Procedure]" is a fundamental step, without which the event won't fire. That aspect is always taken care of.
>
> My question refers to a form where a certain event has been raised via RaiseEvent statement. Let us say a filter and / or search action is carried out through a combination of controls like option groups / combo boxes / text boxes / check boxes etc. On successful filter / search, event EvFilterSearch() is raised via RaiseEvent statement in form's module. In the wrapper class, excessive cluttering up can be avoided by simply trapping EvFilterSearch() - rather than setting up WithEvents pointers to individual controls.
>
> In the wrapper class, if the pointer to such a form (named MyForm) is declared as Access.Form, and an attempt made to hook the events after due instantiation, it will be found that there is no property matching EvFilterSearch event. This event is also found missing from the drop down list at top right of VBA window, which shows all built-in events like Load / Current etc, including those for which explicit hooking by setting event property to "[Event Procedure]" has not been done. Of course, unless hooked, code in stubs for un-hooked events won't execute.
>
> On the other hand, if the pointer is declared as Form_MyForm (instead of Access.Form), EvFilterSearch event gets displayed in the drop down list at top right of VBA window. Built-in events like Load / Current do not get displayed in this list. Moreover, no explicit hooking for this custom event is needed. In fact you can't do it, as the item is still found missing from properties list of the pointer (interestingly, the properties list does include the built-in events). Code in stub generated for this event gets executed smoothly.
>
> Coming back to the original question, it would seem that for such a form, if both sets of events (built-in as well as custom) are to be utilized in a wrapper class, setting up of two separate pointers would be necessary.
>
> Best wishes,
> A.D. Tejpal
> ------------