array variable has incorrect number of...

[gcue]

hello.

i started to get "error: arrage variable has incorrect number of subscripts or subscript dimesnion range exceeded."

Once i started to use stringsplit. are there some strings that can't be split err whats the deal?

$user =  RegRead("\\" & $assets[$x] & "\HKEY_LOCAL_MACHINE\SOFTWARE\Altiris\Altiris Agent\Inventory", "PrimaryUser")
$version = RegRead("\\" & $assets[$x] & "\HKEY_LOCAL_MACHINE\SOFTWARE\Bloomberg L.P.\Components", "sotr_version")

$usr=StringSplit($user,"\")
$ver=StringSplit($version," ")

FileWrite($file, '' &@CRLF & _
                $usr[2] & "," & $ver[2])

Edited April 30, 2008 by gcue

[Jos]

You are not checking the $usr[0] and $ver[0] to see the number of entries in the array.

[gcue]

what do you mean? how do i check [0] ?

[Jos]

what do you mean? how do i check [0] ?

Have you looked in the Helpfile?

Return Value

Returns an array, the first element ($array[0]) contains the number of strings returned, the remaining elements ($array[1], $array[2], etc.) contain the delimited strings.

If no delimiters were found @error is set to 1, the count is 1 ($array[0]) and the full string is returned ($array[1]).

[gcue]

hmm im still confused. i get that usr[0] has the number of strings

so why would i have to bring that out? or how do i?

[Jos]

hmm im still confused. i get that usr[0] has the number of strings

so why would i have to bring that out? or how do i?

You refer to $usr[2], but you have not tested if that really exists. When $usr[0] is 2 or greater it will be there.

[gcue]

and if its not?

thats where i run into the error right?

so i should do...

if $usr[0] = 0 then

$usr[0] = 1

EndIF

??

[Jos]

and if its not?

thats where i run into the error right?

so i should do...

if $usr[0] = 0 then

$usr[0] = 1

EndIF

??

thats up to you what to do when the returned array doesn't contain the required values. SO what do you want to do in that case ?

[gcue]

ahhh i can msgbox saying that array was empty... =D

so thats what you meant by:

"You are not checking the $usr[0] and $ver[0] to see the number of entries in the array."

i have to do something just in case they are 0 =)

[gcue]

hmm i still get the same error

$usr=StringSplit($user,"\")

if $usr[0] = 0 then

Msgbox(0,"Backup XLBs", $assets[$x] & " does not have a primary user.")

EndIF

[Jos]

hmm i still get the same error

$usr=StringSplit($user,"\")

if $usr[0] = 0 then

Msgbox(0,"Backup XLBs", $assets[$x] & " does not have a primary user.")

EndIF

What Line and what Array do you get the error for?

[someone]

Not to step on your toes Jos...

If there was an error with stringsplit, this is what the helpfile says will be returned...

"If no delimiters were found @error is set to 1, the count is 1 ($array[0]) and the full string is returned ($array[1])."

Meaning if stringsplit caused an error, the array will not have a 2 element. You can error check for that by checking @error after stringsplit (what I would do), or by checking if $array[1] is the entire string, or by doing a Ubound on $array to see if it goes up to 2.

Edited May 1, 2008 by someone

[gcue]

ok im getting closer....

i still get the error when it doesnt find the array

If @error Then

Msgbox(0,"Backup XLBs", $assets[$x] & " does not have a primary user.")

EndIf

it does the msgbox but what do i do after that? i tried setting $usr[2] to a valid answer but i still get the error.

=/

[SmOke_N]

The point that Jos is trying to make, is that you need [2] to be filled by something. You are assuming that [2] will always have something though.

Your condition statement should be something like:

If ($assets[0] <> 2) Then Continueloop

;Rest of code here

That way, you'll never get the exception thrown if your code is under the continueloop. 1 line fixes the whole issue.

Of course, this is if you only want information with 2 elements [1] and [2].

If you're not in a loop and you don't care about how many elements there are:

If ($assets[0] > 1) Then
    do something with both elements: $assets[1] & $assets[2]
Else
    do something with only the first element: $asset[1]
EndIf

It's array management, since it seems you are having an issue with understanding maybe how arrays work (or think maybe that every return will always have 2+ elements), you might take some time to think of some proper error handling on your own.

This is also a good link: http://www.autoitscript.com/wiki/index.php?title=Arrays

Edited May 1, 2008 by SmOke_N

[Jos]

Not to step on your toes Jos...

If there was an error with stringsplit, this is what the helpfile says will be returned...

"If no delimiters were found @error is set to 1, the count is 1 ($array[0]) and the full string is returned ($array[1])."

Meaning if stringsplit caused an error, the array will not have a 2 element. You can error check for that by checking @error after stringsplit (what I would do), or by checking if $array[1] is the entire string, or by doing a Ubound on $array to see if it goes up to 2.

Don't worry stepping on my toes:)

Isn't testing for "$array[0] > 1" enough to ensure that the $array[2] doesn't cause an error like posted ?

Should work as far as I can tell.

Edited May 1, 2008 by Jos

[gcue]

that worked!

$usr=StringSplit($user,"\")

If @error Then

Msgbox(0,"Backup XLBs", $assets[$x] & " does not have a primary user.")

ContinueLoop

EndIf

thank you much fellas.