有关代码修改的问题

smallfool 2003-07-15 02:25:50

我最近在浏览公司的一个VC高手写的代码。
看了一些，基本理清了类的的层次关系，但是这里面有一个问题，纯粹是编码风格，我想征求一下大家的看法。
代码中用到了网上比较流行的MFCGridCtrl的原代码，这位高手没有从这个类进行派生，而是直接在MFCGridCtrl的实现函数进行了修改。经过我细致的区别，发现修改比较大，我想问的是，大家对这样的修改的态度是什么？

...全文

34 10 打赏收藏转发到动态举报

写回复

用AI写文章

10 条回复

切换为时间正序

请发表友善的回复…

发表回复

smallfool 2003-07-22

打赏
举报

没有人理睬啊？？？？

smallfool 2003-07-17

打赏
举报

只好贴代码了。纵观整个的CSplitterWndEx的实现代码，几乎就是把ＭＦＣ中CSplitterWnd的实现文件拷贝过来，几乎每个函数中都有被注释的部分。不知道各位对这样的编码风格有什么看法？

smallfool 2003-07-17

打赏
举报

BOOL CSplitterWndEx::CreateView(int row, int col,
CRuntimeClass* pViewClass, SIZE sizeInit, CCreateContext* pContext)
{
#ifdef _DEBUG
ASSERT_VALID(this);
ASSERT(row >= 0 && row < m_nRows);
ASSERT(col >= 0 && col < m_nCols);
ASSERT(pViewClass != NULL);
ASSERT(pViewClass->IsDerivedFrom(RUNTIME_CLASS(CWnd)));
ASSERT(AfxIsValidAddress(pViewClass, sizeof(CRuntimeClass), FALSE));

if (GetDlgItem(IdFromRowCol(row, col)) != NULL)
{
TRACE2("Error: CreateView - pane already exists for row %d, col %d.\n",
row, col);
ASSERT(FALSE);
return FALSE;
}
#endif

// set the initial size for that pane
m_pColInfo[col].nIdealSize = sizeInit.cx;
m_pRowInfo[row].nIdealSize = sizeInit.cy;

BOOL bSendInitialUpdate = FALSE;

CCreateContext contextT;
if (pContext == NULL)
{
// if no context specified, generate one from the currently selected
// client if possible
CView* pOldView = (CView*)GetActivePane();
if (pOldView != NULL && pOldView->IsKindOf(RUNTIME_CLASS(CView)))
{
// set info about last pane
ASSERT(contextT.m_pCurrentFrame == NULL);
contextT.m_pLastView = pOldView;
contextT.m_pCurrentDoc = pOldView->GetDocument();
if (contextT.m_pCurrentDoc != NULL)
contextT.m_pNewDocTemplate =
contextT.m_pCurrentDoc->GetDocTemplate();
}
pContext = &contextT;
bSendInitialUpdate = TRUE;
}

CWnd* pWnd;
TRY
{
pWnd = (CWnd*)pViewClass->CreateObject();
if (pWnd == NULL)
AfxThrowMemoryException();
}
CATCH_ALL(e)
{
TRACE0("Out of memory creating a splitter pane.\n");
// Note: DELETE_EXCEPTION(e) not required
return FALSE;
}
END_CATCH_ALL

ASSERT_KINDOF(CWnd, pWnd);
ASSERT(pWnd->m_hWnd == NULL); // not yet created

DWORD dwStyle = AFX_WS_DEFAULT_VIEW;
// if (afxData.bWin4)
// dwStyle &= ~WS_BORDER;

// Create with the right size (wrong position)
CRect rect(CPoint(0,0), sizeInit);
if (!pWnd->Create(NULL, NULL, dwStyle,
rect, this, IdFromRowCol(row, col), pContext))
{
TRACE0("Warning: couldn't create client pane for splitter.\n");
// pWnd will be cleaned up by PostNcDestroy
return FALSE;
}
ASSERT((int)((UINT)(WORD)::GetDlgCtrlID(pWnd->m_hWnd)) == IdFromRowCol(row, col));

// send initial notification message
if (bSendInitialUpdate)
pWnd->SendMessage(WM_INITIALUPDATE);

return TRUE;
}

void CSplitterWndEx::OnPaint()
{
ASSERT_VALID(this);
CPaintDC dc(this);

CRect rectClient;
GetClientRect(&rectClient);
rectClient.InflateRect(-m_cxBorder, -m_cyBorder);

CRect rectInside;
GetInsideRect(rectInside);

// draw the splitter boxes
if (m_bHasVScroll && m_nRows < m_nMaxRows)
{
OnDrawSplitter(&dc, splitBox,
CRect(rectInside.right /*+ afxData.bNotWin4*/, rectClient.top,
rectClient.right, rectClient.top + m_cySplitter));
}

if (m_bHasHScroll && m_nCols < m_nMaxCols)
{
OnDrawSplitter(&dc, splitBox,
CRect(rectClient.left, rectInside.bottom /*+ afxData.bNotWin4*/,
rectClient.left + m_cxSplitter, rectClient.bottom));
}

// extend split bars to window border (past margins)
DrawAllSplitBars(&dc, rectInside.right, rectInside.bottom);

if (1/*!afxData.bWin4*/)
{
// draw splitter intersections (inside only)
GetInsideRect(rectInside);
dc.IntersectClipRect(rectInside);
CRect rect;
rect.top = rectInside.top;
for (int row = 0; row < m_nRows - 1; row++)
{
rect.top += m_pRowInfo[row].nCurSize + m_cyBorderShare;
rect.bottom = rect.top + m_cySplitter;
rect.left = rectInside.left;
for (int col = 0; col < m_nCols - 1; col++)
{
rect.left += m_pColInfo[col].nCurSize + m_cxBorderShare;
rect.right = rect.left + m_cxSplitter;
OnDrawSplitter(&dc, splitIntersection, rect);
rect.left = rect.right + m_cxBorderShare;
}
rect.top = rect.bottom + m_cxBorderShare;
}
}
/*
CWnd* pView = CViewEx::GetActive();
if( pView != NULL )
{
pView->GetClientRect(&rectClient);
static CWnd* pPreView = pView;
if( pPreView != pView )
{
if( ::IsWindow(pPreView->m_hWnd) )
{
pPreView->GetClientRect(&rectClient);
DrawFocus(rectClient);
}
pPreView = pView;
}
DrawFocus(rectClient);
}
*/
}