c# - 是否有充分的理由在非静态类中使用静态方法？

Question

标题几乎是我的整个问题。

我正在为一位离岸开发人员进行代码审查，该开发人员在非静态类中拥有多个静态方法。在我向开发人员提出质疑并将其标记为“需要更改”之前，我只想确定一下。

我明白了静态类的用途:它不能被实例化，可以直接使用。但我看不出有任何理由在非静态类中使用静态方法。这有任何有效的用例吗？

相关方法都是私有(private)，并且是从非静态方法调用的。

这是一个例子:

    public ViewResult ClaimDetails(ClaimDetails claim)
    {
        if (claim.ClaimNumber != 0)
        {
            claim = Get_ClaimDetails(claim);
        }
        return View("ClaimDetails", claim);
    }

    private static ClaimDetails Get_ClaimDetails(ClaimDetails claim)
    {
        ClaimsRepository claimsRepository = new ClaimsRepository();
        _claimDetails = new ClaimDetails();
        _claimDetails = claimsRepository.GetClaimDetails(claim.ClaimNumber);
        return _claimDetails;
    }

Answer 1

有一些事情会让我感到惊讶，但方法签名很好。

static 是一个关键字，可确保该方法不使用实例成员。那太棒了。它向其他开发人员清楚地表明了这一点。我会主张相反的观点:方法应该标记为静态，除非需要时。

如果您没有将其标记为静态，许多工具甚至 Visual Studio 本身都会标记它:CA1822: Mark members as static .

引用微软的话:

Members that do not access instance data or call instance methods can be marked as static (Shared in Visual Basic). After you mark the methods as static, the compiler will emit nonvirtual call sites to these members. Emitting nonvirtual call sites will prevent a check at run time for each call that makes sure that the current object pointer is non-null. This can achieve a measurable performance gain for performance-sensitive code. In some cases, the failure to access the current object instance represents a correctness issue.

总结一下:它本身就很好。

---

阅读所有附加评论:看来真正的问题是注入(inject)的存储库已经存在，整个方法应该废弃并替换为行 claim = this.injectedClaimsRepository.GetClaimDetails(claim.ClaimNumber);.但这完全是另一个问题。如果使用正确，关键字 static 是完全没问题的，发布的代码没有显示它不是。