避免此代码重复的最佳方法是什么

Question

我有以下方法：

    public  int CountProperty1
    {
        get
        {
            int count = 0;
            foreach (var row in Data)
            {
                count = count + row.Property1;
            }
            return count ;
        }
    }

    public  int CountProperty2
    {
        get
        {
            int count = 0;
            foreach (var row in Data)
            {
                count = count + row.Property2;
            }
            return count ;
        }
    }

避免重复并共享尽可能多的代码的最佳方法是什么

Answer 1

使用 LINQ 和 Sum 扩展方法怎么样？

public int CountProperty1 
{
    get { return Data.Sum(r => r.Property1); }
} 

如果这不是一个选项，您可以将逻辑重构为您自己的 sum 方法：

public  int CountProperty1
{
    get
    {
        return CountProperty(r => r.Property1);
    }
}

public  int CountProperty2
{
    get
    {
        return CountProperty(r => r.Property2);
    }
}

private int CountProperty(Func<Row,int> countSelector)
{
     int count = 0;
     foreach (var row in Data)
     {
         count = count + countSelector(row);
     }
     return count ;
}

请注意最后一个示例：我编写了“Row”类型，因为从您的示例中并不明显。用正确的类型替换。

Answer 2

可能不是您正在寻找的答案，但这不一定是重复。有时存在一种误解，如果两个不同的函数看起来相同，则应该重构它们以删除重复项。以我的拙见，这是错误的。

只有当它们是真正的复制并且相同或接近相同的概念时，才算是重复。对于重复（至少应该删除重复），不仅仅是它们碰巧使用相同的代码，而是出于相同的原因使用相同的代码。

这可能只是因为您发布的示例，但它很简单。尽管这两个属性的实现是相同的，但必须有一些有效的业务原因才能拥有这两个属性，否则最简单的答案就是一起删除第二个属性。但是，如果您确实有两个属性，并且它们现在恰好看起来相同，这并不意味着它们将来在功能上不会出现差异（这是可以的）。

这个想法是为了最小化复杂性和维护成本。只有当您的代码有意义并模拟现实时，您才能做到这一点，但如果它通过将恰好看起来相似的事物混在一起而引入错误的比较，则不能这样做。

http://mooneyblog.mmdbsolutions.com/ index.php/2010/07/30/reusable-code-is-bad/

Answer 3

别打扰。

您可以缩短代码，但如果您尝试重构它以消除重复，则只会使代码变得更加复杂。

将重构工作集中在更复杂、更有价值的案例上。生命太短暂...

几个字节的磁盘空间非常便宜，而复制/粘贴是开发人员最好的朋友。另外，如果我们对此很纯粹，请考虑重构的运行时开销。

Answer 4

我个人不会打扰属性，只需使用 linq 直接在外部调用

myObject.Data.Sum(x=>x.Property1)

Answer 5

这是相当干净的。如果我知道行的类型，我会倾向于将 PropertyN() 放入其类中而不是放在这里，但缺乏这些知识：

public int CountPropertyN(int n)
{
    int count = 0;
    foreach (var row in Data)
    {
        count = count + PropertyN(row, n)
    }
    return count ;
}

private int PropertyN(var row, int n) 
{
     if (n == 1) return row.Property1;
     else return row.Property2;
}

Answer 6

给它一个输入参数，说明哪个参数

public  int CountProperty (int whichProperty) 
    {
        get
        {
            int count = 0;
            foreach (var row in Data)
            {
                if( whichProperty = 1)
                    count = count + row.Property1;
                if( whichProperty = 2)
                    count = count + row.Property2;
            }
            return count ;
        }
    }

Answer 7

public  int CountProperty1
{
    get
    {
        return GetCount(row.Property1);
    }
}

public  int CountProperty2
{
    get
    {
        return GetCount(row.Property2);
    }
}

private int GetCount(object property)
{
    int count = 0;
    foreach (var row in Data)
    {
        if(property == row.Property1)
        {
            count = count + row.Property1;
        }
        else if (property == row.Property2)
        {
            count = count + row.Property2;
        }
    }
    return count ;
}

在 pribvate 方法中，我在签名中使用了对象 - 直到我对属性所需的类型有了更好的了解