C / C++ / MFC

tataxin wrote:

but because of 2 loop for, the performance speed is very slow, so I have to improve it

SetRedraw(false);

// insert data here
....

SetRedraw(true);
myList.RedrawWindow();

tataxin wrote:

do you mean that:

tataxin wrote:

I applied it, it looks better but the actual time is the same with previous.
And I'll try virtual list control now, thks, [Smile]

SetRedraw(false);

// insert data here
for()
{
......
    for()
    {
......
    }
}

SetRedraw(true);
myList.RedrawWindow();

tataxin wrote:

// get color, depends on the real data
COLORREF aColor = GetRowColor(txt);
crRows.InsertAt(0, aColor)

tataxin wrote:

// get color, depends on the real data
COLORREF aColor = GetRowColor(txt);
crRows.InsertAt(0, aColor)

    CArray crRows;
CArray newData;
int nNew = newData.GetCount();
for (int i=0; i<nnew;>        // insert new row at top
    CString txt = newData.GetAt(i);
    myList.InsertItem(0, txt);

    // get color, depends on the real data
    COLORREF aColor = GetRowColor(txt);
    crRows.InsertAt(0, aColor)

    // change the rest of list ctrl
}

void CExample1Dlg::OnNMCustomdrawList2(NMHDR *pNMHDR, LRESULT *pResult) 
{
        NMLVCUSTOMDRAW* pLVCD = reinterpret_cast<nmlvcustomdraw*>( pNMHDR );
	
	int nItem = static_cast<int> (pLVCD->nmcd.dwItemSpec);
	
	pLVCD->clrTextBk = RGB(0, 0, 0); // select background color

	*pResult = CDRF_DODEFAULT;

	// First thing - check the draw stage. If it's the control's prepaint

	// stage, then tell Windows we want messages for every item.


	if ( CDDS_PREPAINT == pLVCD->nmcd.dwDrawStage )
	{
		*pResult = CDRF_NOTIFYITEMDRAW;
	}
	else if ( CDDS_ITEMPREPAINT == pLVCD->nmcd.dwDrawStage )
	{
		// This is the prepaint stage for an item. Here's where we set the

		// item's text color. Our return value will tell Windows to draw the

		// item itself, but it will use the new color we set here.

		// We'll cycle the colors through red, green, and light blue.


		COLORREF aColor = crText.GetAt(nItem);		
		pLVCD->clrText = aColor;
		
		*pResult = CDRF_DODEFAULT;
	}	
}
</int>

UINT ComThreadEvent1(LPVOID lParam)
{
	CNotepad *dlg = (CNotepad *)lParam;
	while(HD != NULL)
	{	
		::Sleep(500);
		int index = dlg->events.Find(_T("\r\n"),0);
		if(index == 0 )
		{
			dlg->thread2->ResumeThread();
			dlg->GetEvent(dlg->events.Left(index));
		}
	}
	return 0;
}

void CNotepad::GetEvent(CString event)
{
	CString str;
	if(event.Mid(9,2) == "C0")
	{
		str = "Hello";
                CMessageBox mBox; 
                mBox.DoModal();
	}
}

buntyrolln wrote:

why you reply message selecting question button,you may select reply or General radio button!!!!