java 未使用 形参 : Avoid unused constructor parameters such as 'upper'

Question

我们有一个名为 Sonar 的应用程序来分析代码以进行正确的编码实践。它指控的违规行为之一是:未使用形式参数:避免未使用的构造函数参数，例如“upper”。

 //Private inner class to set the input to a max length.
 public class TextLimiter extends PlainDocument
{
 private int limit;

 public TextLimiter(int limit)
{
    super();
    this.limit = limit;
}

TextLimiter(int limit, boolean upper)
{
    super();
    this.limit = limit;
}

public void insertString(int offset, String str, AttributeSet attr)
        throws BadLocationException
{
    if (str == null)
    {
        return;
    }

    if ((getLength() + str.length()) <= limit)
    {
        super.insertString(offset, str, attr);
    }
  }
}

我不太确定如何纠正这个问题。

Answer 1

如果你对上面的参数没有用处，我认为你不应该仅仅因为它在原始代码中就保留它。要停止有关重复代码的警告，只需删除 TextLimiter(int limit, boolean upper) 并保留 public TextLimiter(int limit)。