I need help improving this statement

[Mesca]

All other aspects of my personal script are working fine. I need to improve this statment though to make it easier to edit and cleaner.

Basically, what I need to happen is the message in the edit box to change every time the while statement completes a loop.

Once it reaches the last message I need to restart from the top again.

Here is what I got, very simple (just showing the part I need help with in hopes to minimize confusion)...

$Edit1 = GUICtrlCreateEdit("", 8, 136, 129, 21, BitOR($ES_AUTOVSCROLL,$ES_AUTOHSCROLL,$ES_READONLY))
GUICtrlSetData(-1, "")
$message = 1

While
If $message = 3 Then
$message = 1
EndIf

If $message = 1 Then
$Edit1 = GUICtrlCreateEdit("", 8, 136, 129, 21, BitOR($ES_AUTOVSCROLL,$ES_AUTOHSCROLL,$ES_READONLY))
GUICtrlSetData(-1, "Test1")
EndIf

If $message = 2 Then
$Edit1 = GUICtrlCreateEdit("", 8, 136, 129, 21, BitOR($ES_AUTOVSCROLL,$ES_AUTOHSCROLL,$ES_READONLY))
GUICtrlSetData(-1, "Test2")
EndIf

$message = $message + 1
WEnd

This is obviously not the best way to go about this. As you can see just to have 2 messages ("Test1" and "Test2")the code is lengthy. But please forgive me, my knowledge is very limited. Help please.

Edited November 19, 2009 by mesca

[water]

You only need to create the Edit Ctrl one time. Then just set the data. So change your code to

$Edit1 = GUICtrlCreateEdit("", 8, 136, 129, 21, BitOR($ES_AUTOVSCROLL, $ES_AUTOHSCROLL, $ES_READONLY))
GUICtrlSetData(-1, "")
$message = 1

While
    Switch $message
        Case 3
            $message = 1
        Case 1
            GUICtrlSetData($Edit1, "Test1")
        Case 2
            GUICtrlSetData($Edit1, "Test2")
    EndSwitch
    $message = $message + 1
    Sleep(1000)
WEnd

Edited November 19, 2009 by water

[Mesca]

You only need to create the Edit Ctrl one time. Then just set the data. So change your code to

$Edit1 = GUICtrlCreateEdit("", 8, 136, 129, 21, BitOR($ES_AUTOVSCROLL,$ES_AUTOHSCROLL,$ES_READONLY))
GUICtrlSetData(-1, "")
$message = 1

While
 If $message = 3 Then $message = 1
 If $message = 1 Then GUICtrlSetData($Edit1, "Test1")
 If $message = 2 Then GUICtrlSetData($Edit1, "Test2")
 $message = $message + 1
WEnd

Worked like a charm. How easy it was. Thank you very much.