tony Posted February 17, 2009 Share Posted February 17, 2009 Hello, I want to scan a directory for all the files that are part of it. I found in the forum a good easy to implement function and it seems to work fine till I call it again. From the secoid time I call it, it adds to the first find the second finds no matter what I do. I tried to delete the array in between but it doesn't work. There is something I am missing but I can't figure out what. Any idea?? Thanks a lot CODE #include <Array.au3> #Include <File.au3> Local $aRetArray Local $path = "c:\WINDOWS\system" Local $sFindFile = "*" Local $sRet $aRetArray = _FindPathName($path, $sFindFile) _ArrayDisplay($aRetArray) For $i = 1 To UBound($aRetArray)-1 _ArrayDelete ($aRetArray,$i) Next $aRetArray = _FindPathName($path, $sFindFile) _ArrayDisplay($aRetArray) Func _FindPathName($sPath, $sFindFile) Local $sSubFolderPath, $iIndex, $aFolders, $search,$aFoldersSubs $search = FileFindFirstFile($sPath & "\" & $sFindFile) $aFolders = _FileListToArray($sPath, "*", 2) While 1 $file = FileFindNextFile($search) If @error Then ExitLoop Else $sRet &= $sPath & "\" & $file & "|" EndIf WEnd FileClose($search) For $iIndex = 1 To $aFolders[0] $sSubFolderPath = $sPath & "\" & $aFolders[$iIndex] $aFoldersSubs = _FileListToArray($sSubFolderPath, "*", 2) If IsArray($aFoldersSubs) Then _FindPathName($sSubFolderPath, $sFindFile) Next Return StringSplit(StringTrimRight($sRet,1), "|") EndFunc ;==>_FindPathName Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted February 17, 2009 Moderators Share Posted February 17, 2009 tony,You are not emptying $sRet after each pass through the function. So each time you call it, you add another list to the end. Just put $sRet = "" after each call to your function.M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
Authenticity Posted February 17, 2009 Share Posted February 17, 2009 $sRet = 0 Add this after the first call and delete the _ArrayDelete loop, it's useless. Link to comment Share on other sites More sharing options...
tony Posted February 17, 2009 Author Share Posted February 17, 2009 Thanks a lot. I have been staring at the piece of code for hours without seeing the obvious. Link to comment Share on other sites More sharing options...
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now