Sign in to follow this  
Followers 0
AlexChernavsky

Problem with selecting a file through FileOpenDialog

4 posts in this topic

I'm having a strange issue with using FileOpenDialog to select a file. Everything works fine the first time you select a file. But if you change your mind (while the program is still running) and want to select a different file, nothing changes. The program is "stuck" with the first selection you made, and nothing you do will change that selection in the input box.

I pulled the function below out of my program, and I had to modify it a bit to make it run on its own. But it's basically the same as the original.

To see the problem in action, click on either of the two buttons that say, "Choose other file...". Select any file from the list. Click on the "Open" button. So far, so good. Now, repeat the process again. See? The original file you selected is still listed in the input box.

I'm stumped by this one.

Thanks in advance.

-- Alex Chernavsky in Rochester, NY

#include <GUIConstantsEx.au3>
#include <ButtonConstants.au3>
#include <EditConstants.au3>

_ConfirmParametersWithUser("test.txt", "test2.txt", 16, 1000, 3000, "InputFile", "WriteToFile", "Testing")

Func _ConfirmParametersWithUser($sInputFileName, $sOutputFileName, $iNumberOfProblems, $iMinTime, $iMaxTime, $sSourceOfProblems, $sOutputType, $sCondition)
Local $hSetParametersForm ; Handle to GUI
Local $hRadio1
Local $hRadio2
Local $hRadio3
Local $hRadio4
Local $hInputFileName
Local $hChooseInputFile
Local $hOutputFileName
Local $hChooseOutputFile
Local $hTrialsPerSession
Local $hTime1
Local $hTime2
Local $hCondition
Local $hExitButton
Local $hHelpButton
Local $hContinueButton
Local $nMsg
Local $sTemp

$sInputFileName = @ScriptDir & "\" & $sInputFileName
$sOutputFileName = @ScriptDir & "\" & $sOutputFileName
$hSetParametersForm = GUICreate("Reaction Time Program: Set Parameters", 580, 578, -1, -1)
GUICtrlCreateLabel("Reaction Time Program", 175, 3, 205, 23)
GUICtrlSetFont(-1, 14, 400, 0, "MS Sans Serif")

GUICtrlCreateLabel("Based on work by Seth Roberts", 200, 29, 155, 15)
GUICtrlSetFont(-1, 8, 400, 0, "MS Sans Serif")
;GUICtrlSetBkColor(-1, 0xCCFFCC) ; Give it a green background to make it easier to position it (it's a temporary thing to help design the GUI)

GUICtrlCreateGroup("Source of arithmetic problems: ", 40, 50, 528, 105)
GUICtrlSetFont(-1, 12, 400, 0, "MS Sans Serif")
GUIStartGroup()
$hRadio1 = GUICtrlCreateRadio("Use problems included with source code", 64, 80, 257, 17)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
If $sSourceOfProblems = "SourceCode" Then
GUICtrlSetState(-1, $GUI_CHECKED)
EndIf
$hRadio2 = GUICtrlCreateRadio("Read problems from input file: ", 64, 104, 280, 17)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
If $sSourceOfProblems = "InputFile" Then
GUICtrlSetState(-1, $GUI_CHECKED)
EndIf
$hInputFileName = GUICtrlCreateInput($sInputFileName & " ", 81, 124, 380, 20)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
GUICtrlSetState($hInputFileName, $GUI_FOCUS)
Send("{end}") ; This line and the one before it are necessary to right-align the text (useful in case of long path names, so that the left part gets cut off, not the right part)
$hChooseInputFile = GUICtrlCreateButton("Choose other file...", 465, 121)
If $sSourceOfProblems = "InputFile" Then
GUICtrlSetState($hChooseInputFile, $GUI_ENABLE)
Else
GUICtrlSetState($hChooseInputFile, $GUI_DISABLE)
EndIf

GUICtrlCreateLabel("Number of trials (problems) presented per testing session:", 48, 167, 345, 20)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
$hTrialsPerSession = GUICtrlCreateInput($iNumberOfProblems, 397, 165, 30, 22, BitOR($GUI_SS_DEFAULT_INPUT, $ES_NUMBER))
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")

GUICtrlCreateGroup("Random delay between trials (in milliseconds): ", 40, 198, 349, 89)
GUICtrlSetFont(-1, 12, 400, 0, "MS Sans Serif")

GUICtrlCreateLabel("Minimum delay:", 64, 224, 97, 20)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
GUICtrlCreateLabel("Maximum delay :", 64, 256, 104, 20)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
$hTime1 = GUICtrlCreateInput($iMinTime, 176, 224, 41, 21, BitOR($GUI_SS_DEFAULT_INPUT, $ES_NUMBER))
$hTime2 = GUICtrlCreateInput($iMaxTime, 176, 256, 41, 21, BitOR($GUI_SS_DEFAULT_INPUT, $ES_NUMBER))

GUICtrlCreateGroup("Upon completion of the testing session: ", 40, 296, 528, 110)
GUICtrlSetFont(-1, 12, 400, 0, "MS Sans Serif")
$hRadio3 = GUICtrlCreateRadio("Display summary and append data to output file: ", 64, 325, 422, 17)
If $sOutputType = "WriteToFile" Then GUICtrlSetState(-1, $GUI_CHECKED)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
$hOutputFileName = GUICtrlCreateInput($sOutputFileName & " ", 81, 350, 380, 20)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
GUICtrlSetState($hOutputFileName, $GUI_FOCUS)
Send("{end}") ; This line and the one before it are necessary to right-align the text (useful in case of long path names, so that the left part gets cut off, not the right part)
$hChooseOutputFile = GUICtrlCreateButton("Choose other file...", 465, 347)
If $sOutputType = "WriteToFile" Then
GUICtrlSetState($hChooseOutputFile, $GUI_ENABLE)
Else
GUICtrlSetState($hChooseOutputFile, $GUI_DISABLE)
EndIf
$hRadio4 = GUICtrlCreateRadio("Display summary but do not write data to output file", 64, 379, 320, 17)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")
If $sOutputType = "DisplayOnly" Then GUICtrlSetState(-1, $GUI_CHECKED)
GUICtrlCreateGroup("Please describe the condition for this test:", 40, 417, 520, 90)
GUICtrlSetFont(-1, 12, 400, 0, "MS Sans Serif")
$hCondition = GUICtrlCreateInput("", 54, 447, 500, 20)
GUICtrlCreateLabel("Example: ""45 mins. after taking 3 tablespoons of flaxseed oil""", 56, 477, 550, 28)
GUICtrlSetFont(-1, 10, 400, 0, "MS Sans Serif")

$hHelpButton = GUICtrlCreateButton("Help", 272, 534, 75, 25)
$hContinueButton = GUICtrlCreateButton("Continue", 382, 534, 75, 25, $BS_DEFPUSHBUTTON)
$hExitButton = GUICtrlCreateButton("Exit Program", 488, 534, 75, 25)
GUICtrlSetState($hCondition, $GUI_FOCUS) ; Set the focus to the "Condition" input box
GUISetState(@SW_SHOW)

While 1
$nMsg = GUIGetMsg()
Switch $nMsg
Case $GUI_EVENT_CLOSE
Exit
Case $hRadio1
GUICtrlSetState($hChooseInputFile, $GUI_DISABLE)
Case $hRadio2
GUICtrlSetState($hChooseInputFile, $GUI_ENABLE)
Case $hRadio3
GUICtrlSetState($hChooseOutputFile, $GUI_ENABLE)
Case $hRadio4
GUICtrlSetState($hChooseOutputFile, $GUI_DISABLE)
Case $hChooseInputFile
$sTemp = FileOpenDialog("Choose input file", @ScriptDir & "\", "All files (*.*)", 1 + 2)
If $sTemp Then
$sInputFileName = $sTemp
$hInputFileName = GUICtrlSetData($hInputFileName, $sInputFileName & " ")
EndIf
GUICtrlSetState($hCondition, $GUI_FOCUS) ; Return focus to the "Continue" button
Case $hChooseOutputFile
$sTemp = FileOpenDialog("Choose output file", @ScriptDir & "\", "All files (*.*)", 2 + 8)
If $sTemp Then
$sOutputFileName = $sTemp
$hOutputFileName = GUICtrlSetData($hOutputFileName, $sOutputFileName & " ")
EndIf
GUICtrlSetState($hCondition, $GUI_FOCUS) ; Return focus to the "Continue" button
Case $hHelpButton
ShellExecute("http://www.astrocyte-design.com/reaction-time/index.html")
Case $hContinueButton
ExitLoop
Case $hExitButton
Exit
EndSwitch
WEnd
If (BitAND(GUICtrlRead($hRadio1), $GUI_CHECKED) = $GUI_CHECKED) Then
$sSourceOfProblems = "SourceCode"
Else
$sSourceOfProblems = "InputFile"
EndIf
$iNumberOfProblems = GUICtrlRead($hTrialsPerSession)
$iMinTime = GUICtrlRead($hTime1)
$iMaxTime = GUICtrlRead($hTime2)
If (BitAND(GUICtrlRead($hRadio3), $GUI_CHECKED) = $GUI_CHECKED) Then
$sOutputType = "WriteToFile"
Else
$sOutputType = "DisplayOnly"
EndIf
$sCondition = GUICtrlRead($hCondition)
$sCondition = StringReplace($sCondition, ",", "") ; Remove any commas that the user happens to enter (they interfere with the CSV file format)
GUIDelete($hSetParametersForm)
EndFunc ;==>_ConfirmParametersWithUser

Share this post


Link to post
Share on other sites



The problem is due to you redefining your variable. In this section:

Case $hChooseInputFile
$sTemp = FileOpenDialog("Choose input file", @ScriptDir & "", "All files (*.*)", 1 + 2)
If $sTemp Then
$sInputFileName = $sTemp
$hInputFileName = GUICtrlSetData($hInputFileName, $sInputFileName & " ")
EndIf
GUICtrlSetState($hCondition, $GUI_FOCUS) ; Return focus to the "Continue" button

you don't need to say $hInputFileName = GUICtrlSetData($hInputFileName, $sInputFileName & " "). Instead, just set the data:

Case $hChooseInputFile
$sTemp = FileOpenDialog("Choose input file", @ScriptDir & "", "All files (*.*)", 1 + 2)
 If $sTemp Then
  $sInputFileName = $sTemp
  GUICtrlSetData($hInputFileName, $sInputFileName & " ")
 EndIf
GUICtrlSetState($hCondition, $GUI_FOCUS) ; Return focus to the "Continue" button
1 person likes this

√-1 2^3 ∑ π, and it was delicious!

Share this post


Link to post
Share on other sites

Thank you, JLogan3o13. I think that did the trick. I appreciate your help.

Share this post


Link to post
Share on other sites

Glad to be of service :)


√-1 2^3 ∑ π, and it was delicious!

Share this post


Link to post
Share on other sites

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 account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0

  • Similar Content

    • IAMK
      By IAMK
      Hello,
       
      I have a script which uses a GUI with two buttons. It gets shown and hidden., and pressing the buttons (including their hotkeys) highlights the button, making you able to press enter the next time.
      However, I don't want that.
       
      I've tried using WinActivate() to make enter do things on another program, but the enter is still tied to the AutoIT GUI (which still has the button highlighted).
       
      Here's the important part of my code:
      $MY_GUI = GUICreate("Check.exe", 400, 300, ((@DesktopWidth / 2) - 200), -500) WinSetOnTop($MY_GUI, "", 1) ;Keeps the window on the top. $Button1 = GUICtrlCreateButton("Pass (Ctrl+P)", 40, 250, 120, 40, $WS_GROUP) GUICtrlSetFont(-1, 12, 400, 0, "MS Sans Serif") $Button2 = GUICtrlCreateButton("Fail (Ctrl+F)", 240, 250, 120, 40, $WS_GROUP) GUICtrlSetFont(-1, 12, 400, 0, "MS Sans Serif") $GUIText = GUICtrlCreateLabel("", 20, 10, 360, 230) GUICtrlSetFont(-1, 13, 400, 0, "MS Sans Serif") GUICtrlSetData($GUIText, Execute($stepMessage[$step])) $showGUI = "True" GUISetState(@SW_SHOW) While($showGUI = "True") $buttonPressed = GUIGetMsg() Switch $buttonPressed Case $Button1 ;Pass $pass = "True" $buttonPressed = 0 $showGUI = "False" GUISetState(@SW_HIDE) Case $Button2 ;Fail $buttonPressed = 0 $showGUI = "False" GUISetState(@SW_HIDE) While($message = "") $message = InputBox("Check.exe", "Please enter the reason the step failed." & @CRLF, "", "", 300, 200, ((@DesktopWidth / 2) - 150), -400) WEnd Return $message EndSwitch WEnd Local $pass = "False" ;Whether "Pass" or "Fail" was pressed. Local $failLog = xxxfunc($pass) ;Returns a message to log if the step failed. ##THIS CALLS THE CODE SNIPPET ABOVE## FileWrite($file, _Now() & ": Step " & $step & getStatus($pass) & "and took " & Ceiling(TimerDiff($timer) / 1000) & " seconds." & @CRLF) If($failLog <> "") Then FileWrite($file, "Logged comment: " & $failLog & @CRLF) EndIf WinActivate("[TITLE:ProgramName]") ;Hotkeys: HotKeySet("^{p}", "pressPass") HotKeySet("^{f}", "pressFail") Func pressPass() $buttonPressed = $Button1 EndFunc Func pressFail() $buttonPressed = $Button2 EndFunc  
      The GUI's button/hotkey pressing works correctly.
      The same WinActivate code works in another script of mine.
      Together, WinActivate is not take "Active" away from the GUI's last-pressed button. How do I fix this?
       
      Thank you in advance.
    • fatpig
      By fatpig
      Dear AutoIT Community.
      I am currently working on a program that will display lots of images in a scroll down GUI.
      I create the GUI based on the number of images, which works fine.
      But all images I place using GuiCtrlCreatePic above a threshold of around 32000 pixels (cant say exactly) are placed at negative locations and finally will reach 0 again
      and will then populate on top of the images placed there before.
      For $i = 1 To $Images[0] ; Get picture dimensions $hImage = _GDIPlus_ImageLoadFromFile($ImgRep & $Images[$i]) $iX = _GDIPlus_ImageGetWidth($hImage) $iY = _GDIPlus_ImageGetHeight($hImage) _GDIPlus_ImageDispose($hImage) ; Resize & place images in GUI $Pic[$i] = GUICtrlCreatePic($ImgRep & $Images[$i], $Spacer, Round($yPos, 0), $MaxImgWidth, $MaxImgWidth * ($iY / $iX)) $yPos = $yPos + $MaxImgWidth * ($iY / $iX) + $Spacer $log = $log & "|" & $yPos $currentlut = $Images[1] Next I have found a limit at 32767 pixels for some GUI elements when googling around.
      Can someone tell me how to circumvent that?
      thanks!
      cheers
      Felix
    • Shirdish_chakravarthi
      By Shirdish_chakravarthi
      Hello Guys this is my gui and i need the numeric value typed in the highlighted  inputbox. if this value is greater than 1 i need to perform some operations
      \
       
      Here is the summary about that field which i got from wininfo tool

      i tried writing the code as shown below.. but the msgbox is popping up empty value..when the same code is pasted under some button action i am able to access the value present in the field..but when placed alone as written in the attached script i am unable to fetch it... 

      here i am using the valu present in the field

      can you guys tell me what's wrong??? or how can i get that value?
       
      My_GUI.au3

    • Neonovaz
      By Neonovaz
      Hello
       
      Is there anyway to store word documents in Autoit GUI? For example I have a instruction sheet that I want to bundle up with the exe.

      So a user simply clicks the icon and the stored document will launch  (Something like how you can add objects like excel sheets in word documents )

      (I Know we can launch word files from script directory)

       
    • hemichallenger
      By hemichallenger
      Not sure if this is possible or if someone has already tackled this. Is it possible to embed PowerShell into a GUI? Autoit is great for quick function but sometime I need to run a command thru PowerShell. It be great to have PowerShell console embedded into the gui.