array problem

[SpinningCone]

i'm trying to build a function that queries the database then returns an array of results

here is the relavant code:

$i = 1

            While $adoRsBan.eof = False
                $auths[$i] = StringStripCR($adoRsBan.Fields.Item("description").Value)
                $adoRsBan.MoveNext()
                $i += 1
            wend

            return($auths)

but it throws Expected a "=" operator in assignment statement.:

ive tried to DIM the array and changing to 0 index but i get various errors about the format plus i dont know how big to make it since it appears .RecordCount is not supported in autoit. how can i dynamically assign arrays?

[PsaltyDS]

AutoIt supports the COM Object interface used for ADO. Your ADO options (specifically the Cursor Type of your record set) may not allow .RecordCount for that particular query. Ref: W3Schools ADO Tutorial

Note: This property will return -1 for a forward-only cursor; the actual count for a static or keyset cursor; and -1 or the actual count for a dynamic cursor.

It's possible the nested COM property is messing with it though. Try saving the value to a variable first:

$i = 1

            While $adoRsBan.eof = False
                $sVal = $adoRsBan.Fields.Item("description").Value
                $auths[$i] = StringStripCR($sVal)
                $adoRsBan.MoveNext()
                $i += 1
            wend

[SpinningCone]

AutoIt supports the COM Object interface used for ADO. Your ADO options (specifically the Cursor Type of your record set) may not allow .RecordCount for that particular query. Ref: W3Schools ADO Tutorial

yeah i get a -1 so declaring the array with recordcount is out. but thats fine there should be a way to initilize an empty array then fill it per loop iteration without the need for the total members with recordcount.

now i can use this code and it kinda does that

local $auths[1]

            While $adoRsBan.eof = False
                _ArrayAdd($auths, StringStripCR($adoRsBan.Fields.Item("description").Value))
                $adoRsBan.MoveNext()

            wend
            return($auths)

problem is the array's first element is empty and arrayadd sticks it on the end. i could kludge around this with a counter and some logic but it seems like there should be a better solution

[BrewManNH]

Check out the ReDim command, it will allow you to dynamically change the size of the array without deleting the contents of the array.

#include <array.au3>
Local $Array[2]
For $i = 0 To 100
    $Array[$i] = $i
    ReDim $Array[$i + 2]
Next
_ArrayDisplay($Array)

[SpinningCone]

Check out the ReDim command, it will allow you to dynamically change the size of the array without deleting the contents of the array.

#include <array.au3>
Local $Array[2]
For $i = 0 To 100
    $Array[$i] = $i
    ReDim $Array[$i + 2]
Next
_ArrayDisplay($Array)

hmm i had used redim before in a nother language forgot to check if autoit supported it.

still a bit tricky witht he loop i have. since i would still have to initilize it with at least one slot if my search returns only one query (which wily be common) and i redim within the loop ill end up with a dead cell at the end.

lacking an elegant solution i went with this :

local $auths[1]

            While $adoRsBan.eof = False
                _ArrayAdd($auths, StringStripCR($adoRsBan.Fields.Item("description").Value))
                $adoRsBan.MoveNext()

            wend
            _ArrayDelete($auths, 0)

            return($auths)

dim it with a blank space in [0] then use array add to tack on all the crap i need then delete the first element before returning it.

[Tvern]

I'd suggest one of the following:

This one starts writing from element 0 and should be a little faster than your method.

local $auths[1], $iUbound = 1
While $adoRsBan.eof = False
    ReDim $auths[$iUbound]
    $auths[$iUbound -1] = StringStripCR($adoRsBan.Fields.Item("description").Value)
    $adoRsBan.MoveNext()
    $iUbound += 1
WEnd
Return $auths

While this one will only do one Redim, because Redims can take a long time if you do allot of them, meaning it's fast, but a bit messy.

local $auths[160777216], $iUbound = 0 ;16777216 being the maximum array size in autoit. Make a reasonable estimate for yourself as a large array will eat memory until it's schrunk.
While $adoRsBan.eof = False
    $auths[$iUbound -1] = StringStripCR($adoRsBan.Fields.Item("description").Value)
    $adoRsBan.MoveNext()
    $iUbound += 1
WEnd
ReDim $auths[$iUbound]
Return $auths

[SpinningCone]

I'd suggest one of the following:

This one starts writing from element 0 and should be a little faster than your method.

local $auths[1], $iUbound = 1
While $adoRsBan.eof = False
    ReDim $auths[$iUbound]
    $auths[$iUbound -1] = StringStripCR($adoRsBan.Fields.Item("description").Value)
    $adoRsBan.MoveNext()
    $iUbound += 1
WEnd
Return $auths

While this one will only do one Redim, because Redims can take a long time if you do allot of them, meaning it's fast, but a bit messy.

local $auths[160777216], $iUbound = 0 ;16777216 being the maximum array size in autoit. Make a reasonable estimate for yourself as a large array will eat memory until it's schrunk.
While $adoRsBan.eof = False
    $auths[$iUbound -1] = StringStripCR($adoRsBan.Fields.Item("description").Value)
    $adoRsBan.MoveNext()
    $iUbound += 1
WEnd
ReDim $auths[$iUbound]
Return $auths

heh interesting way of looking at it. since most of the stuff is small scale performance isn't a huge issue so ill just stick to what i have right now. if performance was a big deal i would probably run a count loop and dim once with the proper size or possibly add a count(*) to my sql query and use that value to assume how many records i was getting back.