这样的功能如何：
void Map::Display()
{
if(initialized)
{
HRESULT hr;
int hScrollPos = GetScrollPos(M_HWnd, SB_HORZ);
int vScrollPos = GetScrollPos(M_HWnd, SB_VERT);
D2D1_RECT_F region = {0,0,TILE_WIDTH,TILE_HEIGHT};
D2D1_RECT_F tFRegion = {0,0,TILE_WIDTH,21}; // tile front's region
Coor coor;
int tileHeight;
RECT rect;
GetWindowRect(M_HWnd, &rect);
int HWndWidth = rect.right - rect.left;
int HWndHeight = rect.bottom - rect.top;
pRT->BeginDraw();
pRT->Clear(D2D1::ColorF(0.45f, 0.76f, 0.98f, 1.0f));
pRT->SetAntialiasMode(D2D1_ANTIALIAS_MODE_ALIASED);
for(int x=0; x<nTiles; x++)
{
coor = ppTile[x]->Getcoor();
tileHeight = ppTile[x]->Getheight();
pRT->SetTransform(D2D1::Matrix3x2F::Identity());
if((coor.GetX() - 1) * (TILE_WIDTH * 0.5) - hScrollPos > 0 - TILE_WIDTH &&
(coor.GetX() - 1) * (TILE_WIDTH * 0.5) - hScrollPos < HWndWidth &&
((coor.GetY() - 1) * (TILE_HEIGHT * 0.5) * 1.5f) + ((MAX_MAP_HEIGHT - tileHeight) * (TILE_PIXEL_PER_LAYER)) + TILE_HEIGHT - vScrollPos > 0 - (TILE_HEIGHT * 2.5) &&
((coor.GetY() - 1) * (TILE_HEIGHT * 0.5) * 1.5f) + ((MAX_MAP_HEIGHT - tileHeight) * (TILE_PIXEL_PER_LAYER)) + TILE_HEIGHT - vScrollPos < HWndHeight)
{
/* Draws tiles */
pRT->SetTransform(D2D1::Matrix3x2F::Translation(
(coor.GetX() - 1) * (TILE_WIDTH * 0.5) - hScrollPos,
((coor.GetY() - 1) * (TILE_HEIGHT * 0.5) * 1.5f) + ((MAX_MAP_HEIGHT - tileHeight) * (TILE_PIXEL_PER_LAYER)) + TILE_HEIGHT - vScrollPos
));
pRT->FillRectangle( ®ion, pBmpTileBrush[ppTile[x]->GetType() + 1]);
/* Draws tiles' front */
if((coor.Y - 1) / 2 < mapSizeY - 1) // If we are not in the front row,
{
if(coor.X > 1)
{
for(int diffH = tileHeight - ppTile[x + mapSizeX - 1]->Getheight(); diffH == 0; diffH--)
{
pRT->SetTransform(D2D1::Matrix3x2F::Identity());
pRT->SetTransform(D2D1::Matrix3x2F::Translation(
(coor.GetX() - 1) * (TILE_WIDTH * 0.5) - hScrollPos,
((coor.GetY() - 1) * (TILE_HEIGHT * 0.5) * 1.5f) + ((MAX_MAP_HEIGHT - tileHeight) * (TILE_PIXEL_PER_LAYER)) + TILE_HEIGHT - vScrollPos + (TILE_HEIGHT * 0.75) + (diffH * TILE_PIXEL_PER_LAYER)
));
pRT->FillRectangle( &tFRegion, pBmpTileFrontBrush[ppTile[x]->GetType()]);
}
}
if(((coor.X -1) / 2) + 1 < mapSizeX)
{
for(int diffH = tileHeight - ppTile[x + mapSizeX]->Getheight(); diffH == 0; diffH--)
{
pRT->SetTransform(D2D1::Matrix3x2F::Identity());
pRT->SetTransform(D2D1::Matrix3x2F::Translation(
(coor.GetX() - 1) * (TILE_WIDTH * 0.5) - hScrollPos,
((coor.GetY() - 1) * (TILE_HEIGHT * 0.5) * 1.5f) + ((MAX_MAP_HEIGHT - tileHeight) * (TILE_PIXEL_PER_LAYER)) + TILE_HEIGHT - vScrollPos + (TILE_HEIGHT * 0.75) + (diffH * TILE_PIXEL_PER_LAYER)
));
pRT->FillRectangle( &tFRegion, pBmpTileFrontBrush[ppTile[x]->GetType()]);
}
}
if(coor.X == 1 || (coor.X - 1) / 2 == mapSizeY - 1) // If the tile if at any of left or right edge,
{
for(int n = ((TH * 1.5) / TPPL) - (ppTile[x + mapSizeY + mapSizeY - 1]->Getheight() - tileHeight); n>=0; n--)
{
pRT->SetTransform(D2D1::Matrix3x2F::Identity());
pRT->SetTransform(D2D1::Matrix3x2F::Translation(
(coor.X - 1) * (TILE_WIDTH * 0.5) - hScrollPos,
((coor.Y - 1) * (TILE_HEIGHT * 0.5) * 1.5f) + ((MAX_MAP_HEIGHT - tileHeight) * (TILE_PIXEL_PER_LAYER)) + TILE_HEIGHT - vScrollPos + (TILE_HEIGHT * 0.75) + (n * TILE_PIXEL_PER_LAYER)
));
pRT->FillRectangle( &tFRegion, pBmpTileFrontBrush[ppTile[x]->GetType()]);
}
}
}
else // If we are in the front row
{
for(int h = tileHeight; h >= 0; h--)
{
pRT->SetTransform(D2D1::Matrix3x2F::Identity());
pRT->SetTransform(D2D1::Matrix3x2F::Translation(
(coor.GetX() - 1) * (TILE_WIDTH * 0.5) - hScrollPos,
((coor.GetY() - 1) * (TILE_HEIGHT * 0.5) * 1.5f) + ((MAX_MAP_HEIGHT - tileHeight) * (TILE_PIXEL_PER_LAYER)) + TILE_HEIGHT - vScrollPos + (TILE_HEIGHT * 0.75) + (h * TILE_PIXEL_PER_LAYER)
));
pRT->FillRectangle( &tFRegion, pBmpTileFrontBrush[ppTile[x]->GetType()]);
}
}
}
}
pRT->SetAntialiasMode(D2D1_ANTIALIAS_MODE_PER_PRIMITIVE);
hr = pRT->EndDraw();
}
}
这样：
Tile* Map::GetClickedTile(short xPos, short yPos)
{
Tile* pNoClickedTile = NULL;
int hScrollPos = GetScrollPos(M_HWnd, SB_HORZ);
int vScrollPos = GetScrollPos(M_HWnd, SB_VERT);
if(xPos < (mapSizeX * TILE_WIDTH) - hScrollPos) // If the click is within width of the map then...
{
Coor coor;
int height;
int currentTile;
int tileDistanceFromTop;
/* Checks if click is in an odd row of tiles */
int column = (xPos + hScrollPos) / TILE_WIDTH;
for (int y=mapSizeY-1; y>=0; y--)
{
currentTile = column + (y * (mapSizeX+mapSizeX-1));
coor = ppTile[currentTile]->Getcoor();
height = ppTile[currentTile]->Getheight();
tileDistanceFromTop = ((coor.Y / 2) * TILE_HEIGHT * 1.5f) + // Distance between two tiles
( (MAX_MAP_HEIGHT - height) * TILE_PIXEL_PER_LAYER) -
vScrollPos +
SPACE_LEFT_FOR_BACKGROUND;
/*if (tileDistanceFromTop < 0) // If the tile is partially hidden,
tileDistanceFromTop = tileDistanceFromTop % TILE_HEIGHT; // then % TILE_HEIGHT*/
if( yPos > tileDistanceFromTop &&
yPos < tileDistanceFromTop + TILE_HEIGHT)
{
/* Get relative coordinates */
int rpx = xPos % TILE_WIDTH;
int rpy = ( (yPos - SPACE_LEFT_FOR_BACKGROUND) -
(y * (TILE_HEIGHT /2) ) -
( ( MAX_MAP_HEIGHT - height) * TILE_PIXEL_PER_LAYER) +
vScrollPos) %
TILE_HEIGHT;
/* Checks if click is withing area of current tile */
if (rpy + (rpx / (TILE_WIDTH /16)) > TILE_HEIGHT * 0.25f && // if click is Down Right the Upper Left slope and,
rpy + (rpx / (TILE_WIDTH /16)) < TILE_HEIGHT * 1.25f && // it is UL the LR slope and,
rpy - (rpx / (TILE_WIDTH /16)) < TILE_HEIGHT * 0.75f && // it is UR the LL slope and,
rpy - (rpx / (TILE_WIDTH /16)) > TILE_HEIGHT * -0.25f) // it is DL the UR slope,
return ppTile[currentTile]; // Then return currentTile
}
}
/* Checks if click is in an even row of tiles */
column = (xPos + hScrollPos - (TILE_WIDTH/2)) / TILE_WIDTH;
for (int y=mapSizeY-2; y>=0; y--)
{
currentTile = column + (y * (mapSizeX+mapSizeX-1)) + mapSizeX;
coor = ppTile[currentTile]->Getcoor();
height = ppTile[currentTile]->Getheight();
tileDistanceFromTop = (((coor.Y - 1) / 2) * TILE_HEIGHT * 1.5f) + // Distance between two tiles
( (MAX_MAP_HEIGHT - height) * TILE_PIXEL_PER_LAYER) +
(TILE_HEIGHT * 0.75) -
vScrollPos +
SPACE_LEFT_FOR_BACKGROUND;
/*if (tileDistanceFromTop < 0)
tileDistanceFromTop = tileDistanceFromTop % TILE_HEIGHT;*/
if( yPos > tileDistanceFromTop &&
yPos < tileDistanceFromTop + TILE_HEIGHT)
{
/* Get relative coordinates */
int rpx = xPos % TILE_WIDTH;
int rpy = (int)((yPos - SPACE_LEFT_FOR_BACKGROUND) -
(y * (TILE_HEIGHT /2) ) -
( ( MAX_MAP_HEIGHT - height) * TILE_PIXEL_PER_LAYER) -
(TILE_HEIGHT * 0.675) +
vScrollPos) %
TILE_HEIGHT;
/* Checks if click is withing area of current tile */
if (rpy + (rpx / (TILE_WIDTH /16)) > TILE_HEIGHT * 0.25f && // if click is Down Right the Upper Left slope and,
rpy + (rpx / (TILE_WIDTH /16)) < TILE_HEIGHT * 1.25f && // it is UL the LR slope and,
rpy - (rpx / (TILE_WIDTH /16)) < TILE_HEIGHT * 0.75f && // it is UR the LL slope and,
rpy - (rpx / (TILE_WIDTH /16)) > TILE_HEIGHT * -0.25f) // it is DL the UR slope,
return ppTile[currentTile]; // Then return currentTile // Then return currentTile
}
}
}
return pNoClickedTile;
}
甚至这个：
int Map::GetTileNByCoor(Coor coor)
{
return ((coor.X / 2 + ((coor.Y - 1) * mapSizeY) - (coor.Y / 2));
}
变得更容易阅读？随着我的代码变得越来越大，我意识到，如果不是必要的话，有一个干净，易于阅读的代码是多么重要。有哪些提示可以使上面的代码更清洁？
答案 0 :(得分：4)
我的一般重构实践通常是执行以下操作：
拉出代码中不明显的名称。您可以使用局部变量为小块代码提供定义名称。那么，在你最后一个例子的情况下，(coor.X / 2 + ((coor.Y - 1) * mapSizeY)表示什么？
在大多数情况下，最好让事情名称好，而不是担心存储局部变量（当堆栈离开函数时它们会被删除，而且通常你不会太担心代码的内存空间/速度如此精细的颗粒）。
将执行代码组拉出到方法中。一个好的经验法则是，如果你的函数超过6行代码，你可以在其中拉出一个较小的函数。然后你的代码会更好地阅读它实际上正在做的事情。
一个非常常见的地方是循环。您几乎总是可以将循环中的代码拉入其自己的函数中，并具有良好的描述性名称。
拔出方法后，您可以将常用共享功能分组为较小的对象。让较小的物体一起工作来完成工作几乎总是好的，而不是让巨大的物体做很多工作。您希望每个对象都有一个责任。
答案 1 :(得分：1)
非常扎实的代码，干得好。我会考虑：
在高级别注释函数本身，然后为代码中的所有重要块添加更好的注释，以及任何异常棘手的事情。
对您正在使用的所有魔术变量使用描述性的consts或#defines。为什么乘以0.675？ 0.675代表什么？同上0.25,1.25，-0.25等
将“检查点击是否包含当前图块的区域”测试（以及其他方法）转换为您调用的单独方法，例如isClickInsideTile（x，y，tile）。
< / LI>添加调试跟踪，以便下一个负责人可以启用调试以获得诊断。
PS你的变量名和方法名称很好。