void CShowStringView::OnDraw(CDC* pDC)
{
CShowStringDoc* pDoc = GetDocument();
ASSERT_VALID(pDoc);
if (!pDoc)
return;

COLORREF oldcolor;
switch (pDoc->GetColor())
{
case 0:
oldcolor = pDC->SetTextColor(RGB(0,0,0));break;
case 1:
oldcolor = pDC->SetTextColor(RGB(0xFF,0,0));break;
case 2:
oldcolor = pDC->SetTextColor(RGB(0,0xFF,0));break;
}
// pDC->SetTextColor(oldcolor); 这个函数放到这为什么不行，而去掉或放到倒数第二行就行
CRect rect;
GetClientRect(&rect);
pDC->DrawTextW(pDoc->GetString(),&rect,DT_CENTER|DT_VCENTER|DT_SINGLELINE);

pDoc->SetModifiedFlag();
}

你确定？
我感觉好像不太像啊。
再看看，看看

你是说什么不行啊？
是说编译不过去，还是说颜色没有改变？

这样写颜色当然不会改变，pDC设备颜色刚选为RGB(..,..,..),oldcolor返回原先设备颜色，然后你就直接又把pDC设备选回原先的颜色,然后用设备pDC去DrawText,当然是用原先的颜色画的了。
一般情况下是要在你用完这个颜色以后用原先的颜色选回
oldcolor = pDC->SetTextColor(RGB(0,0xFF,0)); //为设备设置文本颜色

CRect rect; //在设备上画字
GetClientRect(&rect);
pDC->DrawTextW(pDoc->GetString(),&rect,DT_CENTER|DT_VCENTER|DT_SINGLELINE);

pDC->SetTextColor(oldcolor); //将设备文本颜色选回