Stuck For... Next loop

[Grantson]

HI Guys 

Once again, I think (I hope) i just need some fresh eyes to tell me what im missing here

I wrote a script to step through the user profiles on a remote PC and clear out the temp files (that bit seems to work)

I wanted to ad a bit at the end to clear the windows temp folder on the remote PC at the same time so i added this chunk form the already working code for the individual user temp files

Local $FileList = _FileListToArray($windowsTemp,"*")
    If @error = 1 Then
        MsgBox(0, "", "No Folders Found.")
    EndIf
    If @error = 4 Then
        MsgBox(0, "1", "No Files Found.")
    EndIf
    _ArrayDisplay($FileList)
    
    For $element In $FileList
        $isDir = FileGetAttrib($windowsTemp&"\"&$FileList[$element])
        If StringInStr($isDir, "D") Then
            $size = DirGetSize($windowsTemp&"\"&$FileList[$element])
            if $size = 0 then DirRemove($temppath&"\"&$FileList[$element])
        Else
            FileDelete($windowsTemp&"\"&$FileList[$element])
            MsgBox(0,"",$windowsTemp&"\"&$FileList[$element]) ;diagnostic
        EndIf
    Next

The file list populates with the directory listing as expected and the array display lists the entire directory listing as expected

The diagnostic message box will give the correct path and then gets stuck on element 0

It then gets stuck in an infinite loop bringing up a message box with targetpcc$windowstemp486 

486 being the number of files in the temp folder

I'm sure I'm missing something silly that's making it stick but I'm now at the point I cant see it for staring at it

Thanks

Grant 

[Danp2]

I believe all references to $FileList[$element] are incorrect and should only be $element.

[SmOke_N]

Element 0 of _FileListToArray is the number of files found.

As stated by Danp2, your use of For/In is incorrect.

Instead of this:

For $element In $FileList

Try this:

For $element = 1 To Ubound($FileList) - 1

[Grantson]

Thanks Danp2 

That's got it

Sm0ke_N Ill have a look at ubound also.

Looping and arrays is always something that seems to break my brain