Jump to content
Jibberish

Error Subscript used on non-accessible variable putting data into an array

Recommended Posts

Jibberish

I am trying to add data to an array and I keep getting the error "Subscript used on non-accessible variable ".

#include <array.au3>
; Includes, etc
~
~
~
Local $aOptionsArray[3]

$aOptionsArray = CheckboxesAndRadioButtons()
_ArrayDisplay($aOptionsArray)
Func CheckboxesAndRadioButtons()
; Create Checkboxes and Radio Buttons and read the results
~
~
~
    ; Radio Buttons to Array
    ; $aOptions[0] = $bTestSelectForever
    ; $aOptions[1] = $bTestSelect3Times
    ; $aOptions[2] = $bTestSelectOnce
    
    If $bSelect1 = 1 Then
        $aOptions[0] = "True"
    Else
        $aOptions[0] = "False"  <<< This is where the error occurs.
    EndIf

    If $bSelect2 = 1 Then
        $aOptions[1] = "True"
    Else
        $aOptions[1] = "False"
    EndIf

    If $bSelect3 = 1 Then
        $aOptions[2] = "True"
    Else
        $aOptions[2] = "False"
    EndIf
    Return $aOptions
EndFunc

Is putting data into an array while in a If - Then - Else illegal?

Share this post


Link to post
Share on other sites
Jibberish

Apologies - it is defined in the Function.  Updated script below. From what I understand if I create an array in a function and then return the array I need two different variable names - one in Main and one in the function. Yes?


 
#include <array.au3>
; Includes, etc
~
~
~
Local $aOptionsArray[3]

$aOptionsArray = CheckboxesAndRadioButtons()
_ArrayDisplay($aOptionsArray)

; lots of other code
~
~
~

Func CheckboxesAndRadioButtons()

Local $aOptions

; Create Checkboxes and Radio Buttons and read the results
; all variables in this Function were assigned either at the top of the script or in the function.
~
~
~
    ; Radio Buttons to Array
    ; $aOptions[0] = $bTestSelectForever
    ; $aOptions[1] = $bTestSelect3Times
    ; $aOptions[2] = $bTestSelectOnce
    
    If $bSelect1 = 1 Then
        $aOptions[0] = "True"
    Else
        $aOptions[0] = "False"  <<< This is where the error occurs.
    EndIf

    If $bSelect2 = 1 Then
        $aOptions[1] = "True"
    Else
        $aOptions[1] = "False"
    EndIf

    If $bSelect3 = 1 Then
        $aOptions[2] = "True"
    Else
        $aOptions[2] = "False"
    EndIf
    Return $aOptions
EndFunc

 

Share this post


Link to post
Share on other sites
Jibberish

OK I figured it out. I neglected to put a [3] after my declaration.

Local @aOptions[3] solved the issue.

This is weird because usually when I declare an array in other code I don't have to add [x] as in :

Local $aArray  << has worked before

vs

Local $aArray[3]  << required for this particular script... I don't understand why!

Share this post


Link to post
Share on other sites
kylomas

Jibberish,

An alternative approach.  Nota Bene, for this to work the controls must be defined consecutively.  This is handy for a large number of controls that can change...

; *** Start added by AutoIt3Wrapper ***
#include <GUIConstantsEx.au3>
; *** End added by AutoIt3Wrapper ***

#include <array.au3>
; Includes, etc

#AutoIt3Wrapper_Add_Constants=n

Local $aOptionsArray[3]

; lots of other code

Local $gui010 = GUICreate('Test')
Local $bSelect1 = GUICtrlCreateCheckbox('#1', 10, 10, 100, 20)
Local $bSelect2 = GUICtrlCreateCheckbox('#2', 10, 30, 100, 20)
Local $bSelect3 = GUICtrlCreateCheckbox('#3', 10, 50, 100, 20)

GUISetState()

While 1
    Switch GUIGetMsg()
        Case $gui_event_close
            Exit
        Case $bSelect1 To $bSelect3
            _CheckboxesAndRadioButtons()
            _ArrayDisplay($aOptionsArray)
    EndSwitch
WEnd

Func _CheckboxesAndRadioButtons()

    ; Create Checkboxes and Radio Buttons and read the results
    ; all variables in this Function were assigned either at the top of the script or in the function.
    ; Radio Buttons to Array
    ; $aOptions[0] = $bTestSelectForever
    ; $aOptions[1] = $bTestSelect3Times
    ; $aOptions[2] = $bTestSelectOnce

    For $i = 0 To UBound($aOptionsArray) - 1
        $aOptionsArray[$i] = (GUICtrlRead(Eval('bSelect' & $i + 1)) = $gui_checked) ? 'True' : 'False'
    Next

EndFunc   ;==>_CheckboxesAndRadioButtons

Also, it might be helpful for you to read (re-read) the helpfile sections concerning "scope" and function calls (byref vs. value).

kylomas

  • Like 1

Forum Rules         Procedure for posting code

"I like pigs.  Dogs look up to us.  Cats look down on us.  Pigs treat us as equals."

- Sir Winston Churchill

Share this post


Link to post
Share on other sites
Jibberish

OK I figured it out. I neglected to put a [3] after my declaration.

Local @aOptions[3] solved the issue.

This is weird because usually when I declare an array in other code I don't have to add [x] as in :

Local $aArray  << has worked before

vs

Local $aArray[3]  << required for this particular script... I don't understand why!

Share this post


Link to post
Share on other sites
Jibberish

LOL - That message was still in the text box. I thought I had closed it.. must have hit Submit Reply by mistake.

Good Catch! At least one member is awake :lmao:

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

    • Skeletor
      By Skeletor
      Hi Virtual People,
      My array works perfectly fine. However, what is the best practice if the line in the array doesn't have the correct amount of columns and if I can add a placeholder?

       
      For $count = 1 To _FileCountLines($FileRead1) Step 1 $string = FileReadLine($FileRead1, $count) $input = StringSplit($string, ",", 1) $value1 = $input[1] $value2 = $input[2] $value3 = $input[3] _Excel_RangeWrite($oWorkbook, $oWorkbook.Activesheet, $value2, "A1") _Excel_RangeWrite($oWorkbook, $oWorkbook.Activesheet, $value1, "B1") _Excel_RangeWrite($oWorkbook, $oWorkbook.Activesheet, $value3, "C1") Next  
    • MrCheese
      By MrCheese
      hi all,
      reviewing the forum, this thread is applicable: 
       
       
      I wanted to know if there is now a better way to do this?
      In essence, I load a tab delimited txt file into an array (works well). I used tab, as some fields in the original csv contains commas.
      However, I needed autoit to manipulate this array, and output it as a csv.
      IF my array contains items with a comma, without double quotes around the field, then how best do I get a csv out of this?
      My current workaround is to filewritefromarray tab delimited, then open it in excel and save as a csv. I will need to check this to see how the address fields behave that contain a comma.
       
      Any thoughts would be appreciated.
       
    • Skeletor
      By Skeletor
      Hi All,

      I would like to know how you would take a FileLineRead and insert it into an array which then inserts it into Excel?
      One thing to know is the files content is broken up, so I only use half of the content within $FileRead1.
      So its imperative that the $value1, $value2, etc variables be used. 
      Code below:
      $FileRead1 = FileReadLine("C:\temp\sample.txt",1) For $count = 1 To _FileCountLines($FileRead1) Step 1 $string = FileReadLine($FileRead1, $count) $input = StringSplit($string, ",", 1) $value1 = $input[1] $value2 = $input[2] $value3 = $input[3] $value4 = $input[4] _Excel_RangeWrite($oWorkbook, $oWorkbook.Activesheet, $value1, "A1") _Excel_RangeWrite($oWorkbook, $oWorkbook.Activesheet, $value2, "B1") _Excel_RangeWrite($oWorkbook, $oWorkbook.Activesheet, $value3, "C1") _Excel_RangeWrite($oWorkbook, $oWorkbook.Activesheet, $value4, "D1") Next  
    • AnonymousX
      By AnonymousX
      Hello,
      I'm trying to write a script that moves copies excel cells into an array. I'll than manipulate the values and send array into another program. 
      I don't want range to be specific to a workbook, or sheet, or set of cells.
      I want user to be able to highlight desired cells and to copy either normally ("Ctrl+C") or by a hotkey ("Alt+C"). 
      Could someone help me with this?
      Thank you,
      I've tried to write the framework: (edited)
      #include <MsgBoxConstants.au3> #include <Array.au3> #include <Excel.au3> HotKeySet("!v", "Pastedata") While True Sleep(1000) WEnd func Makearray() local $bArray ;User has cells already copied ;Convert clipboard into an array ;I don;t know how excel stores data to clipboard so don;t know how to bring it into array _Arraydisplay($bArray) MsgBox(0,0,$bArray) return $bArray endfunc func Pastedata() Local $aArray MsgBox(0,0,"wait",1) ;make array based on assumption user has already copied a range to clipboard $aArray = Makearray() ;paste code ;don;t worry about this I got the rest endfunc  
    • Dzenan03
      By Dzenan03
      I want to make a while loop, that creates variables based on a array. For thist I created the array $iDsO with the number and the name of folders in an other folder. Every folder has a different name an I want to create variables(arrays) for each folder that show me all the files in that folder. For example: I have the Folder \Folder1. In it there are the Folders \1, \2, \3. In 1, 2 and 3 there are some files(.png). The array for Folder1 is $iDsO and now I want to crate the arrays $iDsO1, $iDsO2 and $iDsO3 with the files in them can I make something like this:
      While $iDs > 0 ;$iDs is the number of files in Folder1>> $iDsO[0] $iDs#here should come the Foldername for example '1'# = _FileListtoArray(@ProgramFilesDir&"\Folder1\"&$iDsO[$iDs]) $iDs = $iDs - 1 Wend So that in the End I have three variabels ($iDs1, $iDs2 and $iDs3)
       
      Is this posible or if not what could I do instead ( I don´t know the number of folders in Folder1 in the begining).
×