JonasLocke Posted March 1, 2016 Share Posted March 1, 2016 Im trying to make a script that download a bulk of images. The script reads a local file with 4 information per line, separated by ";" here is the code: #include <InetConstants.au3> #include <MsgBoxConstants.au3> #include <file.au3> global $lcdx lerArquivo() func lerArquivo() $list = FileOpenDialog("Pesquisar", @ScriptDir&"\", "Todos os arquivos (*.*)", 1+4) _FileReadToArray($list, $lcdx, Default, ";") For $i = 1 to $lcdx[0][0] $sCard = String($lcdx[$i][0]) $sEdicao = String($lcdx[$i][1]) $sNCarta = String($lcdx[$i][2]) $sURL = String($lcdx[$i][3]) downloadcard($sCard,$sURL,$sEdicao) sleep(150) Next EndFunc Func downloadcard($a,$b,$c) $sFilePath = "C:\xmage-project\" $sFilePath = $sFilePath & $c DirCreate($sFilePath) $sFilePath = $sFilePath & "\" & $a &".full.jpg" Sleep(50) InetGet($b,$sFilePath,$INET_FORCERELOAD,$INET_DOWNLOADWAIT) EndFunc the local file looks like this: Bane of Bala Ged;BFZ;1;http://magiccards.info/scans/en/BFZ/1.jpg Blight Herder;BFZ;2;http://magiccards.info/scans/en/BFZ/2.jpg The 2 functions work separately, but when I put them together nothing happens, the folder is created but the images are not shown Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted March 1, 2016 Moderators Share Posted March 1, 2016 JonasLocke, Looking at those filenames makes me think this is for a game. Have you read the Forum rules? Does this question fall under the "no game interaction" prohibition? 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...
JonasLocke Posted March 1, 2016 Author Share Posted March 1, 2016 Its about a game, but its about downloading images, nothing releated to macroing or cheat. Im building this code to correct an error the client had to download images on windows 10 Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted March 1, 2016 Moderators Share Posted March 1, 2016 (edited) JonasLocke, Quote Im building this code to correct an error the client had to download images on windows 10 That sounds like game interaction to me - so this thread is now locked. By all means PM me and try to convince me otherwise - if you succeed I will reopen the thread. M23 Edit: A polite, reasoned PM and all is clear - the images are to be downloaded outside of the game itself. Thus the script itself merely downloads some image files and is quite legal - thread reopened. P.S. A lesson to all - we are always open to sensible discussion about moderation decisions - just approach us reasonably and you will find we are quite prepared to listen to your point of view. But please bear in mind that we will not always agree! Edited March 1, 2016 by Melba23 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...
mikell Posted March 1, 2016 Share Posted March 1, 2016 JonasLocke, Your code works nice for me - but I had to replace in the urls "BFZ" by "bfz" (lowercase) So you could try $sURL = StringLower(String($lcdx[$i][3])) Link to comment Share on other sites More sharing options...
JonasLocke Posted March 2, 2016 Author Share Posted March 2, 2016 17 hours ago, mikell said: JonasLocke, Your code works nice for me - but I had to replace in the urls "BFZ" by "bfz" (lowercase) So you could try $sURL = StringLower(String($lcdx[$i][3])) and it works! Thanx for the help! 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