_GUICtrlListView_AddArray / Listview issue

[Fossil Rock]

I'm having a little difficulty getting beyond this error with the script below:

C:\.....\Include\GuiListView.au3 (543) : ==> Array variable has incorrect number of subscripts or subscript dimension range exceeded.:
DllStructSetData($tBuffer, "Text", $aItems[$iI][0])
DllStructSetData($tBuffer, "Text", ^ ERROR

Can someone point me in the right direction? Sample Lua script in spoiler.

expandcollapse popup

#include <ButtonConstants.au3>
#include <EditConstants.au3>
#include <GUIConstantsEx.au3>
#include <GUIListBox.au3>
#include <WindowsConstants.au3>
#include <String.au3>
#include <Array.au3>
#include <File.au3>
#include <GuiListView.au3>

;Opt('GuiOnEventMode', 1) ; prevents application from exiting ??

$Form1_1 = GUICreate("Script Parser", 1001, 801, -1, -1)
$Button1 = GUICtrlCreateButton("Open File", 16, 24, 91, 25)
$Button2 = GUICtrlCreateButton("Parse File", 125, 24, 91, 25)
$Button3 = GUICtrlCreateButton("{undefined}", 234, 24, 91, 25)
$Button4 = GUICtrlCreateButton("{undefined}", 343, 24, 91, 25)
$Button5 = GUICtrlCreateButton("{undefined}", 452, 24, 91, 25)
$Button6 = GUICtrlCreateButton("{undefined}", 561, 24, 91, 25)
$Button7 = GUICtrlCreateButton("{undefined}", 670, 24, 91, 25)
$Button8 = GUICtrlCreateButton("Save Output", 779, 24, 91, 25)
$Button9 = GUICtrlCreateButton("Exit", 888, 24, 91, 25)
$List1 = GUICtrlCreateList("", 8, 128, 177, 564, -1, $WS_EX_STATICEDGE)
GUICtrlSetData(-1, "")
GUICtrlSetFont(-1, 10, 400, 0, "Courier New")
$Edit1 = GUICtrlCreateEdit("", 200, 128, 793, 633, -1, $WS_EX_STATICEDGE)
GUICtrlSetData(-1, "")
GUICtrlSetFont(-1, 10, 400, 0, "Courier New")
$Label1 = GUICtrlCreateLabel("{empty}", 10, 700, 200, 27)
GUICtrlSetFont(-1, 14, 400, 0, "Calibri")
GUISetState(@SW_SHOW)

GetFunctions() ; Assign to button 2

While 1
$nMsg = GUIGetMsg()
Switch $nMsg
  Case $GUI_EVENT_CLOSE
   Exit
EndSwitch
WEnd

Func OpenFile()
EndFunc

Func GetFunctions()
Local $aFileName = FileOpenDialog("Select a file", @ScriptDir & "\", "(*.lua)", 0)
Local $aArray = FileRead($aFileName)
Local $aFunctions = _StringBetween($aArray, "Function ", "(")
_ArrayDisplay($aFunctions)
_GUICtrlListView_AddArray($List1,$aFunctions) ; ERROR
EndFunc

Func _ListViewEvent()
    For $i = 0 To $List1[0]
        If @GUI_CtrlId = $List1[$i] Then ExitLoop
    Next
    GUICtrlSetData($Label1, GUICtrlRead($List1[$i]))
EndFunc

Func _Exit()
    Exit
EndFunc

  Reveal hidden contents

expandcollapse popup

--[[
rem 20080805
@title yet another accurate intervalometer
@param a Duration (min)/-1 disable
@default a -1
@param b Duration (sec)/n of seqs
@default b 5
@param c Delay 1st sequence (min)
@default c 0
@param d Delay 1st sequence (sec)
@default d 3
@param e Trigger every n min
@default e 0
@param f ...every n sec
@default f 3
@param g ...every .n sec
@default g 0
@param h Endless?
@default h 0
@param i Seq dur (m)/-1
@default i -1
@param j Seq dur (s)/n of shots/seq
@default j 1
]]

drivemode_continuous = 1

if a < -1 then a = -1 end
if ( a > -1 and b < 0 ) then b = 0 end
if ( a == -1 and b < 1 ) then b = 1 end
if c < 0 then c = 0 end
if d < 0 then d = 0 end
if e < 0 then e = 0 end
if f < 0 then f = 0 end
if g < 0 then g = 0 end
if ( h < 0 or h > 1 ) then h = 0 end
if i < -1 then i = -1 end
if ( i > -1 and j < 0 ) then j = 0 end
if ( i == -1 and j < 1 ) then j = 1 end
if (( i == -1 and j > 1 ) or i > -1 ) then
  if get_drive_mode() ~= drivemode_continuous then
    print( "set drive mode" )
    print( "to continuous" )
    sleep(1500)
    cannot_continue = true
  end
end

if a > -1 then duration = a*60000 + b*1000 else duration = b end
delay_first= c*60000 + d*1000
if g < 0 then delay = e*60000 + f*1000 + g*10 else delay = e*60000 + f*1000 + g*100 end
if i > -1 then sequence = i*60000 + j*1000 else sequence = j end


function shoot_by_numbers( sequence_target )
sequence_current = 0
repeat
  tick_target = get_tick_count() + delay
  sequence_current = sequence_current + 1
  print( "sequence " .. sequence_current .. " of " .. sequence_target )
  if i == -1 then shoot_count( sequence ) else shoot_tick( sequence ) end
  while ( get_tick_count() < tick_target and sequence_current < sequence_target ) do
  end
until sequence_current >= sequence_target
end

function shoot_by_duration( duration )
duration_target = get_tick_count() + duration
repeat
  tick_target = get_tick_count() + delay
  if i == -1 then shoot_count( sequence ) else shoot_tick( sequence ) end
  print( (duration_target-get_tick_count())/1000 .. " sec to go" )
  while ( get_tick_count() < tick_target and get_tick_count() < duration_target ) do
  end
until get_tick_count() >= duration_target
end

function shoot_forever()
tick_initial = get_tick_count()
sequence_current = 0
repeat
  tick_target = get_tick_count() + delay
  sequence_current = sequence_current + 1
  print( "sequence " .. sequence_current )
  if i == -1 then shoot_count( sequence ) else shoot_tick( sequence ) end
  print ( (get_tick_count()-tick_initial)/1000 .. " sec elapsed")
  while ( get_tick_count() < tick_target ) do
  end
until false
end

function shoot_count( count_inc )
count_target = get_exp_count() + count_inc
if count_target > 9999 then count_target = count_target - 9999 end
press( "shoot_half" )
press( "shoot_full" )
repeat
until get_exp_count() == count_target
release( "shoot_full" )
repeat
until get_shooting() == false
end

function shoot_tick( tick_duration )
tick_target = get_tick_count() + tick_duration
press( "shoot_half" )
press( "shoot_full" )
while ( get_tick_count() < tick_target ) do
end
release( "shoot_full" )
repeat
until get_shooting() == false
end

if not cannot_continue then
  tick_target = get_tick_count() + delay_first
  print( "waiting " .. delay_first/1000 .. " sec" )
  while ( get_tick_count() < tick_target ) do
    tick_current = get_tick_count()
  end
  if h == 1 then shoot_forever() end
  if a > -1 then shoot_by_duration( duration ) else shoot_by_numbers( duration ) end
end

[BrewManNH]

Well, one thing I can see right away is that you're using a ListView function on a ListBox. GUICtrlCreateList makes a listbox, not a listview.

EDIT: I also forgot to mention, _GUICtrlListView_AddArray requires a 2D array, you're sending it a 1D array.

Edited January 19, 2012 by BrewManNH

[Melba23]

Fossil Rock,

You are trying to use _GUICtrlListView_AddArray to add items to a control created with GUICtrlCreateList. Not the same type of control at all.

And even if it was, the array format is wrong - you need a 2D array.

M23

[Fossil Rock]

First of all, thanks for the replies.

3 days later and this is as close as I've gotten. The Scrollbar shows up like there's data in the control, but no data. Does the control need to be refreshed somehow to show the data?

#include <String.au3>
#include <Array.au3>
#include <GUIConstantsEx.au3>
#include <ListViewConstants.au3>
#include <WindowsConstants.au3>
#include <GuiListView.au3>
$Form1 = GUICreate("Form1", 615, 438, -1, -1)
$ListView1 = GUICtrlCreateListView("", 24, 24, 250, 390)
GUISetState(@SW_SHOW)
Local $aFileName = @ScriptDir & "\Script.lua" ; save the sample script from the OP as Script.lua in the script dir
Local $aFileLines = FileRead($aFileName)
Local $aFunctions = _StringBetween($aFileLines, "Function ", "(")

Local $aArray[UBound($aFunctions)][1]

For $i = 0 To UBound($aFunctions) -1
  $aArray[$i][0] = $aFunctions[$i]
Next

Sleep(1000)
_GUICtrlListView_AddArray(GUICtrlGetHandle($ListView1), $aArray)

While 1
$nMsg = GUIGetMsg()
Switch $nMsg
  Case $GUI_EVENT_CLOSE
   Exit
EndSwitch
WEnd

  Reveal hidden contents

[Melba23]

Fossil Rock,

It seems that _GUICtrlListView_AddArray only works on ListViews created with the UDF - looking inside the function shows it uses the UDF functions to add the items. So you need to create the ListView like this:

#include <string.au3>
#include <array.au3>
#include <guiconstantsex.au3>
#include <listviewconstants.au3>
#include <windowsconstants.au3>
#include <guilistview.au3>

$Form1 = GUICreate("Form1", 615, 438, -1, -1)

$ListView1 = _GUICtrlListView_Create($Form1, "", 24, 24, 250, 390) ; <<<<<<<<<<<<<<<<<
_GUICtrlListView_SetExtendedListViewStyle($ListView1, BitOR($LVS_EX_GRIDLINES, $LVS_EX_FULLROWSELECT)) ; <<<<<<<<<<<<<<<<<
_GUICtrlListView_InsertColumn($ListView1, 0, "", 250) ; <<<<<<<<<<<<<<<<<

GUISetState()

Local $aFileName = @ScriptDir & "Script.lua" ; save the sample script from the OP as Script.lua in the script dir
Local $aFileLines = FileRead($aFileName)
Local $aFunctions = _StringBetween($aFileLines, "Function ", "(")

Local $aArray[UBound($aFunctions)][1]

For $i = 0 To UBound($aFunctions) - 1
    $aArray[$i][0] = $aFunctions[$i]
Next

_GUICtrlListView_AddArray($ListView1, $aArray)

While 1
    $nMsg = GUIGetMsg()
    Switch $nMsg
        Case $GUI_EVENT_CLOSE
            Exit
    EndSwitch
WEnd

It works for me now.

M23

[Fossil Rock]

Does the data show up in the ListView for you? Must be something wrong with my computer.

[Melba23]

Fossil Rock,

Certainly does.

M23

[BrewManNH]

_GUICtrlListView_AddArray will work with a listview created with the native function, I changed the script M23 posted to use the native function and it worked ok for me. By the way, the script worked both ways for me, with and without the UDF created ListView control.

[Spenhouet]

Hey

I have a similar Problem.

But i know i have a 1D Array and for the ListView_AddArray i need a 2D. But i have not a 2D.

Which command can i use to write an 1D Array in a ListView?

[Melba23]

Spenhouet,

Why do you need a function? Why not just loop through the array and add each item individually?

M23

[Spenhouet]

  On 6/5/2012 at 11:41 AM, 'Melba23 said:

Spenhouet,

Why do you need a function? Why not just loop through the array and add each item individually?

M23

It figures. Melba you are my hero today

This works thx to you

$testlist = _GUICtrlListView_Create($Form1, "", 20, 15, 350, 75, BitOR($WS_HSCROLL, $WS_VSCROLL, $WS_BORDER, $LVS_REPORT, $LVS_NOCOLUMNHEADER), 0)
GUICtrlSetFont(-1, 12, 400, 0, "Arial Narrow")
_GUICtrlListView_AddColumn($testlist, "", 330)
Global $aIndex = addtomenu()
_ArrayPush($aIndex, "")
local $k = 0
While $aIndex[$k] <> ""
   _GUICtrlListView_AddItem($testlist, $aIndex[$k], 0)
   $k += 1
WEnd