重构多重循环的一个实例

最新推荐文章于 2023-03-20 23:22:09 发布

acloud_csu

最新推荐文章于 2023-03-20 23:22:09 发布

阅读量2.1k

点赞数 1

分类专栏： C/C++ 文章标签： c#

本文链接：https://blog.csdn.net/acloudhuang/article/details/7166694

版权

C/C++ 专栏收录该内容

72 篇文章 3 订阅

订阅专栏

重构多重循环的一个实例

黄国强 2011-12-31

一位同事提供了以下这段C#示例代码。

List<MaterialInfo> returnValue = new List<MaterialInfo>();
for (int i = 0; i < totalPoints.Count - 1; i++)
{
    CoordsDouble leftPoint = totalPoints[i];
    CoordsDouble rightPoint = totalPoints[i + 1];

    Boolean hasFound = false;
    foreach (KeyValuePair<ModelInfo2D, IList<CoordsDouble>> kvp in modelPointDictionary)
    {
        for (int j = 0; j < kvp.Value.Count - 1; j += 2)
        {
            if (kvp.Value[j] == leftPoint && kvp.Value[j + 1] == rightPoint)
            {
                returnValue.Add(kvp.Key.Material);
                hasFound = true;
                break;
            }
        }

        if (hasFound)
        {
            break;
        }
    }

    if (!hasFound)
    {
        returnValue.Add(mGrating.IncidenceMaterial);
    }
}

return returnValue;

以下是我重构后的代码示例，通过将原先循环内的代码移到新的函数中，提高了代码的可读性。

List<MaterialInfo> func1()
{
    List<MaterialInfo> returnValue = new List<MaterialInfo>();
    for (int i = 0; i < totalPoints.Count - 1; i++)
    {
        CoordsDouble leftPoint = totalPoints[i];
        CoordsDouble rightPoint = totalPoints[i + 1];
        func2(returnValue, leftPoint, rightPoint);
    }
    return returnValue;
}

void func2(returnValue, leftPoint, rightPoint))
{
    foreach (KeyValuePair<ModelInfo2D, IList<CoordsDouble>> kvp in modelPointDictionary)
    {
   if(func3(kvp,leftPoint,rightPoint))
        return ;
    }
    returnValue.Add(mGrating.IncidenceMaterial);
}

bool func3(returnValue.Add,kvp, leftPoint, rightPoint)
{
   for (int j = 0; j < kvp.Value.Count - 1; j += 2)
   {
        if (kvp.Value[j] == leftPoint && kvp.Value[j + 1] == rightPoint)
        {
            returnValue.Add(kvp.Key.Material);
            return true;
        }
   }
   return false;
}

重构是门非常重要的技术，对于不会设计的初学者，重构是个不错的起点。