I have a variable problem.. any ideas?

[flagg777]

This is most likely a simple problem however I can to find out why..

The Script i uploaded:

You must click all the radio buttons or it bombs ( I will fix this later).

However after you submit the results you get a "score" msg box. On this box you have an "enter email address" field. This pipes to the var @EmailTo (I added the example variable for testing, I made the var global for testing: ALL fail) For some reason I always get the @EmailTo to always return "68" and not what the user inputs.

BTW: I love the email send, it is simple a slick.. Thanks to this form for the code!

Thanks in advance for the help!

Kevin

Alpha-v020-email-v001.au3

[Melba23]

flagg777,

"68" is the ControlID of the input control - you need to use GUICtrlRead to get the content of the control.

M23

[flagg777]

Sorry, that sounds like it is my problem however I do not understand

$Date = GUICtrlCreateInput("", 72, 80, 97, 24)
GUICtrlSetFont(-1, 10, 400, 0, "Arial")

This stores the variable $Date = "what ever the user typed"

This works with my script

when I use

Global $EmailTO = GUICtrlCreateInput("Email", 8, 88, 241, 21)

Why do I get the ControlID when all along I get the user input.

 

 

[water]

No.
$Date stores the ControlID as returned by GUICtrlCreateInput. See the help file for GUICtrlCreateInput and you will see what I'm talking about.
Even $EmailTO will hold the ControlID.

Another good idea is to use Hungarian Notation for variable names so you know which type of data they hold.
That's described in the "Best Coding practices" which you find in the wiki.

BTW: It's bad coding style to define a Global variable in a function (on line 505).

[flagg777]

THANK YOU!

 

$EmailToUser = GUICtrlRead($EmailTO)

This nailed it!

(the noob will rub off, I promise)

Thanks again

[water]

I still propose to use the naming convention I suggested above. So so make sure to always use the correct variable:

Global $idEmailTO = GUICtrlCreateInput("Email", 8, 88, 241, 21)
...
$sEmailTO = GUICtrlRead($idEmailTO)