主持人注:这里已经有39个答案了(有些已经删除了)。在你发表你的答案之前，考虑一下你是否可以为讨论添加一些有意义的东西。你很可能只是在重复别人已经说过的话。
我偶尔发现自己需要将类中的私有方法设为public，只是为了为它编写一些单元测试。
通常这是因为该方法包含类中其他方法之间共享的逻辑，并且单独测试逻辑更整洁，或者另一个原因可能是我想测试同步线程中使用的逻辑，而不必担心线程问题。
其他人发现他们这样做是因为我不喜欢吗?我个人认为，公开一个方法的好处超过了它在类之外没有提供任何服务的问题……
更新
谢谢大家的回答，似乎引起了大家的兴趣。我认为普遍的共识是测试应该通过公共API进行，因为这是使用类的唯一方式，我非常同意这一点。在我上面提到的几个案例中，我会这样做，这是不常见的情况，我认为这样做的好处是值得的。
然而，我可以看到，每个人都指出它不应该真的发生。再仔细想想，我觉得改变你的代码来适应测试是一个坏主意——毕竟我认为测试在某种程度上是一个支持工具，而改变一个系统来“支持一个支持工具”是明显的坏做法。
当前回答
这都是实用主义。你的单元测试在某种程度上是代码的客户端，为了达到良好的代码覆盖率，你需要让你的代码非常可测试。如果为了让您能够在代码中没有有效的公共接缝的情况下设置必要的极端情况，测试代码非常复杂，那么您的解决方案将是潜在的失败。使用IoC也有助于解决这个问题。
其他回答
我想说这是一个坏主意，因为我不确定你是否会得到任何好处，并可能出现问题。如果你改变一个调用的契约，只是为了测试一个私有方法，你并不是在测试这个类将如何使用它，而是在创建一个你从未打算发生的人为场景。
此外，通过将方法声明为公共，在六个月的时间内(在忘记将方法公开的唯一原因是为了测试之后)，您(或者如果您已经移交了项目)完全不同的人将不会使用它，从而导致潜在的意想不到的后果和/或维护噩梦。
不，因为有更好的方法来剥猫皮。
一些单元测试利用依赖于类定义中的宏，当在测试模式中构建时，这些宏会自动扩展以创建钩子。很有C风格，但是很好用。
一个更简单的OO习惯用法是使您想测试的任何东西都是“受保护的”，而不是“私有的”。测试工具继承自被测试类，然后可以访问所有受保护的成员。
或者你选择“朋友”。就我个人而言，这是我最不喜欢的c++特性，因为它打破了封装规则，但它恰好是c++实现某些特性所必需的。
无论如何，如果您正在进行单元测试，那么您很可能需要向这些成员中注入值。白盒短信是完全有效的。这真的会破坏您的封装。
Guava有一个@VisibleForTesting注释，用于标记那些扩大了作用域(包或公共)的方法。我使用@Private注释来做同样的事情。
虽然必须测试公共API，但有时获取通常不是公共的东西是方便和明智的。
当:
通过将一个类分解为多个类，类的可读性会显著降低， 为了让它更容易测试， 并且提供一些对内部的测试访问就可以做到这一点
宗教似乎战胜了工程学。
You should never ever ever let your tests dictate your code. I'm not speaking about TDD or other DDs I mean, exactly what your asking. Does your app need those methods to be public. If it does then test them. If it does not then then don't make them public just for testing. Same with variables and others. Let your application's needs dictate the code, and let your tests test that the need is met. (Again I don't mean testing first or not I mean changing a classes structure to meet a testing goal).
相反，你应该“考高一点”。测试调用私有方法的方法。但是您的测试应该测试您的应用程序需求，而不是您的“实现决策”。
例如(此处为bod伪代码);
public int books(int a) {
return add(a, 2);
}
private int add(int a, int b) {
return a+b;
}
没有理由测试“add”，你可以测试“books”。
永远不要让你的测试为你做代码设计决策。测试你是否得到了预期的结果，而不是你如何得到结果。
把它包成私有的怎么样?然后您的测试代码可以看到它(以及包中的其他类)，但它仍然对用户隐藏。
但实际上，您不应该测试私有方法。这些是实施细节，不是合同的一部分。它们所做的所有事情都应该通过调用公共方法来覆盖(如果它们的代码不是由公共方法执行的，那么应该去掉)。如果私有代码太复杂，类可能做了太多的事情，需要重构。
公开一个方法是一个很大的承诺。一旦你这样做了，人们就能使用它，你不能再改变他们了。
