编写高质量代码改善C#程序的157个建议——建议148：不重复代码

建议148：不重复代码

如果发现重复的代码，则意味着我们需要整顿一下，在继续前进。

重复的代码让我们的软件行为不一致。举例来说，如果存在两处相同的加密代码。结果在某一天，我们发现加密代码有个小Bug，然后修改了它，却又忘记了角落里的某处存在着一份相同的代码，那么这个Bug就会隐藏起来。

让我们重现这个例子：

void PagerEncrypt()
{
//加密代码
}

void AnswerEncrypt()
{
//相同的加密代码
}

在这段代码中，方法PagerEncrypt和AnswerEncrypt使用了相同的加密代码。在实际使用过程中，这两个方法可能不会靠得如此近，它们甚至可能在不同的命名空间下。糟糕的是，重构人员在进行重构的时候，没有发现AnswerEncrypt，于是代码结构变成了：

void PagerEncrypt()
{
Encrypt("密文");
}

void AnswerEncrypt()
{
//相同的加密代码
}

string Encrypt(string intput)
{
string output = string.Empty;
//加密代码
return output;
}

在开发人员在Encrypt中发现Bug并修正它后，两端原本相同的加密代码变成不同的了。一份值正确的，一份是错误的。而对于这些代码的调用者来说，他看到了类似的方法接口，却没想到其内部实现是完全不一样的。这增加了调用者的排错难度。

所以，对于开发者来说，务必注意不要让代码中出现相同的片段，这不仅是为了代码的整洁，还可以避免更多难以察觉的Bug的出现。

转自：《编写高质量代码改善C#程序的157个建议》陆敏技