SonarQube 投诉要么删除 class 这个无用的对象实例化，要么使用它

Question

我正在尝试修复我从 SonarQube 获得的这个错误，但他们的解决方案建议对我的情况没有太大帮助。我从他们那里得到的建议是：“没有充分的理由创建一个新对象而不对它做任何事情。大多数时候，这是由于缺少一段代码，因此可能导致意外行为在生产中。

如有任何关于如何处理的建议，我们将不胜感激。

private RootElement GetParentAsRoot(Element element, string method) {
        if (element.Parent == null) {
            new RootElement(element, NewControlsNotifier);
       //Either remove this useless object instantiation of class 'RootElement' or use it.
        }
        var root = element.Parent as RootElement;
        if (root == null) {
            throw new ArgumentException(method + " method is applicable only on top-most element");
        }
        return root;
    }

Answer 1

导致此问题的代码是这样的：

new RootElement(element, NewControlsNotifier);

您正在使用 new 运算符创建 RootElement 但忽略了它的结果。您应该完全删除对象创建或使用您创建的对象。

如果你想return你的对象试试这个：

private RootElement GetParentAsRoot(Element element, string method) {
        if (element.Parent == null) {
            return new RootElement(element, NewControlsNotifier);
        }
        var root = element.Parent as RootElement;
        if (root == null) {
            throw new ArgumentException(method + " method is applicable only on top-most element");
        }
        return root;
    }

SonarQube 投诉要么删除 class 这个无用的对象实例化，要么使用它

SonarQube Complaint Either remove this useless object instantiation of class or use it

c#

object

sonarqube