Click here to Skip to main content
Licence 
First Posted 26 Feb 2002
Views 142,834
Bookmarked 34 times

Example of CListCtrl::SortItems(...) in MSDN

By | 27 Feb 2002 | Article
The given example in the documentation of CListCtrl::SortItems(...) shows us exactly the WRONG way of using the function.

If we look at the documentation of CListCtrl::SortItems(...) in MSDN we find the following example of how to sort the items in reverse alphabetical order:

// Sort the item in reverse alphabetical order.
static int CALLBACK 
MyCompareProc(LPARAM lParam1, LPARAM lParam2, LPARAM lParamSort)
{
   // lParamSort contains a pointer to the list view control.
   CListCtrl* pListCtrl = (CListCtrl*) lParamSort;
   CString    strItem1 = pListCtrl->GetItemText(lParam1, 0);
   CString    strItem2 = pListCtrl->GetItemText(lParam2, 0);

   return strcmp(strItem2, strItem1);
}

void snip_CListCtrl_SortItems()
{
   // The pointer to my list view control.
   extern CListCtrl* pmyListCtrl;

   // Sort the list view items using my callback procedure.
   pmyListCtrl->SortItems(MyCompareProc, (LPARAM) pmyListCtrl);
}
This approach obviously does not work and here is why: The parameters of MyCompareProc(...) - lParam1 and lParam2 are not the index of the items but their 32-bit associated value.

There is a second problem: even if we set the 32-bit associated value of each item to be the same as the item's index this example will still not work. This is because the items will shift positions after each call to MyCompareProc(...) and the 32-bit associated values will no longer represent the item's index.

The function CListCtrl::SortItems(...) is only usefull if we store a pointer in the 32-bit associated value and we want to sort the items by some value that comes from that pointer.

One easy way of implementing reverse alphabetical sort in CListCtrl derived class is using an STL set:

#pragma warning(disable : 4786)
#include <functional>
#include <set>

struct LVITEM_less : public std::binary_function<LVITEM*, LVITEM*, bool>
{
    bool operator()(const LVITEM* pItem1, const LVITEM* pItem2) const
    {
        CString strItem1(pItem1->pszText);
        CString strItem2(pItem2->pszText);

        return (strItem1 < strItem2);
    }
};

void CSortableListCtrl::SortItemsDescending()
{
    // Sort all items in descending aphabetical order using an STL set
    typedef std::set<LVITEM*, LVITEM_less> ItemSet;
    ItemSet setItems;
    int iCount = GetItemCount();
    for (int i = 0; i < iCount; i++)
    {
        LVITEM* pLVI = new LVITEM();
        ::memset(pLVI, 0, sizeof(LVITEM));
        pLVI->iItem = i;
        pLVI->mask = LVIF_IMAGE | LVIF_INDENT | LVIF_PARAM | LVIF_STATE | LVIF_TEXT;
        pLVI->pszText = new TCHAR[1024];
        pLVI->cchTextMax = 1024;
        GetItem(pLVI);

        setItems.insert(pLVI);
    }

    // Remove all items from the list control
    DeleteAllItems();

    // Put the items back in the list control in reverse order
    int iIndex = 0;
    for (ItemSet::reverse_iterator it = setItems.rbegin(); it != setItems.rend(); ++it)
    {
        (*it)->iItem = iIndex++;
        InsertItem(*it);
        delete [] (*it)->pszText;
        delete *it;
    }
}

License

This article has no explicit license attached to it but may contain usage terms in the article text or the download files themselves. If in doubt please contact the author via the discussion board below.

A list of licenses authors might use can be found here

About the Author

Ivor S. Sargoytchev

Software Developer (Senior)

Canada Canada

Member



Sign Up to vote   Poor Excellent
Add a reason or comment to your vote: x
Votes of 3 or less require a comment

Comments and Discussions

 
You must Sign In to use this message board. (secure sign-in)
 
Search this forum  
 FAQ
    Noise  Layout  Per page   
  Refresh
QuestionIgnoring the repeatations PinmemberAt_work22:06 3 Jun '08  
QuestionExplanation Pinmembertom groezer12:08 25 May '07  
GeneralMuch simpler solution!! PinmemberAndy Bray23:30 3 Aug '06  
Generalauthor does not understand how to use the sort issue at all Pinmemberincludeh104:19 16 Jun '06  
GeneralI use it with NO problem PinmemberRancidCrabtree9:36 24 Aug '05  
GeneralRe: I use it with NO problem PinmemberLeonhardt Wille21:29 11 May '06  
GeneralError correction of MSDN Sample... Pinmembermartin.compel6:18 22 May '03  
GeneralRe: Error correction of MSDN Sample... PinsussAnonymous5:23 27 Aug '03  
GeneralRe: Error correction of MSDN Sample... PinmemberPriya_Sundar2:50 16 Apr '08  
GeneralRe: Error correction of MSDN Sample... PinsussAnonymous3:58 15 Oct '03  
GeneralRe: Error correction of MSDN Sample... Pinmembercibercop5:55 16 Aug '05  
Generalmore simple example . Pinmemberradezz12:28 15 Sep '05  
QuestionHow to: make lParam1 and lParam2 being indices PinmemberGeert Delmeiren1:14 8 Apr '02  
AnswerRe: How to: make lParam1 and lParam2 being indices PinPopularmemberrsmenezes7:20 25 Aug '02  
GeneralAlternative to slow SortItems PinmemberTim Kosse7:19 28 Feb '02  
GeneralRe: Alternative to slow SortItems PinmemberIvor S. Sargoytchev7:26 28 Feb '02  
GeneralThis was the Introduction here some tips... PinmemberKarstenK0:29 28 Feb '02  
QuestionSolution? PinmemberDavide Calabro21:33 27 Feb '02  
AnswerRe: Solution? PinmemberChris Maunder4:33 28 Feb '02  
GeneralSorry... PinmemberDavide Calabro5:37 28 Feb '02  
GeneralRe: Solution? PinmemberDavid Wulff13:30 28 Feb '02  

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.

Permalink | Advertise | Privacy | Mobile
Web02 | 2.5.120517.1 | Last Updated 28 Feb 2002
Article Copyright 2002 by Ivor S. Sargoytchev
Everything else Copyright © CodeProject, 1999-2012
Terms of Use
Layout: fixed | fluid