Jump to content
anub13

Combo Box Read Value Bugging

Recommended Posts

anub13

Hi,

I am having difficulties wondering why my code bugging like this,
the symptom : I have a form with combo box, and details in bottom of the app, when user choose another option the details changes immediately but when I go to details, it goes up and down endlessly

here is the code

 

Global $cChkPatrol, $YES= "YES", $NO = "NO"

if GUICtrlRead($combo1) = "YES" Then
    $cChkPatrol = $YES
Else
    $cChkPatrol = $NO
EndIf

func StartForm()

    GUICtrlSetData($txtInfo,"ADDITIONAL INFORMATION THAT WILL BE SENT WITH THE MESSAGE:" & @CRLF & asdfgSystemInfo())
    GUISetState(@SW_SHOW)
    AdlibRegister("_CheckPatrol")
    While 1
        $nMsg = GUIGetMsg()
        Select
            case $nMsg = $txtEmailAddress
                GUICtrlSetData($txtInfo,"ADDITIONAL INFORMATION THAT WILL BE SENT WITH THE MESSAGE:" & @CRLF & asdfgSystemInfo())
            case $nMsg = $txtPhone
                GUICtrlSetData($txtInfo,"ADDITIONAL INFORMATION THAT WILL BE SENT WITH THE MESSAGE:" & @CRLF & asdfgSystemInfo())

            case $nMsg = $txtFullName
                GUICtrlSetData($txtInfo,"ADDITIONAL INFORMATION THAT WILL BE SENT WITH THE MESSAGE:" & @CRLF & asdfgSystemInfo())

            case $nMsg = $cChkPatrol
                GUICtrlSetData($txtInfo,"ADDITIONAL INFORMATION THAT WILL BE SENT WITH THE MESSAGE:" & @CRLF & asdfgSystemInfo())
            case $nMsg = $cmdScreenShot
                $gScreenShotFile =  SaveScreenShot()

            case $nMsg = $cmdSubmit
                if ValidateForm() then
                    if asdfgSubmit() Then
                        UpdateStatus("Your request has been sent successfully")
                        MsgBox(0,"Help Requested","Your request has been sent successfully")
                        ExitLoop
                    Else

                        MsgBox(0,"Help Failed","Something is wrong you request was not sent, please try again")

                    Endif
                endIf
            case $nMsg = $cmdAttach
                $gAttachFile = AttachFile()

            Case $nMsg = $GUI_EVENT_CLOSE
                ExitLoop
        EndSelect
    WEnd
    AdlibUnRegister("_CheckPatrol")
EndFunc

func asdfgSystemInfo()

    dim $asdfgInfo
    $asdfgInfo = ""  & @CRLF

    if $isAdminVersion=1 then
        $asdfgInfo = $asdfgInfo & @CRLF & " Ticket Tags:" & @CRLF
        $asdfgInfo = $asdfgInfo & "~#userphone='" & GUICtrlRead($txtPhone) & "'" & @CRLF
        $asdfgInfo = $asdfgInfo & "~#submitterphone='" & GUICtrlRead($txtPhone) & "'" & @CRLF
        dim $MachineID
        $MachineID = KaseyaIni($gKIniFile,"SERVER COMMUNICATIONS","User_Name","NotFound")
        $asdfgInfo = $asdfgInfo & "~#machineid='" & $MachineID & "'" & @CRLF

        dim $OrgGroups
        $OrgGroups= StringSplit($MachineID,".")

        dim $OrgGroupIndex
        $OrgGroupIndex = UBound($OrgGroups) - 1
        $asdfgInfo = $asdfgInfo & "~#organization='" & $OrgGroups[$OrgGroupIndex] & "'" & @CRLF

        dim $EmailUser
        $EmailUser = GUICtrlRead($txtEmailAddress)
        $EmailUser = StringMid($EmailUser,1,StringInStr($EmailUser,"@") - 1)  & @CRLF  & @CRLF


        $asdfgInfo = $asdfgInfo & "~#username='" & $EmailUser & "'" & @CRLF

        ;dim
        ;Call("isChecked")
        ;$categ = GUICtrlRead($chkPatrol)
        ;$asdfgInfo = $asdfgInfo & "-#category='" & $categ & "'" & @CRLF

    EndIf

    $asdfgInfo = $asdfgInfo & "Email:" & GUICtrlRead($txtEmailAddress) & @CRLF & @CRLF
    $asdfgInfo = $asdfgInfo & "Full Name:" & GUICtrlRead($txtFullName) & @CRLF & @CRLF
    $asdfgInfo = $asdfgInfo & "Phone:" & GUICtrlRead($txtPhone) & @CRLF & @CRLF


    $asdfgInfo = $asdfgInfo & "DateTime:" & _Now() & @CRLF
    $asdfgInfo = $asdfgInfo & "MachineName:" & @ComputerName & @CRLF
    $asdfgInfo = $asdfgInfo & "UserName:" & @UserName & @CRLF
    $asdfgInfo = $asdfgInfo & "OS:" & @OSType & " " & @OSVersion & @CRLF
    $asdfgInfo = $asdfgInfo & "LogonDomain:" & @OSType & " " & @LogonDomain & @CRLF
    $asdfgInfo = $asdfgInfo & "LogonDNSDomain:" & @OSType & " " & @LogonDNSDomain & @CRLF
    $asdfgInfo = $asdfgInfo & "OS:" & @OSType & " " & @OSVersion & @CRLF

    $asdfgInfo = $asdfgInfo & "IP Address:" & @IPAddress1  & @CRLF
    $asdfgInfo = $asdfgInfo & "Public IP Address:" & GetPublicIP() &@CRLF
    ;$asdfgInfo = $asdfgInfo & "Public IP Address:" & _GetIP () &@CRLF

    $asdfgInfo = $asdfgInfo & "OS:" & NetAdapterInfo()

    $asdfgInfo = $asdfgInfo & "KaseyaInstalled:" & KaseyaInstalled() & @CRLF
    $asdfgInfo = $asdfgInfo & "KaseyaRunning:" & KaseyaRunning() & @CRLF
    $asdfgInfo = $asdfgInfo & "KaseyaAgentGUID:" & KaseyaIni($gKIniFile,"SERVER COMMUNICATIONS","Agent_Guid","NotFound") & @CRLF
    $asdfgInfo = $asdfgInfo & "asdfgPaid:" & isasdfgIDPaid() & @CRLF

    if $isAdminVersion=0 then
        $asdfgInfo = $asdfgInfo & @CRLF & " Ticket Tags:" & @CRLF
        $asdfgInfo = $asdfgInfo & "~userphone='" & GUICtrlRead($txtPhone) & "'" & @CRLF
        $asdfgInfo = $asdfgInfo & "~submitterphone='" & GUICtrlRead($txtPhone) & "'" & @CRLF

        dim $MachineID
        $MachineID = KaseyaIni($gKIniFile,"SERVER COMMUNICATIONS","User_Name","NotFound")
        $asdfgInfo = $asdfgInfo & "~machineid='" & $MachineID & "'" & @CRLF

        dim $OrgGroups
        $OrgGroups= StringSplit($MachineID,".")

        dim $OrgGroupIndex
        $OrgGroupIndex = UBound($OrgGroups) - 1
        $asdfgInfo = $asdfgInfo & "~organization='" & $OrgGroups[$OrgGroupIndex] & "'" & @CRLF

        dim $EmailUser
        $EmailUser = GUICtrlRead($txtEmailAddress)
        $EmailUser = StringMid($EmailUser,1,StringInStr($EmailUser,"@") - 1)

        $asdfgInfo = $asdfgInfo & "~username='" & $EmailUser & "'" & @CRLF

        dim $categ
        $categ = $chkPatrol
        $asdfgInfo = $asdfgInfo & "~category='" & $categ & "'" & @CRLF

        dim $track
        $track = $cChkPatrol
        $asdfgInfo = $asdfgInfo & "~test='" & $cChkPatrol & "'" & @CRLF

        $asdfgInfo = $asdfgInfo & "~afterhour='" & $AfterHour & "'" & @CRLF
    endif

    Return $asdfgInfo

endFunc

 

Yes it was same app as previous post in the forum I asked, I also having experience this happening before but I fail to understand the reason why this is happening. Thanks in advance.

ButtonSupport - Copy.au3

Share this post


Link to post
Share on other sites
BrewManNH
35 minutes ago, anub13 said:

when user choose another option the details changes immediately but when I go to details, it goes up and down endlessly

What exactly do you mean by this? You're only reading the combobox in one place, and then you never read from it again.


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 Gude
How 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

Share this post


Link to post
Share on other sites
anub13
6 minutes ago, BrewManNH said:

What exactly do you mean by this? You're only reading the combobox in one place, and then you never read from it again.

please find attached files for the img

when I change the option to YES, ADDITIONAL INFORMATION ........

when the cursor goes to that area, and scroll it down, always back to up I can't go down at all.

Capture.PNG

Edited by anub13
typos

Share this post


Link to post
Share on other sites
BrewManNH

There's no way for me to run this because of the internet connection test you have in it, and I'm not going to go through the effort to make it runnable.

 


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 Gude
How 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

Share this post


Link to post
Share on other sites
anub13
5 minutes ago, BrewManNH said:

There's no way for me to run this because of the internet connection test you have in it, and I'm not going to go through the effort to make it runnable.

 

Here I edited the code, now should be running without internet connection.

ButtonSupport - Copy.au3

Share this post


Link to post
Share on other sites
BrewManNH

I still had to heavily modify it to get it to run.

The problem is caused in 2 places.

  1. Your function _CheckPatrol is constantly reloading the information in the edit box, for no good reason that I can see.
  2. You have a line in your GUI message handler looking at the variable $cChkPatrol, this isn't a control on your GUI, it's just a variable that contains Yes or No, and it's getting processed every time the loop runs. Delete this section, as it's unnecessary as it's written.
    case $nMsg = $cChkPatrol
        GUICtrlSetData($txtInfo,"ADDITIONAL INFORMATION THAT WILL BE SENT WITH THE MESSAGE:" & @CRLF & asdfgSystemInfo())

    You might want to consider moving the GUICtrlSetData inside your If statements in _CheckPatrol and see if that eliminates some of the excessive rewriting of the edit box.


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 Gude
How 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

Share this post


Link to post
Share on other sites
anub13
17 minutes ago, BrewManNH said:

I still had to heavily modify it to get it to run.

The problem is caused in 2 places.

  1. Your function _CheckPatrol is constantly reloading the information in the edit box, for no good reason that I can see.
  2. You have a line in your GUI message handler looking at the variable $cChkPatrol, this isn't a control on your GUI, it's just a variable that contains Yes or No, and it's getting processed every time the loop runs. Delete this section, as it's unnecessary as it's written.
    case $nMsg = $cChkPatrol
        GUICtrlSetData($txtInfo,"ADDITIONAL INFORMATION THAT WILL BE SENT WITH THE MESSAGE:" & @CRLF & asdfgSystemInfo())

    You might want to consider moving the GUICtrlSetData inside your If statements in _CheckPatrol and see if that eliminates some of the excessive rewriting of the edit box.

sorry about it, I thought I ran it well in here

OK will try as you said and be right back.

Share this post


Link to post
Share on other sites
anub13

@BrewManNH

Still need some time... at the moment still figuring out to solve it.:D

Thanks for responding.

EDIT:

Sorry for double post.

EDIT 2: 

@BrewManNH

just complete the code, put simple code 

Func checkCombo()
    if GUICtrlRead($combo1) = "YES" Then
        $chkTrack = $YES
    Else
        $chkTrack = $NO
    EndIf
EndFunc

and called it on here

case $nMsg = $combo1
checkCombo()
GUICtrlSetData($txtInfo,"ADDITIONAL INFORMATION THAT WILL BE SENT WITH THE MESSAGE:" & @CRLF & TCDSystemInfo())

:) thanks for your help

Edited by anub13
typos

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

  • Similar Content

    • nooneclose
      By nooneclose
      My program has to first search for names in Column D that do not match up with column C. I got that search to work using arrays. It was slow and I could not figure out how to delete them so I just manually put coded the names that do not belong. I found their cell location but I do not know how to store that location and delete it.
      This is what I have so far.
      Local $NameToDelete1[6]  = _Excel_RangeFind($OpenWorkbook, "Smith, Bill") _ArrayDisplay($NameToDelete1, "Excel UDF: _Excel_RangeFind Example 1", "", 0, "|", "Sheet|Name|Cell|Value|Formula|Comment") _Excel_RangeDelete($OpenWorkbook.ActiveSheet, $NameToDelete1[2], $xlShiftUp)  
      Please help, I wanted to have this program done yesterday but I did not see this problem until yesterday. 
    • TrashBoat
      By TrashBoat
      So Im trying to make a simple 2d game and make some sort of collision detection so why not to make a 2 dimensional array but i have no clue how  to write it in multiple lines
      Global $map[5,5] = [0,0,0,0,0 _ [0,0,0,0,0 _ [0,0,0,0,0 _ [0,0,0,0,0 _ [0,0,0,0,0] something like this but it doesn't work
    • Zein
      By Zein
      #include "..\Include\Array.au3" #include "..\Include\File.au3" #include "..\Include\AutoItConstants.au3" Local $aRetArray Local $sFilePath = "n.csv" _FileReadToArray($sFilePath, $aRetArray, ",") ; _FileReadToArray($sFilePath, $aRetArray, $FRTA_COUNT, ",") _ArrayDisplay($aRetArray, "Original", Default, 8) The above code shows two versions of _FileReadToArray and both don't work as expected.
      The first one doesn't use the comma as a delimiter. (so I get a single column array)  I tried adding "Default" between $aRetArray and "," then it told me it had an incorrect number of parameters. 

      I looked again at the documentation:
       
      #include <File.au3> _FileReadToArray ( $sFilePath, ByRef $vReturn [, $iFlags = $FRTA_COUNT [, $sDelimiter = ""]] )
      And I with or without the flags params I should be getting a 2D array due to my file being a csv. 
      I then tried a regular flag, $FRTA_COUNT, and it tells me that I'm using a variable $FRTA_COUNT while it's not declared. Tried putting in 1 instead and it told me again, incorrect number of params. 

       
    • nooneclose
      By nooneclose
      I need to perform a subtotal in excel and I would like to automate this process using Autoit if possible like always any and all help will be greatly appreciated. 
      I can not find a good example but the two from Microsoft. Here is one of the two from msdn.microsoft.com/en-us/vba/excel-vba/articles/range-subtotal-method-excel
      I do not really understand how to translate this into AutoIt, but I gave it a try and here is what I have.
      $OpenRange      = "A1:E200" $xlSum          = -4157 $Added_Array[2] = [2, 3] $OpenRange.Subtotal("B1", $xlSum, $Added_Array, True, False, True) I just need to perform a subtotal on a range based on a header called department, and then perform a sum on the results.
    • nooneclose
      By nooneclose
      How to use _Excel_RangeSort to sort my excel file by three different headers Column A1, B1, and C1 have headers on which I want to sort by. The headers on which I want to sort are department, employee type, and name.
      I still really new to AutoIt so I do not actually know how to properly start this line or lines of code, to be honest. The example code is the best I can do.
      _Excel_RangeSort($OpenWorkbook, Default, "A1:C1", "1:1", $xlDescending, Default, $xlYes, Default, $xlSortRows) I just need to sort by those three headers in that order of department, employee type, and name, plus in descending order.
       
      any and all help would be greatly appreciated.  Thank you!
×