主持人注:这里已经有39个答案了(有些已经删除了)。在你发表你的答案之前，考虑一下你是否可以为讨论添加一些有意义的东西。你很可能只是在重复别人已经说过的话。
我偶尔发现自己需要将类中的私有方法设为public，只是为了为它编写一些单元测试。
通常这是因为该方法包含类中其他方法之间共享的逻辑，并且单独测试逻辑更整洁，或者另一个原因可能是我想测试同步线程中使用的逻辑，而不必担心线程问题。
其他人发现他们这样做是因为我不喜欢吗?我个人认为，公开一个方法的好处超过了它在类之外没有提供任何服务的问题……
更新
谢谢大家的回答，似乎引起了大家的兴趣。我认为普遍的共识是测试应该通过公共API进行，因为这是使用类的唯一方式，我非常同意这一点。在我上面提到的几个案例中，我会这样做，这是不常见的情况，我认为这样做的好处是值得的。
然而，我可以看到，每个人都指出它不应该真的发生。再仔细想想，我觉得改变你的代码来适应测试是一个坏主意——毕竟我认为测试在某种程度上是一个支持工具，而改变一个系统来“支持一个支持工具”是明显的坏做法。
当前回答
如果你正在使用c#，你可以让方法是内部的。这样就不会污染公共API。
然后将属性添加到dll
[组装:InternalsVisibleTo(“MyTestAssembly”)]
现在所有的方法都在MyTestAssembly项目中可见。也许不完美，但总比为了测试而将私有方法设为公共要好。
其他回答
许多答案建议只测试公共接口，但恕我直言，这是不现实的——如果一个方法执行的操作需要5个步骤，那么您将希望分别测试这5个步骤，而不是一起测试。这需要测试所有五个方法，否则它们(除了测试之外)可能是私有的。
测试“私有”方法的通常方法是给每个类一个自己的接口，并将“私有”方法设为公共的，但不将它们包含在接口中。这样，它们仍然可以被测试，但不会使界面膨胀。
是的，这将导致文件和类膨胀。
是的，这确实使公共和私有说明符变得多余。
是啊，这真让人头疼。
不幸的是，这是我们为使代码可测试而做出的众多牺牲之一。也许未来的语言(或者甚至是c# /Java的未来版本)将具有使类和模块可测试性更加方便的特性;但与此同时，我们得跳过这些障碍。
There are some who would argue that each of those steps should be its own class, but I disagree - if they all share state, there is no reason to create five separate classes where five methods would do. Even worse, this results in file- and class-bloat. Plus, it infects the public API of your module - all those classes must necessarily be public if you want to test them from another module (either that, or include the test code in the same module, which means shipping the test code with your product).
把它包成私有的怎么样?然后您的测试代码可以看到它(以及包中的其他类)，但它仍然对用户隐藏。
但实际上，您不应该测试私有方法。这些是实施细节，不是合同的一部分。它们所做的所有事情都应该通过调用公共方法来覆盖(如果它们的代码不是由公共方法执行的，那么应该去掉)。如果私有代码太复杂，类可能做了太多的事情，需要重构。
公开一个方法是一个很大的承诺。一旦你这样做了，人们就能使用它，你不能再改变他们了。
当您想要测试的某个方法中有复杂的逻辑时，这是一个很好的指标，表明该类违反了单一责任原则。
一个很好的解决方案是:
为原始方法的功能创建一个接口。 在类中实现并测试该接口。 将接口注入到原始类中。
更新:我已经在其他很多地方对这个问题增加了一个更广泛、更完整的答案。这可以在我的博客上找到。
如果我需要公开一些东西来测试它，这通常暗示被测试的系统没有遵循单一责任原则。因此，应该引入一个缺失的类。将代码提取到一个新类后，将其设为public。现在您可以轻松地进行测试，并且遵循SRP。您的其他类只需通过组合调用这个新类。
使方法公开/使用语言技巧，例如将代码标记为对测试程序集可见，应该始终是最后的手段。
例如:
public class SystemUnderTest
{
public void DoStuff()
{
// Blah
// Call Validate()
}
private void Validate()
{
// Several lines of complex code...
}
}
通过引入验证器对象重构此对象。
public class SystemUnderTest
{
public void DoStuff()
{
// Blah
validator.Invoke(..)
}
}
现在我们要做的就是测试验证器是否被正确调用。验证的实际过程(以前的私有逻辑)可以在完全隔离的情况下进行测试。不需要复杂的测试设置来确保验证通过。
如果你正在使用c#，你可以让方法是内部的。这样就不会污染公共API。
然后将属性添加到dll
[组装:InternalsVisibleTo(“MyTestAssembly”)]
现在所有的方法都在MyTestAssembly项目中可见。也许不完美，但总比为了测试而将私有方法设为公共要好。
