通用功能接口使用使用者的简单示例

Question

我正在尝试编写一个接受功能接口的通用方法。 参见示例。 任何帮助表示赞赏。

更新：使其编译并运行。 对改进或澄清的意见或建议感兴趣。

我将不会创建类型化抽象基类。 将一个附加参数添加到repeat方法中，指定在哪个机器人上执行命令，这与其他类似示例更加一致。

思考？

import java.util.function.Consumer;

/**
 * The intent is to create a generic function repeat() in the Robot base class
 * that will repeat actions for all subclasses of robots
 */
public class GenericFunctionalInterface
{
    public static void main(String[] args)
    {
        // Consumer<Robot> actionTurn = Robot::turnLeft;
        // Consumer<EnhancedRobot> anyAction = EnhancedRobot::anyAction;

        Robot r = new Robot();
        r.repeat(3, a->a.turnLeft() );

        EnhancedRobot e = new EnhancedRobot();
        e.repeat(3, a->a.anyAction() ); // cannot find symbol - method anyAction()
        // if ((EnhancedRobot)a).anyAction() is used this works
    }
}

class Robot 
{
    public void move() { System.out.println("MOVE"); }

    public void turnLeft() { System.out.println("TURNLEFT"); }

    public <T extends Robot> void repeat(int n, Consumer<T> cmd )
    {
        for(int i=0; i<n; i++) cmd.accept( (T) this); // UPDATED: casting this to T removed -> incompatible types: Robot cannot be converted to T
        // this will be whatever type repeat is called on
    } 

    // added this override - specifying what robot to send the command to
    public <T extends Robot> void repeat(int n, Consumer<T> cmd, T robot )
    {
        for(int i=0; i<n; i++) cmd.accept( robot );
    } 

}

class EnhancedRobot extends Robot
{
    public void turnRight() { System.out.println("TURNRIGHT"); }

    public void move() {  System.out.println("MOVEOVERIDE"); }

    public void anyAction() { System.out.println("ANYACTION"); }
}    

Answer 1

对于e.repeat()问题：假设您有以下代码：

class SuperRobot extends Robot {}

EnhancedRobot e = new EnhancedRobot();
e.repeat(3, new Consumer<SuperRobot>() {
  @Override
  public void accept(SuperRobot t) {
    // ...
  }
});

然后在您要调用的repeat方法内：

Consumer<T> cmd;
cmd.accept(this);

这将失败，因为this属于EnhancedRobot类型，但cmd希望获得SuperRobot 。 这就是为什么您不能在那叫它的原因。

存在a.anyAction()问题是因为a是类型为Robot的类型，它没有anyAction()方法。 根据声明（ T extends Robot ），关于其类型的唯一安全假设是Robot或对其进行扩展的东西。 因此，您不能从EnhancedRobot调用方法。

Answer 2

您将需要进行参数Robot与类型Consumer就会接受秒，在的方式Enum ，并添加返回适当类型的方法this （这需要Robot是抽象的，因为任何默认的实现必须正确返回null ）。

abstract class Robot<THIS extends Robot<THIS>> {
    ...
    public void repeat(int n, Consumer<THIS> cmd) {
        for (int i=0; i<n; i++) {
            cmd.accept(getThis());
        }
    }    
    protected abstract THIS getThis();
}

class EnhancedRobot extends Robot<EnhancedRobot> {
    @Override protected abstract EnhancedRobot getThis() {
        return this;
    }
    ...
}

一般而言，我认为您可以将repeat声明为

    public void repeat(int n, Consumer<? super THIS> cmd) {

Answer 3

在更新的代码中， repeat方法不使用您正在调用该方法的Robot实例。 因此该方法可以是static 。

// added this override - specifying what robot to send the command to
public static <T extends Robot> void repeat(int n, Consumer<T> cmd, T robot )
{
    for(int i=0; i<n; i++) cmd.accept( robot );
} 

像

EnhancedRobot e = new EnhancedRobot();
Robot.repeat(3, a->a.anyAction(), e );

但是对于repeat方法，甚至不需要robot参数。 您可以将其声明为

public static void repeat(int n, Runnable cmd)
{
    for(int i=0; i<n; i++) cmd.run();
}

像

EnhancedRobot e = new EnhancedRobot();
Robot.repeat(3, () -> e.anyAction());

这显示了设计的实际问题。 重复动作是否与Robot实例相关，与repeat方法的逻辑无关。 试图通过通用签名加强这种关系会产生问题，而没有任何好处。

实际上，已经有通用工具允许重复。 例如，你可以写

EnhancedRobot e = new EnhancedRobot();
Collections.nCopies(3, e).forEach(EnhancedRobot::anyAction);

要么

EnhancedRobot e = new EnhancedRobot();
IntStream.range(0, 3).forEach(i -> e.anyAction());