_GUICtrlListView_DeleteItemsSelected w/ Checkboxes

[timdecker]

_GUICtrlListView_DeleteItemsSelected($listview) works fine to delete items in my listeview if I am "selecting" them by clicking on their name....

What I would like to do, is delete items in my listview based off of what is "checked" and not "selected" in blue.

Also, is there a way disable "selecting", so that only my checkboxes serve a purpose?

Thanks,

Tim

[MikahS]

Have a look at this topic for your first question pertaining to selecting the listview items by checkbox: >link

Have a look at this topic for your second question: >link

[timdecker]

Have a look at this topic for your first question pertaining to selecting the listview items by checkbox: >link

Have a look at this topic for your second question: >link

For my first question, I have this code:

Local $ItemCount = _GUICtrlListView_GetItemCount($listview), $Checked[$ItemCount + 1]
Local $X = 1 ; <<<<<<<<<<<<<< updated code
    For $i = 0 To $ItemCount - 1 ; check all items in the lv and see if any are checked off
        If _GUICtrlListView_GetItemChecked($listview, $i) Then
            _GUICtrlListView_DeleteItem($listview,$i)
        EndIf
    Next
EndFunc

It deletes what is selected, but only 1 at a time. So if I have 5 things selected, I have to hit delete multiple times.

For question 2: I don't need it anymore, since I am using DeleteItem instead of DeleteItemSelected....but I would still like to fix my loop so I only have to press delete once instead of multiple times.

Edited April 23, 2015 by timdecker

[MikahS]

Have a look at:

_GUICtrlListView_DeleteItemsSelected($listview)

EDIT: My apologies, this does not work with checkboxes.

I'm thinking of a solution. 

Edited April 23, 2015 by MikahS

[BrewManNH]

Why not use the exact same code I gave you in your >other topic to do this? Just use it to delete items instead.

[timdecker]

Have a look at:

_GUICtrlListView_DeleteItemsSelected($listview)

EDIT: My apologies, this does not work with checkboxes.

I'm thinking of a solution. 

Thanks, I am have problems understanding why I have to hit delete multiple times. What's wrong with my loop?

[timdecker]

Why not use the exact same code I gave you in your >other topic to do this? Just use it to delete items instead.

 I did that....

Local $ItemCount = _GUICtrlListView_GetItemCount($listview), $Checked[$ItemCount + 1]
Local $X = 1 ; <<<<<<<<<<<<<< updated code
    For $i = 0 To $ItemCount - 1 ; check all items in the lv and see if any are checked off
        If _GUICtrlListView_GetItemChecked($listview, $i) Then
            _GUICtrlListView_DeleteItem($listview,$i)
        EndIf
    Next
EndFunc

But if I have multiple things selected, I have to hit delete multiple times. Sometimes it deletes one item at a time, sometimes it deletes 3 items, then 2 items, then 4 items....

EDIT: What it seems to be doing, is only deleting 1/2 of it at a time. So if I select 12, it will delete 6, then 3, then 2, then the last 1. How do I get it to delete all 12?

Edited April 23, 2015 by timdecker

[BrewManNH]

I did that....

No you didn't, that is not script I posted in #13, that's the code from #7 which won't work as posted for what you're doing. You should look at the full code in #13 where I posted how to identify all of the checked items, put them in an array, and loop through the array to do something with those checked items. You're picking and choosing what code you THINK works without sitting down and seeing how the code ACTUALLY works. If you don't know what the code is doing, you're never going to be able to work this out for yourself.

[timdecker]

No you didn't, that is not script I posted in #13, that's the code from #7 which won't work as posted for what you're doing. You should look at the full code in #13 where I posted how to identify all of the checked items, put them in an array, and loop through the array to do something with those checked items. You're picking and choosing what code you THINK works without sitting down and seeing how the code ACTUALLY works. If you don't know what the code is doing, you're never going to be able to work this out for yourself.

 

I am fairly sure I understand what I am reading for the code....

If you look at my ConsoleWrite statement, it writes the line number of the row I selected.

Local $ItemCount = _GUICtrlListView_GetItemCount($listview), $Checked[$ItemCount + 1]
        Local $X = 1 ; <<<<<<<<<<<<<< updated code
        For $i = 0 To $ItemCount - 1 ; check all items in the lv and see if any are checked off
            If _GUICtrlListView_GetItemChecked($listview, $i) Then
                $Checked[$X] = $i
                $X += 1
                ;ConsoleWrite($Checked[$X])
            EndIf
        Next
        ReDim $Checked[$X]
        $Checked[0] = $X - 1
        If $Checked[0] > 0 Then
            For $rows = 0 To $Checked[0]
                if GUICtrlRead($Checked[$rows]) > 0 then ContinueLoop
                ConsoleWrite($Checked[$rows])
                _GUICtrlListView_DeleteItem($listview,$Checked[$rows])
            Next
        EndIf
EndFunc

What I can't understand is why it delete only 50% of the rows at a time per press of the button...

Edited April 23, 2015 by timdecker

[BrewManNH]

In this case you need to loop through the array in reverse, more than likely. Once you delete an item, the lines shift up by one so you're looking at the wrong line the next time through the loop.

Also, what is this line doing?

if GUICtrlRead($Checked[$rows]) > 0 then ContinueLoop

There are no controls that this could possibly work correctly with, the numbers in the $Checked array are listview indexes, not controls, so checking to see if you can read from a control ID that has no connection to your listview makes no sense.

[timdecker]

In this case you need to loop through the array in reverse, more than likely. Once you delete an item, the lines shift up by one so you're looking at the wrong line the next time through the loop.

Also, what is this line doing?

if GUICtrlRead($Checked[$rows]) > 0 then ContinueLoop

There are no controls that this could possibly work correctly with, the numbers in the $Checked array are listview indexes, not controls, so checking to see if you can read from a control ID that has no connection to your listview makes no sense.

 

I honestly think this is all I need:

Local $ItemCount = _GUICtrlListView_GetItemCount($listview), $Checked[$ItemCount + 1]
        For $i = 0 To $ItemCount - 1 ; check all items in the lv and see if any are checked off
            If _GUICtrlListView_GetItemChecked($listview, $i) Then
                ConsoleWrite($i)
            EndIf
        Next

EndFunc

The consoleWrite here, returns the index of the rows. What I am unsure of is how to go in reverse (like you said).

[kylomas]

Tim,

Local $ItemCount = _GUICtrlListView_GetItemCount($listview), $Checked[$ItemCount + 1]
        For $i = $ItemCount - 1 to 0 step -1 ; check all items in the lv and see if any are checked off
            If _GUICtrlListView_GetItemChecked($listview, $i) Then
                ConsoleWrite($i)
            EndIf
        Next

EndFunc

[timdecker]

 

Tim,

Local $ItemCount = _GUICtrlListView_GetItemCount($listview), $Checked[$ItemCount + 1]
        For $i = $ItemCount - 1 to 0 step -1 ; check all items in the lv and see if any are checked off
            If _GUICtrlListView_GetItemChecked($listview, $i) Then
                ConsoleWrite($i)
            EndIf
        Next

EndFunc

 

Thank you for that kylomas, that worked! I tried multiple variations of that, but I guess not that combo....

 

Could someone explain this to me:

For $i = $ItemCount - 1 to 0 step -1

Thanks!

Tim

Edit: Looking at it it, it makes sense now...but just to clarify the "Step - 1" part is saying to go in reverse, right?

Edited April 24, 2015 by timdecker

[kylomas]

step -1 say increment the step variable by -1 (subtract 1).