Game: API der PlayerComponent zwingt zu redundanten Code-Schnipseln

[cagix]

Mir ist aufgefallen, dass die API der PlayerComponent ... suboptimal ist. Man wird als Nutzer gezwungen, folgenden redundanten Code zu schreiben:

VelocityComponent vc = new VelocityComponent(5f, 5f);
PlayerComponent pc = new PlayerComponent();

pc.registerCallback(
        KeyboardConfig.MOVEMENT_UP.value(),
        entity -> vc.currentYVelocity(vc.yVelocity())
        );

Was hat es mit dem value() auf sich?! Warum muss ich das als Nutzer der API selbst schreiben? Das sollte die PlayerComponent bitte selbst hinkriegen.

Es soll deshalb eine Convenience-Methode hinzugefügt werden:

public final class PlayerComponent implements Component {

    ...

    public Optional<Consumer<Entity>> registerCallback(
            ConfigKey<Integer> key, final Consumer<Entity> callback) {
        return registerCallback(key.value(), callback);
    }

}

Perspektivisch muss die alte API raus. Dazu muss aber (a) eine Deprecation-Warnung erzeugt werden und (b) alle Verwendungen umgebaut werden und (c) die Verschränkung mit dem DSL-Kram geprüft werden.