我试图弄清楚我是否应该开始使用更多的内部访问修饰符。

我知道如果我们使用internal并设置程序集变量InternalsVisibleTo,我们可以测试不想在测试项目中声明为公共的函数。

这让我觉得我应该总是使用internal,因为至少每个项目(应该?)都有自己的测试项目。

为什么不应该这样做呢?什么时候应该使用private?


当前回答

你也可以使用private,你可以调用带有反射的private方法。如果你使用Visual Studio Team Suite,它有一些很好的功能,会生成一个代理来调用你的私有方法。下面是一篇代码项目文章,演示了如何自己对私有方法和受保护方法进行单元测试:

http://www.codeproject.com/KB/cs/testnonpublicmembers.aspx

至于应该使用哪个访问修饰符,我的一般经验法则是先使用private,然后根据需要逐步升级。通过这种方式,您将尽可能少地暴露类的内部细节,这有助于保持实现细节的隐藏,就像它们应该隐藏的那样。

其他回答

将InternalsVisibleTo.cs文件添加到项目根文件夹中。csproj文件。

InternalsVisibleTo.cs的内容应该如下

using System.Runtime.CompilerServices;

[assembly: InternalsVisibleTo("AssemblyName.WhichNeedAccess.Example.UnitTests")]

你也可以使用private,你可以调用带有反射的private方法。如果你使用Visual Studio Team Suite,它有一些很好的功能,会生成一个代理来调用你的私有方法。下面是一篇代码项目文章,演示了如何自己对私有方法和受保护方法进行单元测试:

http://www.codeproject.com/KB/cs/testnonpublicmembers.aspx

至于应该使用哪个访问修饰符,我的一般经验法则是先使用private,然后根据需要逐步升级。通过这种方式,您将尽可能少地暴露类的内部细节,这有助于保持实现细节的隐藏,就像它们应该隐藏的那样。

默认使用private。如果一个成员不应该在该类型之外公开,那么它也不应该在该类型之外公开,即使是在同一个项目中。这使事情更安全、更整洁——当你使用对象时,你可以更清楚地使用哪些方法。

话虽如此,我认为有时出于测试目的将自然私有方法设置为内部方法是合理的。比起反射,我更喜欢这种方法,因为反射对重构不友好。

需要考虑的一件事可能是“ForTest”后缀:

internal void DoThisForTest(string name)
{
    DoThis(name);
}

private void DoThis(string name)
{
    // Real implementation
}

然后,当您在同一个项目中使用该类时,很明显(现在和将来)您不应该真正使用这个方法——它只是用于测试目的。这有点俗气,我自己也不这么做,但至少值得考虑。

在。net Core 2.2中,将这一行添加到Program.cs中:

using ...
using System.Runtime.CompilerServices;

[assembly: InternalsVisibleTo("MyAssembly.Unit.Tests")]

namespace
{
...

除了Eric的回答,你还可以在csproj文件中配置:

<ItemGroup>
    <AssemblyAttribute Include="System.Runtime.CompilerServices.InternalsVisibleTo">
      <_Parameter1>MyTests</_Parameter1>
    </AssemblyAttribute>
</ItemGroup>

或者如果你每个项目都有一个测试项目要测试,你可以在Directory.Build.props文件中这样做:

<ItemGroup>
    <AssemblyAttribute Include="System.Runtime.CompilerServices.InternalsVisibleTo">
      <_Parameter1>$(MSBuildProjectName).Test</_Parameter1>
    </AssemblyAttribute>
</ItemGroup>

参见:https://stackoverflow.com/a/49978185/1678053 例如:https://github.com/gldraphael/evlog/blob/master/Directory.Build.props L5-L12