VisualStadio2005 MFC のリストコントロールでソートの方法

Question

MFCのリストコントロールでソート処理を行いたいのですが、以下の様に作りましたがソートが行われません。
コールバック関数の引数LPARAM lParam1, LPARAM lParam2,は共に0が入ったままになっています。
わかる人がいましたらぜひ教えて下さい。

// フォルダ内からOutput_xxxx_xxxx.binファイルを探してリストに登録
void CSampleDlg::OnEnChangeEdit1()
{
	BOOL			bContinue;
	CFileFind		cFind;
	CString			strFile;
	CString strFolder = szDisplayName;
	CString strExtension;
	LVITEM       lvi;

	if(strFolder.Right(1) != "\"){
		strFolder += "\Output*.bin";
	}
	strFolder += strExtension;

	bContinue = cFind.FindFile(strFolder);
// 要検証
	while(bContinue){
		int i=0;

		bContinue = cFind.FindNextFile();

		if(cFind.IsDirectory() == FALSE)
		{
			INT i=0, index ;
		
			strFile = cFind.GetFileName();
			
			int f = strFile.Find('_');
			int l = strFile.ReverseFind('_');
			int e = strFile.ReverseFind('.');
			
			// Mid(offset, length)
			CString csSeq = strFile.Mid(f+1, l-(f+1));
			CString csLen = strFile.Mid(l+1, e-(l+1));

			lvi.mask = LVIF_TEXT;
			lvi.iItem = i++;
			lvi.iSubItem = 0;
			lvi.pszText = (LPTSTR)(LPCTSTR)strFile;
			if (index = m_xcList.InsertItem(&lvi) == -1) break;

			lvi.iItem = index;
			lvi.iSubItem = 1;
			lvi.pszText =  const_cast<LPTSTR>(static_cast<LPCTSTR>(csSeq));
			if (!m_xcList.SetItem(&lvi)) break;

			lvi.iItem = index;
			lvi.iSubItem = 2;
			lvi.pszText = (LPTSTR)(LPCTSTR)csLen;
			if (!m_xcList.SetItem(&lvi)) break;

			CString sum;
			sum.Format("%lu", (strtoul(csSeq,NULL,10))+atol(csLen));
			lvi.iItem = index;
			lvi.iSubItem = 3;
			lvi.pszText = (LPTSTR)(LPCTSTR)sum;
			if (!m_xcList.SetItem(&lvi)) break;

		}
	}
}

// コールバック関数 クラス内に定義する場合はStaticを付ける事
int CALLBACK CSampleDlg::CompareData(LPARAM lParam1, LPARAM lParam2, LPARAM lParamSort);


// カラムクリックイベント
void CBinUnitDlg::OnLvnColumnclickList2(NMHDR *pNMHDR, LRESULT *pResult)
{
	LPNMLISTVIEW pNMLV = reinterpret_cast<LPNMLISTVIEW>(pNMHDR);	// <LPNMLISTVIEW>が必要
	m_xcList.SortItems( &CBinUnitDlg::CompareData, pNMLV->iSubItem);

	// フラグを逆にする	
	flag = !flag;

}

// ソート
int CALLBACK CSampleDlg::CompareData(LPARAM lParam1, LPARAM lParam2, LPARAM lParamSort)
{
	LV_FINDINFO lvf;
    int idx1 , idx2;
    TCHAR szBuf1[256],szBuf2[256];
	
	
    lvf.flags = LVFI_PARAM ;
    lvf.lParam = lParam1 ;
    idx1 = m_xcList.FindItem(&lvf,-1) ;
    lvf.lParam = lParam2 ;
    idx2 = m_xcList.FindItem(&lvf,-1) ;
	memset(szBuf1,0x00,sizeof(szBuf1));
	memset(szBuf2,0x00,sizeof(szBuf2));
    m_xcList.GetItemText( idx1 , (int)lParamSort , szBuf1 , sizeof(szBuf1)) ;
    m_xcList.GetItemText( idx2 , (int)lParamSort , szBuf2 , sizeof(szBuf2)) ;


	if(flag==TRUE)
        return(_tcscmp(szBuf1,szBuf2)) ;
    else
        return(_tcscmp(szBuf1,szBuf2)* -1) ;


}

Answer 1

取りあえず動くようにするならリスト明細を登録しているのループの最後に

item.mask = LVIF_PARAM;
item.lParam= idx;
_pList->SetItem( &item );

とすれば lp1とlp2に希望の値が入ってくると思います。

Comments

[izuki_y]

回答ありがとうございました。

教えて頂いた構文を追加してみた所、for文の末端だと上手くいきませんでしたが、
InsertItem の 後に記載すると上手くいきました。

やはり原因はアイテムにLVIF_PARAMを関連付けてない事だった様で無事ソートされる事を確認したしました。

ありがとうございました。

参考
http://rarara.cafe.coocan.jp/cgi-bin/lng/vc/vclng.cgi?print+200006/00060030.txt

// リストのヘッダ
_pList->InsertColumn(0, _T("ファイル名"), LVCFMT_LEFT,50, -1);
_pList->InsertColumn(1, _T("SEQ"), LVCFMT_LEFT, 100, -1);
_pList->InsertColumn(2, _T("Len"), LVCFMT_LEFT, 50, -1);

// リストの明細
LV_ITEM item;
for(short idx=0;idx < 4 ; idx++){
item.iItem = idx;
item.mask = LVIF_TEXT;
item.pszText = info[idx].file;
item.iSubItem = 0;
_pList->InsertItem( &item ) ;

// ここに入れるとソートされる(2015/02/23)
item.mask = LVIF_PARAM;
item.lParam= idx;
_pList->SetItem( &item );

item.mask= LVIF_TEXT;
item.pszText = info[idx].seq;
item.iSubItem = 1 ;
_pList->SetItem( &item ) ;

item.pszText = info[idx].len;
item.iSubItem = 2 ;
_pList->SetItem( &item ) ;

// ここだとソートが反映されない(2015/02/23)
// item.mask = LVIF_PARAM;
// item.lParam= idx;
// _pList->SetItem( &item );
}
return TRUE; // フォーカスをコントロールに設定した場合を除き、TRUE を返します。
}

Answer 2

SortItems() に渡す関数、及びパラメータの意味を勘違いしていると思います。
lParam1,lParam2 はLVITEM内の lParam の値が渡されますが、提示されているソースには設定されているようすがありません。
適切な値を LVITEM内のlParam に設定すれば lParam1,lParam2 に渡ってくるはずです。

あと CompareData() は静的メンバ関数なのでクラス内のメンバ変数にはアクセス出来ないはずですが、このソースコンパイル通りますか？

Comments

[izuki_y]

回答ありがとうございました。

>あと CompareData() は静的メンバ関数なのでクラス内のメンバ変数にはアクセス出来ないはずですが、このソースコンパイル通りますか？
うっかりしていました、確かにその通りです。
クラス内に定義していないのにスコープ演算子を使って定義していました。
紛らわしくて申し訳ありませんでした。

もう一度リストコントロールの処理のみに焦点を置いてMFCダイアログベースで
作り直してみましたが、やはりソートされません。

もしよろしければ確認をお願い致します。

// CtrlviewListSampleDlg.cpp : 実装ファイル
//

...前略.....

// ソートデータ
#define LIST_SIZE 5

typedef struct __tagListinfo{
TCHAR file[50];
TCHAR seq[50];
TCHAR len[50];
} LISTINFO;
LISTINFO info[LIST_SIZE];

// ソートフラグ（降順、昇順）
#define UP 1
#define DOWN 2
int sortFlg[]={UP,UP,UP,UP} ;

// コールバック関数
int CALLBACK MyListComp(LPARAM ,LPARAM ,LPARAM );

// リストコントロール
CListCtrl* _pList;

....中略......

// CCtrlviewListSampleDlg ダイアログ

CCtrlviewListSampleDlg::CCtrlviewListSampleDlg(CWnd* pParent /*=NULL*/)
: CDialog(CCtrlviewListSampleDlg::IDD, pParent)
{
m_hIcon = AfxGetApp()->LoadIcon(IDR_MAINFRAME);

_tcscpy(info[0].file,_T("Output_0001.bin"));
_tcscpy(info[0].seq,_T("1000"));
_tcscpy(info[0].len,_T("100"));

_tcscpy(info[1].file,_T("Output_0002.bin"));
_tcscpy(info[1].seq,_T("1100"));
_tcscpy(info[1].len,_T("100"));

_tcscpy(info[2].file,_T("Output_0003.bin"));
_tcscpy(info[2].seq,_T("1200"));
_tcscpy(info[2].len,_T("100"));

_tcscpy(info[3].file,_T("Output_0004.bin"));
_tcscpy(info[3].seq,_T("1300"));
_tcscpy(info[3].len,_T("100"));

_tcscpy(info[4].file,_T("Output_0005.bin"));
_tcscpy(info[4].seq,_T("1400"));
_tcscpy(info[4].len,_T("100"));
}

void CCtrlviewListSampleDlg::DoDataExchange(CDataExchange* pDX)
{
CDialog::DoDataExchange(pDX);
DDX_Control(pDX, IDC_LIST1, Ctrllist);
}

BEGIN_MESSAGE_MAP(CCtrlviewListSampleDlg, CDialog)
ON_WM_SYSCOMMAND()
ON_WM_PAINT()
ON_WM_QUERYDRAGICON()
//}}AFX_MSG_MAP
ON_NOTIFY(LVN_COLUMNCLICK, IDC_LIST1, &CCtrlviewListSampleDlg::OnLvnColumnclickList1)
END_MESSAGE_MAP()

// CCtrlviewListSampleDlg メッセージ ハンドラ

BOOL CCtrlviewListSampleDlg::OnInitDialog()
{
CDialog::OnInitDialog();

.....中略.....

// TODO: 初期化をここに追加します。
_pList = &Ctrllist;

// リストのスタイル（レポート、１行選択等）
_pList->SetExtendedStyle(_pList->GetExtendedStyle()|LVS_EX_SUBITEMIMAGES|LVS_EX_FULLROWSELECT);
_pList->ModifyStyle(NULL,LVS_REPORT);

// カラムの削除
if(_pList->GetHeaderCtrl()){
int cnt = _pList->GetHeaderCtrl()->GetItemCount();
for (int i=0;i < cnt;i++)
_pList->DeleteColumn(0);
}

// リストのヘッダ
_pList->InsertColumn(0, _T("ファイル名"), LVCFMT_LEFT,50, -1);
_pList->InsertColumn(1, _T("SEQ"), LVCFMT_LEFT, 100, -1);
_pList->InsertColumn(2, _T("Len"), LVCFMT_LEFT, 50, -1);

// リストの明細
LV_ITEM item;
for(short idx=0;idx < 4 ; idx++){
item.iItem = idx;
item.mask = LVIF_TEXT;
item.pszText = info[idx].file;
item.iSubItem = 0;
_pList->InsertItem( &item ) ;

item.mask= LVIF_TEXT;
item.pszText = info[idx].seq;
item.iSubItem = 1 ;
_pList->SetItem( &item ) ;

item.pszText = info[idx].len;
item.iSubItem = 2 ;
_pList->SetItem( &item ) ;
}
return TRUE; // フォーカスをコントロールに設定した場合を除き、TRUE を返します。
}

..中略...

void CCtrlviewListSampleDlg::OnLvnColumnclickList1(NMHDR *pNMHDR, LRESULT *pResult)
{
UpdateData();

LPNMLISTVIEW pNMLV = reinterpret_cast<LPNMLISTVIEW>(pNMHDR);
// フラグを逆にする
if (sortFlg[pNMLV->iSubItem] == UP){
sortFlg[pNMLV->iSubItem] = DOWN ;
}
else{
sortFlg[pNMLV->iSubItem] = UP ;
}

// ソート処理
_pList->SortItems( MyListComp , pNMLV->iSubItem );

*pResult = 0;
}

// コールバック
int CALLBACK MyListComp( LPARAM lp1 ,LPARAM lp2, LPARAM lp3 )
{
LV_FINDINFO lvf;
int idx1 , idx2;
TCHAR szBuf1[256],szBuf2[256];

lvf.flags = LVFI_PARAM ;
lvf.lParam = lp1 ;
idx1 = _pList->FindItem(&lvf,-1) ;

lvf.lParam = lp2 ;
idx2 = _pList->FindItem(&lvf,-1) ;

_pList->GetItemText( idx1 , (int)lp3 , szBuf1 , sizeof(szBuf1)) ;
_pList->GetItemText( idx2 , (int)lp3 , szBuf2 , sizeof(szBuf2)) ;

if(sortFlg[(int)lp3]==UP)
return(_tcscmp(szBuf1,szBuf2)) ;
else
return(_tcscmp(szBuf1,szBuf2)* -1) ;
}