先决条件:Don'；不要重复自己或单一责任原则

在升级一些旧代码时，我发现这两个OO原则似乎相互冲突。

考虑以下伪代码（这是我遇到的简化版本）：

int numberOfNewRecords;
int numberOfOldRecords;
int numberOfUndefinedRecords;
void ColourAndCount()
{
    foreach(Row row in dataGridView1.Rows)
    {
        switch(row.Cells["Status"])
        {
            case: "Old"
                row.BackColor = Color.Red;
                numberOfOldRecords++;
                break;
            case: "New"
                row.BackColor = Color.Blue;
                numberOfNewRecords++;
                break;
            default:
                row.BackColor = Color.White;
                numberOfUndefinedRecords++;
                break;
        }
    }
}

这段代码做了两件事：它根据记录的状态统计记录的数量，还根据它们的状态为每一行着色。这很混乱，但由于这两个操作（到目前为止）总是同时调用，所以它没有造成任何问题，并使添加额外状态等维护要求变得容易。

尽管如此，单一责任原则告诉我应该把它分成两种不同的方法：

（编辑）小提示：我刚刚意识到我可能在这里滥用了"单一责任原则"这个词，据我所知，它指的是阶级。"每种方法一个操作"设计模式的术语是什么？

int numberOfNewRecords;
int numberOfOldRecords;
int numberOfUndefinedRecords;
void Count()
{
    foreach(Row row in dataGridView1.Rows)
    {
        switch(row.Cells["Status"])
        {
            case: "Old"
                numberOfOldRecords++;
                break;
            case: "New"
                numberOfNewRecords++;
                break;
            default:
                numberOfUndefinedRecords++;
                break;
        }
    }
}
void Colour()
{
    foreach(Row row in dataGridView1.Rows)
    {
        switch(row.Cells["Status"])
        {
            case: "Old"
                row.BackColor = Color.Red;
                break;
            case: "New"
                row.BackColor = Color.Blue;
                break;
            default:
                row.BackColor = Color.White;
                break;
        }
    }
}

但这违反了Don't Repeat Yourself：循环和switch语句在两个方法中都是重复的，而且由于此代码最有可能的升级路径是添加其他Statues，这会使未来的升级更加困难，而不是更少。

我很难找到最优雅的重构方法，所以我觉得最好向社区咨询，以防我明显遗漏了什么。你将如何应对这种情况？

（编辑）

我想出了一个可能的解决方案，但在我看来，这就像是一个过度设计简单问题的例子（它并没有真正解决最初的单一责任问题）。

struct Status
{
    public string Name,
    public int Count,
    public Color Colour,
}
Dictionary<string, Status> StatiiDictionary = new Dictionary<string, int>();
void Initialise()
{
    StatiiDictionary.Add(new Status("New", 0, Color.Red));
    StatiiDictionary.Add(new Status("Old", 0, Color.Blue));
    StatiiDictionary.Add(new Status("Undefined", 0, Color.White));
}
void ColourAndCountAllRows()
{
    foreach(Row row in dataGridView1.Rows)
    {
        CountRow(row, StatiiDictionary);
        ColourRow(row, StatiiDictionary);
    }
}
void CountRow(Row row, Dictionary<string, Status> StatiiDictionary)
{
    StatiiDictionary[row.Cells["Status"]].Count++; 
}
void ColourRow(Row row, Dictionary<string, Status> StatiiDictionary)
{
    row.BackColour = StatiiDictionary[row.Cells["Status"]].Colour;
}