当我们重写一个方法时，我们应该使用方法签名中的所有参数吗？

Question

我发现一些覆盖方法没有使用方法签名中的所有参数。

例如：

@Override
protected void setSomething(Object a, Object b, Object c) {
    this.a = a
    this.b = b;
    // the parameter c is not used (ignored)
}

通常父类不应该关心子类将如何实现抽象方法。

但是在MHO中，要使用方法的参数，当子类实现不需要参数时，这是非常罕见的，当这种情况发生时，可能是接口的设计或抽象的设计有问题类。

一个功能的基础是：输入->输入过程->输出。

有时您需要计算这些输入，但如果您在函数的过程中不使用其中的一些输入，则不应将这些输入作为函数的输入。

你可以跳过这些输入的计算，调用一个使用所有输入的精确函数，所以是精确函数。

这种情况可以接受的唯一情况是，当我们不想要父类的行为时，我们可以这样写：

@Override
protected void setSomething(Object a, Object b, Object c) {
      //Nothing to do
}

或者

@Override
 protected void setSomething(Object a, Object b, Object c) {
  throw new UnsupportedOperationException(...);
}

声纳说：

未使用的参数具有误导性。无论传递给此类的值如何 参数是，行为将是相同的。

我的问题是： 当我们重写一个方法时，我们应该使用方法签名中的所有参数吗？

当我说“使用所有参数”时，我试图说方法签名中的所有参数实际上都在方法的主体（实现）中使用。

Answer 1

您需要使用所有参数吗？不会。您经常会看到以下示例：

@Override
public void doFoo(String thingy) {
  // no-op
}

或者

@Override
public void doFoo(String thingy) {
  throw new UnsupportedOperationException(...);
}

但两者都是某处设计存在问题的标志。例如，java.util.List 甚至 java.util.Iterable 都通过提供变异方法排除了不可变集合的可能性。不可变的实现必须抛出UnsupportedOperationException。

Answer 2

正如您所说：“我们应该”——但我们不必这样做。有时一个实施方法甚至会抛出一个RuntimeException，例如Java 集合框架的UnsupportedOperationException。

Answer 3

当我们重写一个方法时，我们应该使用方法签名中的所有参数吗？

当您覆盖一个方法时，被覆盖的方法必须定义与超方法相同的参数。

您没有义务使用实现中的所有参数 - 这取决于您希望通过此实现实现的目标，有时可能不需要所有参数。

但是，在方法实现中包含未使用的方法参数是设计不佳的标志。当定义一个方法（被abstract或实现）时，你应该尝试回答“我为什么需要这个参数？”和“这个参数会一直被使用吗？”。如果在实现中可能会出现某些参数不会被使用的情况，那么您可以定义一些重载方法。

以这个例子为例。让我们有这个方法

void someMethod(String first, String optionalParameter) { ... }

第二个参数是可选的（即可能需要也可能不需要）- 当不需要该参数时，您可以传递 null 或任何其他参数。在这种情况下，我会重载两个方法

void someMethod(String first) { ... }

void someMethod(String first, String second) { ... }

我也会确保所有的参数都用在相应的实现中。

Answer 4

如果你重写了一个方法，那么你必须使用所有的参数。

但是你也可以重载一个方法——写一个同名但参数不同的方法。所以如果你不使用所有的参数，你就是在重载，而不是覆盖。

class Accessor {

    public void doSomething(String attr) {}

}

class Child extends Accessor {

    // This is overriding
    @Override
    public void doSomething(String attr) {
       // ...
    }

    // This is overloading
    public void doSomething() {
       // ...
    }
}