distancetime Posted March 1, 2010 Share Posted March 1, 2010 How come this code does not work? The text box displays the correct path and file. I can also place the exact path and file into the the same varaible and it works. Please help. Thanks,sort_help.au3 Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted March 1, 2010 Moderators Share Posted March 1, 2010 distancetime,Welcome to the Autoit forum. A good start - some code to work on and a clear question. I wish some other newcomers would do the same.2 problems with the code as it stands:1. You have the wrong syntax for ShellExecute. Look carefully at the second Help file example - you need:ShellExecute($sFileToPrint, "" , "C:\Program Files\AutoIt3\Examples", "open") 2. You have a @CRLF at the end of the filename you get from DOS which is confusing AutoIt. You need to strip that with StringStripWS before passing the string to the print function:Local $sFileToPrint = $sPath & StringStripWS($sLastFile, 3)Top Tip: Always use ConsoleWrite with leading and trailing characters on a recalcitrant filename to check if there is unwanted whitespace - like this:$Correct = "C:\Program Files\AutoIt3\Examples\_ReadMe_.txt" ConsoleWrite("-" & $sFileToPrint & "-" & @CRLF & "-" & $Correct & "-" & @CRLF)That made the additional @CRLF immediately obvious. By the way, you can use the "print" verb directly in the ShellExecute command - it might shorten the script a bit. 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...
distancetime Posted April 14, 2010 Author Share Posted April 14, 2010 Melba, Thanks for the help. It works. 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