Solved: ready to try functions

[dvenn]

okay.. having learned so much from here... I think I am ready to move into functions..

Here is what I want to try...

[VBA] Public Function DoStuff (Proc as ??)
Load userform1
Applications.OnTime Now + TimeValue("00:00:01"), Proc
End Function
[/VBA]

Proc = a macro name..

The userform actually watches the macros run.. and captures information (I have that part), and until now I was running this as a sub.. but I added another routine that I want the user form to watch... so instead of doing seomthing like adding a second copy of the sub.. i thought "If I could pass the maro name to a function then I wouldn't need to do that.."

anyways.. I tried as String, as Variant, & as Object but I get and error on the line in the macro

[VBA]doStuff somemacro[/VBA]

compile Error:

expected Function or variable

[Bob Phillips]

It should be a string, because OnTime takes a string parameter, the name of the proc.

[Norie]

If you are trying to pass a string try enclosing it in quotes.

[Bob Phillips]

BTW, it doesn't hurt, but it doesn't need to be a function. Functions are usually used to return a result to the caller.

[dvenn]

can you pass a string to a sub?

[dvenn]

Also I tried enclosing the proc with " "

[VBA] doStuff "somemacro"
[/VBA]

But now I get an object required error on

[VBA] Applications.OnTime Now + TimeValue("00:00:01"), Proc [/VBA]

Getting closer

[Bob Phillips]

Like this

[vba]
Sub CallerProc()
myOtherProc "Bob"
End Sub

Sub myOtherProc(nme As String)
MsgBox nme
End Sub
[/vba]

[dvenn]

had tried that but when I was getting the errors I thought maybe i needed a function for it...

Still getting the object required errror with both function and sub

[dvenn]

okay (hangs head in shame) found the error...

thanks for not laughing at me too hard.. (at least publically )

Applications.onTime.. darn Typo..

[mdmackillop]

Hi Daniel,
I posted an OnTime code here yesterday
http://www.vbaexpress.com/forum/show...29&postcount=5
Regards
MD

[Bob Phillips]

okay (hangs head in shame) found the error...

thanks for not laughing at me too hard.. (at least publically )

Applications.onTime.. darn Typo..

LOL. Didin't spot that.