Sign in to follow this  
Followers 0
Clark

I'm obviously missing something..

7 posts in this topic

Hi all

For the life of me I can't figure out why this doesn't work.

The idea is to exit the for/next loop and then use the value of $Stat outside the function.

Obviously my understanding of something is deeply flawed, as this does not work as I think it should.

global $Stat,$var=3
 
_Test_Stat()
msgbox(0,"Test","The value of $Stat is " & $Stat)
 
Func _Test_Stat()
    for $Stat = 0 to 5
        if $Stat = $var Then
                ExitLoop
        EndIf
    Next
EndFunc

Share this post


Link to post
Share on other sites



You never assign a value to $Stat.

Why do you want to check the value of $Stat in a loop? You could check $Stat directly.


My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (NEW 2017-04-18 - Version 1.4.8.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2017-02-27 - Version 1.3.1.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2015-04-01 - Version 0.4.0.0) - Download - General Help & Support - Example Scripts
Excel - Example Scripts - Wiki
Word - Wiki
PowerPoint (2015-06-06 - Version 0.0.5.0) - Download - General Help & Support

Tutorials:
ADO - Wiki

 

Share this post


Link to post
Share on other sites

It is just code to demonstrate the issue, not the real code. :graduated:

If you run this code in the debugger, you will see that $Stat does indeed have a value as it goes through the loop (incrementing each time). It even has a value (in the debuggers watch list) once it exits the loop. The only time the value disappears is if you use $Stat for anything.

By implication, the "For" statement is assigning a value to $Stat. Read it out aloud. For $Stat equals 0 to 5. If $Stat equals something then it must have a value, right?

Share this post


Link to post
Share on other sites

#4 ·  Posted (edited)

I see.

Solution: Please check the help file (For...To...Step...Next):

The Variable will be created automatically with a LOCAL scope, even when MustDeclareVars is on.

So you have a Global plus a Local variable named $Stat.

What you need is something like:

Global $Stat, $Var = 3
$Stat = _Test_Stat($Var)
MsgBox(0, "Test", "The value of $Stat is " & $Stat)
Func _Test_Stat($Param1)
For $iIndex = 0 To 5
  If $iIndex = $Var Then
   ExitLoop
  EndIf
Next
Return $iIndex
EndFunc   ;==>_Test_Stat
Edited by water

My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (NEW 2017-04-18 - Version 1.4.8.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2017-02-27 - Version 1.3.1.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2015-04-01 - Version 0.4.0.0) - Download - General Help & Support - Example Scripts
Excel - Example Scripts - Wiki
Word - Wiki
PowerPoint (2015-06-06 - Version 0.0.5.0) - Download - General Help & Support

Tutorials:
ADO - Wiki

 

Share this post


Link to post
Share on other sites

Well, there you go. My understanding was definitely flawed. I would not have thought for a second that the For Next loop would create a temporary variable.

Thanks for your help.

Share this post


Link to post
Share on other sites

#6 ·  Posted (edited)

Clark,

The variable in a For...Next loop is automatically created in Local scope (as stated in the Help file) and will be used in place of a Global variable of the same name if it exists (which it does in this case). So you get the correct check inside the loop (as the modified script below shows) but when you return from the function you revert to the Global variable which has no value:

Global $Stat, $var = 3
 
_Test_Stat()
MsgBox(0, "Test", "The value of $Stat is " & $Stat)
 
Func _Test_Stat()
    For $Stat = 0 To 5
        ConsoleWrite($Stat & " - " & $var & @CRLF)
        If $Stat = $var Then
            ExitLoop
        EndIf
    Next
EndFunc   ;==>_Test_Stat

I hope that explains the lack of a value in $Stat.

If you want a value, then you could either use a different variable name for the loop and then set $Stat to that value within the function:

Global $Stat, $var = 3
 
_Test_Stat()
MsgBox(0, "Test", "The value of $Stat is " & $Stat)
 
Func _Test_Stat()
    For $i = 0 To 5
        ConsoleWrite($i & " - " & $var & @CRLF)
        If $i = $var Then
            $Stat = $i
            ExitLoop
        EndIf
    Next
EndFunc   ;==>_Test_Stat

Or return the value from the function like this:

Global $Stat, $var = 3
 
$Stat = _Test_Stat()
MsgBox(0, "Test", "The value of $Stat is " & $Stat)
 
Func _Test_Stat()
    For $i = 0 To 5
        ConsoleWrite($i & " - " & $var & @CRLF)
        If $i = $var Then
            ExitLoop
        EndIf
    Next
    Return $i
EndFunc   ;==>_Test_Stat

All clear? :graduated:

M23

Edit:

Good morning water! ;)

Edited by Melba23

Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind._______My UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites

#7 ·  Posted (edited)

Hi, fellows!

Now the cleanest way is to follow Melba last example and return the variable from the function. Procedures with such undue side effects are a plague and lead to another form of spaghetti code (or rather, spaghetti vars).

EDIT: BS removed!

Edited by jchd

This wonderful site allows debugging and testing regular expressions (many flavors available). An absolute must have in your bookmarks.
Another excellent RegExp tutorial. Don't forget downloading your copy of up-to-date pcretest.exe and pcregrep.exe here
RegExp tutorial: enough to get started
PCRE v8.33 regexp documentation latest available release and currently implemented in AutoIt beta.

SQLitespeed is another feature-rich premier SQLite manager (includes import/export). Well worth a try.
SQLite Expert (freeware Personal Edition or payware Pro version) is a very useful SQLite database manager.
An excellent eBook covering almost every aspect of SQLite3: a must-read for anyone doing serious work.
SQL tutorial (covers "generic" SQL, but most of it applies to SQLite as well)
A work-in-progress SQLite3 tutorial. Don't miss other LxyzTHW pages!
SQLite official website with full documentation (may be newer than the SQLite library that comes standard with AutoIt)

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0