Sorting a listview

[Rishav]

Hi folks

I have written a simple UI to show the contents of a csv file in a listview. However, i wanted the columns to be sortable by clicking on the column headers. I tried to use _GUICtrlListView_SimpleSort function but have absolutely no idea how to proceed. the helpfile gives no description of the $iCol parameter and the example provided is too tough for me to understand.

expandcollapse popup

$csv_file = FileOpenDialog("Select CSV file", "C:\Documents and Settings\rishavs\My Documents\My Dropbox\Workspace\Automata\Automata Scripts\Output Log\System Log.csv","csv files (*.csv)",1)
;~ $csv_file = "C:\Documents and Settings\rishavs\My Documents\My Dropbox\Workspace\Automata\Automata Scripts\Output Log\System Log.csv"

#include <GUIConstants.au3>
#include <File.au3>
#include <GuiListView.au3>


Dim $csvArray

;Read CSV to array
_FileReadToArray($csv_file, $csvArray)

;Replace all commas with pipe symbols
For $i = 1 To $csvArray[0]
    $csvArray[$i] = StringReplace($csvArray[$i], ",", "|")
Next

$LogViewer_gui = GUICreate("System Log Viewer", 1200, 900, -1, -1)

;Use first row as header
$listview = GUICtrlCreateListView($csvArray[1], 10, 10, 1180, 880, $LVS_REPORT, $LVS_EX_GRIDLINES )
GUICtrlSetBkColor($listview, $GUI_BKCOLOR_LV_ALTERNATE)
_GUICtrlListView_SimpleSort($listview, True, 1)


_GUICtrlListView_BeginUpdate($listview)
;Create all listview items
For $i = 2 To $csvArray[0]
    GUICtrlCreateListViewItem($csvArray[$i], $listview)
    GUICtrlSetBkColor(-1, 0xEcEcEc)
Next

For $a = 0 To _GUICtrlListView_GetColumnCount($listview)
    _GUICtrlListView_SetColumnWidth($listview, $a, $LVSCW_AUTOSIZE_USEHEADER)
Next

_GUICtrlListView_EndUpdate($listview)

GUISetState()

Do
    $msg = GUIGetMsg()
Until $msg = $GUI_EVENT_CLOSE

I have also added a csv file incase you want one.

thanks and regards

Rishav

System Log.zip

[water]

Based on the example found in the help file for _GUICtrlListView_Create I catch $LVN_COLUMNCLICK and use the following code to sort:

$tInfo = DllStructCreate($tagNMLISTVIEW, $ilParam)
_GUICtrlListView_SimpleSort($hWndListView, $bSort, DllStructGetData($tInfo, "SubItem"))

You have to set $bSort in advance to "False". This variable keeps the sort direction.

Your example script should now look like:

expandcollapse popup

$csv_file = FileOpenDialog("Select CSV file", "C:\Documents and Settings\rishavs\My Documents\My Dropbox\Workspace\Automata\Automata Scripts\Output Log\System Log.csv","csv files (*.csv)",1)
;~ $csv_file = "C:\Documents and Settings\rishavs\My Documents\My Dropbox\Workspace\Automata\Automata Scripts\Output Log\System Log.csv"

#include <GUIConstants.au3>
#include <File.au3>
#include <GuiListView.au3>
#include <WindowsConstants.au3>

Dim $csvArray
Global $bSort = False                ; Sort direction

;Read CSV to array
_FileReadToArray($csv_file, $csvArray)

;Replace all commas with pipe symbols
For $i = 1 To $csvArray[0]
    $csvArray[$i] = StringReplace($csvArray[$i], ",", "|")
Next

GUIRegisterMsg($WM_NOTIFY, "WM_NOTIFY")        

$LogViewer_gui = GUICreate("System Log Viewer", 1200, 900, -1, -1)

;Use first row as header
$hlistview = GUICtrlCreateListView($csvArray[1], 10, 10, 1180, 880, $LVS_REPORT, $LVS_EX_GRIDLINES )
GUICtrlSetBkColor($hlistview, $GUI_BKCOLOR_LV_ALTERNATE)

_GUICtrlListView_BeginUpdate($hlistview)
;Create all listview items
For $i = 2 To $csvArray[0]
    GUICtrlCreateListViewItem($csvArray[$i], $hlistview)
    GUICtrlSetBkColor(-1, 0xEcEcEc)
Next

For $a = 0 To _GUICtrlListView_GetColumnCount($hlistview)
    _GUICtrlListView_SetColumnWidth($hlistview, $a, $LVSCW_AUTOSIZE_USEHEADER)
Next

_GUICtrlListView_EndUpdate($hlistview)

GUISetState()

Do
    $msg = GUIGetMsg()
Until $msg = $GUI_EVENT_CLOSE

Func WM_NOTIFY($hWnd, $iMsg, $iwParam, $ilParam)
    #forceref $hWnd, $iMsg, $iwParam
    Local $hWndFrom, $iIDFrom, $iCode, $tNMHDR, $hWndListView, $tInfo
;~  Local $tBuffer
    $hWndListView = $hListView
    If Not IsHWnd($hListView) Then $hWndListView = GUICtrlGetHandle($hListView)

    $tNMHDR = DllStructCreate($tagNMHDR, $ilParam)
    $hWndFrom = HWnd(DllStructGetData($tNMHDR, "hWndFrom"))
    $iIDFrom = DllStructGetData($tNMHDR, "IDFrom")
    $iCode = DllStructGetData($tNMHDR, "Code")
    Switch $hWndFrom
        Case $hWndListView
            Switch $iCode
                Case $LVN_COLUMNCLICK ; A column was clicked
                    $tInfo = DllStructCreate($tagNMLISTVIEW, $ilParam)
                    _GUICtrlListView_SimpleSort($hWndListView, $bSort, DllStructGetData($tInfo, "SubItem"))
                    ; No return value
            EndSwitch
    EndSwitch
    Return $GUI_RUNDEFMSG
EndFunc   ;==>WM_NOTIFY

Edited September 9, 2009 by water

[Rishav]

Thanks for the reply Water.

I also got it to work by trial and error. realized that i was missing the _GUICtrlListView_RegisterSortCallBack function.

expandcollapse popup

$csv_file = FileOpenDialog("Select CSV file", "C:\Documents and Settings\rishavs\My Documents\My Dropbox\Workspace\Automata\Automata Scripts\Output Log\System Log.csv", "csv files (*.csv)", 1)
;~ $csv_file = "C:\Documents and Settings\rishavs\My Documents\My Dropbox\Workspace\Automata\Automata Scripts\Output Log\System Log.csv"

#include <GUIConstants.au3>
#include <File.au3>
#include <GuiListView.au3>


Dim $csvArray

;Read CSV to array
_FileReadToArray($csv_file, $csvArray)

;Replace all commas with pipe symbols
For $i = 1 To $csvArray[0]
    $csvArray[$i] = StringReplace($csvArray[$i], ",", "|")
Next

$LogViewer_gui = GUICreate("System Log Viewer", 1200, 950, -1, -1)

$LogViewer_button = GUICtrlCreateButton("Refresh", 550, 910, 100)

;Use first row as header
$listview = GUICtrlCreateListView($csvArray[1], 10, 10, 1180, 880)
_GUICtrlListView_SetExtendedListViewStyle($listview, BitOR($LVS_EX_GRIDLINES, $LVS_EX_FULLROWSELECT, $LVS_EX_SUBITEMIMAGES, $LVS_EX_DOUBLEBUFFER))
_GUICtrlListView_RegisterSortCallBack($listview)
GUICtrlSetBkColor($listview, $GUI_BKCOLOR_LV_ALTERNATE)

_GUICtrlListView_BeginUpdate($listview)

;Create all listview items
For $i = 2 To $csvArray[0]
    GUICtrlCreateListViewItem($csvArray[$i], $listview)
    GUICtrlSetBkColor(-1, 0xEcEcEc)
Next

For $a = 0 To _GUICtrlListView_GetColumnCount($listview)
    _GUICtrlListView_SetColumnWidth($listview, $a, $LVSCW_AUTOSIZE_USEHEADER)
Next

_GUICtrlListView_EndUpdate($listview)

GUISetState()

While 1
    Switch GUIGetMsg()
        Case $GUI_EVENT_CLOSE
            _GUICtrlListView_UnRegisterSortCallBack($listview)
            ExitLoop
        Case $listview
            _GUICtrlListView_SortItems($listview, GUICtrlGetState($listview))

        Case $LogViewer_button
            _GUICtrlListView_DeleteAllItems($listview)

            _GUICtrlListView_BeginUpdate($listview)

            ;Create all listview items
            For $i = 2 To $csvArray[0]
                GUICtrlCreateListViewItem($csvArray[$i], $listview)
                GUICtrlSetBkColor(-1, 0xEcEcEc)
            Next

            For $a = 0 To _GUICtrlListView_GetColumnCount($listview)
                _GUICtrlListView_SetColumnWidth($listview, $a, $LVSCW_AUTOSIZE_USEHEADER)
            Next

            _GUICtrlListView_EndUpdate($listview)

    EndSwitch
WEnd

The only problem now is that sorting takes 1-2 seconds while i was hoping for it to be instanteneous.

[PartyPooper]

The only problem now is that sorting takes 1-2 seconds while i was hoping for it to be instanteneous.

You and me both. Wait until you get a listview of 65,000 items - it takes over an hour to sort. If you find the answer to quicker sorting, post it up please. I'm sure there are more than a few of us waiting for that one.

[martin]

You and me both. Wait until you get a listview of 65,000 items - it takes over an hour to sort. If you find the answer to quicker sorting, post it up please. I'm sure there are more than a few of us waiting for that one.

I expect you will get a big improvement simply by using _GUICtrlListView_BeginUpdate before you start sorting and _GUICtrlListView_EndUpdate after sorting.

You could also try copying to an array, sorting the array and copying back which I would expect to be faster.

[PartyPooper]

I expect you will get a big improvement simply by using _GUICtrlListView_BeginUpdate before you start sorting and _GUICtrlListView_EndUpdate after sorting.

You could also try copying to an array, sorting the array and copying back which I would expect to be faster.

I'll look into those functions thanks Martin.