youtuber Posted June 3, 2019 Share Posted June 3, 2019 I want to display _ArrayDisplay after the For loop #include <File.au3> #include <Array.au3> Local $FolderList = @ScriptDir Local $len = 259 Local $aRec = "" Local $AllFilesFolder $AllFilesFolder = _FileListToArrayRec($FolderList, "*", $FLTAR_FILES, $FLTAR_RECUR, $FLTAR_SORT, $FLTAR_FULLPATH) If IsArray($AllFilesFolder) Then For $i = 1 To $AllFilesFolder[0] If StringLen($AllFilesFolder[$i]) > $len Then $aRec &= $AllFilesFolder[$i] & @CRLF EndIf Next ConsoleWrite($aRec & @CRLF) ;No problem MsgBox(48,"Files folders length is more than 259", $aRec) ;No problem _ArrayDisplay($aRec, " Files folders length is more than 259") ;problem! EndIf Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted June 3, 2019 Moderators Share Posted June 3, 2019 youtuber, As $aRec is not an array but a string, why would you expect _ArrayDisplay to work? If you were to use StringSplit on $aRec before the _ArrayDisplay call then you would then have an array to pass to the 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...
youtuber Posted June 3, 2019 Author Share Posted June 3, 2019 I added but failed $AllFilesFolder = _FileListToArrayRec($FolderList, "*", $FLTAR_FILES, $FLTAR_RECUR, $FLTAR_SORT, $FLTAR_FULLPATH) If IsArray($AllFilesFolder) Then For $i = 1 To $AllFilesFolder[0] If StringLen($AllFilesFolder[$i]) > $len Then $aRec &= StringSplit($AllFilesFolder[$i],@CRLF) EndIf Next ConsoleWrite($aRec & @CRLF) ;No problem MsgBox(48,"Files folders length is more than 259", $aRec) ;No problem _ArrayDisplay($aRec, " Files folders length is more than 259") ;problem! EndIf Link to comment Share on other sites More sharing options...
Subz Posted June 3, 2019 Share Posted June 3, 2019 (edited) As Melba23 pointed out $aRec is a string,, use something like _ArrayAdd to create an array or just use the $AllFilesFolder Array for example: #include <File.au3> #include <Array.au3> Local $FolderList = @ScriptDir Local $len = 259 Local $AllFilesFolder $AllFilesFolder = _FileListToArrayRec($FolderList, "*", $FLTAR_FILES, $FLTAR_RECUR, $FLTAR_SORT, $FLTAR_FULLPATH) If IsArray($AllFilesFolder) Then For $i = $AllFilesFolder[0] To 1 Step - 1 If StringLen($AllFilesFolder[$i]) <= $len Then _ArrayDelete($AllFilesFolder, $i) Next $AllFilesFolder[0] = UBound($AllFilesFolder) - 1 MsgBox(48,"Files folders length is more than 259", _ArrayToString($AllFilesFolder, @CRLF, 1)) ;No problem _ArrayDisplay($AllFilesFolder, " Files folders length is more than 259") ;problem! EndIf Edited June 3, 2019 by Subz youtuber 1 Link to comment Share on other sites More sharing options...
youtuber Posted June 3, 2019 Author Share Posted June 3, 2019 Does the search on file names take longer, do you have a faster suggestion? Link to comment Share on other sites More sharing options...
Nine Posted June 4, 2019 Share Posted June 4, 2019 I believe it is the multiple _ArrayDelete that's taking more time. Just use your code with a StringSplit like this : #include <File.au3> #include <Array.au3> Local $FolderList = @ScriptDir Local $len = 259 Local $aRec = "" Local $AllFilesFolder $AllFilesFolder = _FileListToArrayRec($FolderList, "*", $FLTAR_FILES, $FLTAR_RECUR, $FLTAR_SORT, $FLTAR_FULLPATH) If IsArray($AllFilesFolder) Then For $i = 1 To $AllFilesFolder[0] If StringLen($AllFilesFolder[$i]) > $len Then $aRec &= $AllFilesFolder[$i] & @LF EndIf Next ConsoleWrite($aRec) ;No problem MsgBox(48,"Files folders length is more than 259", $aRec) ;No problem $aRec = StringSplit ($aRec, @LF) _ArrayDisplay($aRec, " Files folders length is more than 259") ;NO MORE problem! EndIf youtuber 1 “They did not know it was impossible, so they did it” ― Mark Twain Spoiler Block all input without UAC Save/Retrieve Images to/from Text Monitor Management (VCP commands) Tool to search in text (au3) files Date Range Picker Virtual Desktop Manager Sudoku Game 2020 Overlapped Named Pipe IPC HotString 2.0 - Hot keys with string x64 Bitwise Operations Multi-keyboards HotKeySet Recursive Array Display Fast and simple WCD IPC Multiple Folders Selector Printer Manager GIF Animation (cached) Screen Scraping Multi-Threading Made Easy 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