shiv Posted September 24, 2013 Share Posted September 24, 2013 Hi Auto IT help & support 1. i want to run the for loop on an array 2. the array can contain multiple files 3. from those files 2 names are known to the programmer 4. i want to compare those 2 names with the array 5. and then want to store that location into some other variable, to use the location of that file i have written some code but it is not working , local $64bit = "file1_\d+\.\d+\.\d+\.\d+_x64.msi" local $32bit = "file2_\d+\.\d+\.\d+\.\d+.msi" local $imax Local $i Local $b1 Local $b2 Local $sFileList = _FileListToArray($sPath) ; $sPath is the path of a folder which has four files If IsArray($sFileList) Then $imax =UBound($sFileList) For $i = 1 to $imax - 1 $b1 = StringRegExp($sFileList[$i],$64bit,1) If @error == 0 Then $Name64bit = $sFileList[$i] EndIf $b2 = StringRegExp($sFileList[$i],$32bit,1) If @error == 0 Then $Name32bit = $sFileList[$i] EndIf Next EndIf please provide some help Link to comment Share on other sites More sharing options...
BrewManNH Posted September 24, 2013 Share Posted September 24, 2013 What isn't working? If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag GudeHow to ask questions the smart way! I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from. Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays. - ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script. - Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label. - _FileGetProperty - Retrieve the properties of a file - SciTE Toolbar - A toolbar demo for use with the SciTE editor - GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI. - Latin Square password generator Link to comment Share on other sites More sharing options...
jchd Posted September 25, 2013 Share Posted September 25, 2013 You test @error instead of $b1 and $b2. Re-read help file. Your script can surely be made more efficient but its goal is currently unclear. 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 hereRegExp tutorial: enough to get startedPCRE 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) Link to comment Share on other sites More sharing options...
shiv Posted September 25, 2013 Author Share Posted September 25, 2013 Hi All, I have executed the same code and it working, but i just want to understand that why it is printing the value for variable $b2 first. and then it prints $b1 local $64bit = "file1_\d+\.\d+\.\d+\.\d+_x64.msi" local $32bit = "file2_\d+\.\d+\.\d+\.\d+.msi" local $imax Local $i Local $b1 Local $b2 Local $sFileList = _FileListToArray($sPath) ; $sPath is the path of a folder which has four files If IsArray($sFileList) Then $imax =UBound($sFileList) For $i = 1 to $imax - 1 $b1 = StringRegExp($sFileList[$i],$64bit,1) If @error == 0 Then MsgBox(0,0,$b1[0]) $Name64bit = $sFileList[$i] EndIf $b2 = StringRegExp($sFileList[$i],$32bit,1) If @error == 0 Then MsgBox(0,0,$b2[0]) $Name32bit = $sFileList[$i] EndIf Next EndIf ; value for variable $b2 is printed first instead of $b1 , however it should first print $b1 and then $b2 ; can you please explain why it happening so Link to comment Share on other sites More sharing options...
jchd Posted September 25, 2013 Share Posted September 25, 2013 What makes you believe it does $b2 before $b1? Hint: think about the order of files from different perspectives! 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 hereRegExp tutorial: enough to get startedPCRE 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) Link to comment Share on other sites More sharing options...
mikell Posted September 25, 2013 Share Posted September 25, 2013 Do you really need these variables $b1 and $b2 ? Local $Name64bit, $Name32bit If IsArray($sFileList) Then For $i = 1 to $sFileList[0] If StringRegExp($sFileList[$i], $64bit) Then $Name64bit = $sFileList[$i] If StringRegExp($sFileList[$i], $32bit) Then $Name32bit = $sFileList[$i] Next EndIf Link to comment Share on other sites More sharing options...
shiv Posted September 25, 2013 Author Share Posted September 25, 2013 I used your code but got error <variable used without being declared>, however all variables are declared Link to comment Share on other sites More sharing options...
shiv Posted September 25, 2013 Author Share Posted September 25, 2013 Oh I am sorry, its working thanks a lot 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