Interface oder Vererbung?

[Mariexshhx]

Hey, meine Aufgabe ist es die Typenunterscheidung aus der Klasse zu entfernen. Ich habe für die einzelnen Typen (also Spielfiguren) schon Klassen erstellt und die Methode act für den jeweiligen Spieler entsprechenden überschriebenben. Nun ist die Methode act aber "leer". Was mache ich mit dieser Methode. Gleiches gilt für die Methoden isLivingBug etc. die wurden auch alle in den verschiedenen Klassen für die Spielertypen überschrieben und hätten jetzt quasi keine Implemetierung. Ich habe jetzt erstmal immer false zurückgeben lassen. Das klingt halt alles nach Interface, aber die Methoden, wie sich die Spieler bewegen sind ja für alle Arten gleich, deswegen passt kein Interface. Achso, natürlich habe ich meine einzelnen Spielerklassen alle von Unit erben lassen. Jemand eine Idee, was ich jetzt mit den Methoden machen kann? Abstrakt können sie ja auch nicht sein....

 

[temi]

Irgendwie kommt mir die Frage bekannt vor, ohne jetzt den Code angeschaut zu haben. Ich lade i.d.R. nichts herunter, insofern wäre es besser die Code-Tags zu verwenden.

 

[Mariexshhx]

ublic class Unit {

  private final GamePanel game;
  private int row;
  private int col;
  private UnitType type;

  protected int hitpoints;
  protected int strength;

  public Unit(GamePanel game, int row, int col,UnitType type) {
    this.game = game;
    this.type=type;
    this.row = row;
    this.col = col;



  }

  public void placeUnit(int row, int col) {
    if (game.occupied(row, col)) return;
    game.removeUnit(this.row, this.col);
    game.placeUnit( row, col,type);
    this.row = row;
    this.col = col;
  }

  public int getRow() {
    return row;
  }

  public int getCol() {
    return col;
  }

  public boolean isStationary() {
    return false;
  }

  public boolean isLivingPlant() {
    return false;
  }

  public boolean isLivingBug() {
    return false;
  }

  public boolean isLivingEnemy() {
    return false;
  }

  public void act(Set<Unit> units) {


  }

  protected void moveRandomly() {
    placeUnit(randomWalk(row), randomWalk(col));
  }

  protected void moveTowards(Unit other) {
    int nextrow = row;
    int nextcol = col;
    if (row < other.row) nextrow++;
    if (row > other.row) nextrow--;
    if (col < other.col) nextcol++;
    if (col > other.col) nextcol--;
    placeUnit(nextrow, nextcol);
  }

  private int randomWalk(int r) {
    if (Math.random() < 0.25) return r - 1;
    if (Math.random() > 0.75) return r + 1;
    return r;
  }


  protected void attack(Unit other) {
    System.out
        .println(this.type + " hits " + other.type + " Remaining Hitpoints: " + other.hitpoints);
    other.hit(strength);
  }

  private void hit(int strength) {
    hitpoints -= strength;
  }

  protected boolean isNeighbor(Unit other, int dist) {
    return Math.abs(row - other.row) <= dist &&
        Math.abs(col - other.col) <= dist;
  }

  protected int distance(Unit me, Unit a, Unit b) {
    var d1 = distance(me, a);
    var d2 = distance(me, b);
    return Double.compare(d1, d2);
  }

  private double distance(Unit me, Unit b) {
    return Math.sqrt(square(me.row - b.row) + square(me.col - b.col));
  }

  private double square(int v) {
    return v * v;
  }

  public boolean isDead() {
    return hitpoints <= 0;
  }

  public boolean isAlive() {
    return hitpoints > 0;
  }
}

 

[mihe7]

@temi

package de.propra.defense;

import static de.propra.defense.UnitType.BUG1;
import static de.propra.defense.UnitType.BUG2;
import static de.propra.defense.UnitType.CROW;
import static de.propra.defense.UnitType.FROG;
import static de.propra.defense.UnitType.PLANT;
import static de.propra.defense.UnitType.SCARECROW;


import de.propra.defense.ui.GamePanel;
import java.util.Optional;
import java.util.Set;

public class Unit {

  private final GamePanel game;
  private int row;
  private int col;
  private UnitType type;

  protected int hitpoints;
  protected int strength;

  public Unit(GamePanel game, int row, int col,UnitType type) {
    this.game = game;
    this.type=type;
    this.row = row;
    this.col = col;



  }

  public void placeUnit(int row, int col) {
    if (game.occupied(row, col)) return;
    game.removeUnit(this.row, this.col);
    game.placeUnit( row, col,type);
    this.row = row;
    this.col = col;
  }

  public int getRow() {
    return row;
  }

  public int getCol() {
    return col;
  }

  public boolean isStationary() {
    return false;
  }

  public boolean isLivingPlant() {
    return false;
  }

  public boolean isLivingBug() {
    return false;
  }

  public boolean isLivingEnemy() {
    return false;
  }

  public void act(Set<Unit> units) {


  }

  protected void moveRandomly() {
    placeUnit(randomWalk(row), randomWalk(col));
  }

  protected void moveTowards(Unit other) {
    int nextrow = row;
    int nextcol = col;
    if (row < other.row) nextrow++;
    if (row > other.row) nextrow--;
    if (col < other.col) nextcol++;
    if (col > other.col) nextcol--;
    placeUnit(nextrow, nextcol);
  }

  private int randomWalk(int r) {
    if (Math.random() < 0.25) return r - 1;
    if (Math.random() > 0.75) return r + 1;
    return r;
  }


  protected void attack(Unit other) {
    System.out
        .println(this.type + " hits " + other.type + " Remaining Hitpoints: " + other.hitpoints);
    other.hit(strength);
  }

  private void hit(int strength) {
    hitpoints -= strength;
  }

  protected boolean isNeighbor(Unit other, int dist) {
    return Math.abs(row - other.row) <= dist &&
        Math.abs(col - other.col) <= dist;
  }

  protected int distance(Unit me, Unit a, Unit b) {
    var d1 = distance(me, a);
    var d2 = distance(me, b);
    return Double.compare(d1, d2);
  }

  private double distance(Unit me, Unit b) {
    return Math.sqrt(square(me.row - b.row) + square(me.col - b.col));
  }

  private double square(int v) {
    return v * v;
  }

  public boolean isDead() {
    return hitpoints <= 0;
  }

  public boolean isAlive() {
    return hitpoints > 0;
  }
}

Das klingt halt alles nach Interface, aber die Methoden, wie sich die Spieler bewegen sind ja für alle Arten gleich, deswegen passt kein Interface. Achso, natürlich habe ich meine einzelnen Spielerklassen alle von Unit erben lassen. Jemand eine Idee, was ich jetzt mit den Methoden machen kann? Abstrakt können sie ja auch nicht sein....

a) natürlich können die Methoden abstrakt sein
b) natürlich kannst Du ein Interface verwenden

 

[Mariexshhx]

@temi

package de.propra.defense;

import static de.propra.defense.UnitType.BUG1;
import static de.propra.defense.UnitType.BUG2;
import static de.propra.defense.UnitType.CROW;
import static de.propra.defense.UnitType.FROG;
import static de.propra.defense.UnitType.PLANT;
import static de.propra.defense.UnitType.SCARECROW;


import de.propra.defense.ui.GamePanel;
import java.util.Optional;
import java.util.Set;

public class Unit {

  private final GamePanel game;
  private int row;
  private int col;
  private UnitType type;

  protected int hitpoints;
  protected int strength;

  public Unit(GamePanel game, int row, int col,UnitType type) {
    this.game = game;
    this.type=type;
    this.row = row;
    this.col = col;



  }

  public void placeUnit(int row, int col) {
    if (game.occupied(row, col)) return;
    game.removeUnit(this.row, this.col);
    game.placeUnit( row, col,type);
    this.row = row;
    this.col = col;
  }

  public int getRow() {
    return row;
  }

  public int getCol() {
    return col;
  }

  public boolean isStationary() {
    return false;
  }

  public boolean isLivingPlant() {
    return false;
  }

  public boolean isLivingBug() {
    return false;
  }

  public boolean isLivingEnemy() {
    return false;
  }

  public void act(Set<Unit> units) {


  }

  protected void moveRandomly() {
    placeUnit(randomWalk(row), randomWalk(col));
  }

  protected void moveTowards(Unit other) {
    int nextrow = row;
    int nextcol = col;
    if (row < other.row) nextrow++;
    if (row > other.row) nextrow--;
    if (col < other.col) nextcol++;
    if (col > other.col) nextcol--;
    placeUnit(nextrow, nextcol);
  }

  private int randomWalk(int r) {
    if (Math.random() < 0.25) return r - 1;
    if (Math.random() > 0.75) return r + 1;
    return r;
  }


  protected void attack(Unit other) {
    System.out
        .println(this.type + " hits " + other.type + " Remaining Hitpoints: " + other.hitpoints);
    other.hit(strength);
  }

  private void hit(int strength) {
    hitpoints -= strength;
  }

  protected boolean isNeighbor(Unit other, int dist) {
    return Math.abs(row - other.row) <= dist &&
        Math.abs(col - other.col) <= dist;
  }

  protected int distance(Unit me, Unit a, Unit b) {
    var d1 = distance(me, a);
    var d2 = distance(me, b);
    return Double.compare(d1, d2);
  }

  private double distance(Unit me, Unit b) {
    return Math.sqrt(square(me.row - b.row) + square(me.col - b.col));
  }

  private double square(int v) {
    return v * v;
  }

  public boolean isDead() {
    return hitpoints <= 0;
  }

  public boolean isAlive() {
    return hitpoints > 0;
  }
}

a) natürlich können die Methoden abstrakt sein
b) natürlich kannst Du ein Interface verwenden

muss die Klasse nicht abstrakt sein damit die Methode abstrakt sein kann?
und ein Interface gibt doch nur die zu Implementierenden Methoden vor, aber enthält doch selber keine Implemetierung ?

 

[mihe7]

Ja, beides richtig - ist aber beides kein Problem.

Zum Beispiel: Interface Unit, abstrakte Klasse AbstractUnit, davon abgeleitete konkrete Klassen für die Spielertypen.

 

[Mariexshhx]

ah achso hatte irgendwie im Kopf, dass eine abstrakte Klasse nur abstrakte Methoden haben darf

Ja, beides richtig - ist aber beides kein Problem.

Zum Beispiel: Interface Unit, abstrakte Klasse AbstractUnit, davon abgeleitete konkrete Klassen für die Spielertypen.

 

[KonradN]

Ja, beides richtig - ist aber beides kein Problem.

Nö das würde ich anders sehen: das Zweite ist nicht richtig. Ein Interface gibt zwar die zu implementierenden Methoden vor aber kann zusätzlich auch Implementierungen enthalten - die default Methoden.

Beispiele finden sich viele, z.B. bei sehr vielen functional Interfaces. Einfach mal Predicate angeschaut (War irgendwie das Erste, das mit eingefallen ist):

Predicate (Java SE 17 & JDK 17)

declaration: module: java.base, package: java.util.function, interface: Predicate

docs.oracle.com

Und da findet man gleich 3 default Methoden. Und auch noch 2 static Methoden - das geht natürlich auch noch

 

[mihe7]

ah achso hatte irgendwie im Kopf, dass eine abstrakte Klasse nur abstrakte Methoden haben darf

Nein, Du musst nur Klasse und die betreffenden Methoden abstrakt deklarieren.

Ein Interface gibt zwar die zu implementierenden Methoden vor aber kann zusätzlich auch Implementierungen enthalten - die default Methoden.

Ja, wenn man es genau nimmt

 

[Mariexshhx]

das Problem ist, wenn ich die Methode act abstrakt mache muss auch die Klasse Plant diese implemetieren. Eine Pflanze hat aber keine implemetierung für act.
act betrifft nur Frog, Bug1 und Bug2
also es ist eine void Methode theoretisch kann ich die Implemtierung in den Klassen wo act nix tut weglassen. Aber ich denke nicht, dass das die Lösung ist...

 

[KonradN]

das Problem ist, wenn ich die Methode act abstrakt mache muss auch die Klasse Plant diese implemetieren. Eine Pflanze hat aber keine implemetierung für act

Dann gehört diese Methode nicht da hin. Wenn nicht alle Unit Instanzen act haben, dann ist das einfach kein Bestandteil von Act.

Das ist ja das, was man immer hat. Nehmen wir einfach mal die Biologie:
Du hast von mir aus eine Katze und Hund. Beide säugen Nachkommen. Beides sind Lebewesen. Also könnte man meinen, dass man säugen in Lebewesen packt. Aber dann kriegen wir ein neues Lebewesen - das säugt Nachkommen nicht: Die Biene.

Also merken wir: säugen gehört nicht in Lebewesen. Man kann aber jetzt eine neue Klasse erzeugen: Säugetier. Säugetier erbt von Lebewesen, Hund und Katze erben von Säugetier. Und das säugen wandert von Lebewesen zu Säugetier.

Das nur um es einmal auf die Schnelle zu skizzieren.

 

[Mariexshhx]

Dann gehört diese Methode nicht da hin. Wenn nicht alle Unit Instanzen act haben, dann ist das einfach kein Bestandteil von Act.

Das ist ja das, was man immer hat. Nehmen wir einfach mal die Biologie:
Du hast von mir aus eine Katze und Hund. Beide säugen Nachkommen. Beides sind Lebewesen. Also könnte man meinen, dass man säugen in Lebewesen packt. Aber dann kriegen wir ein neues Lebewesen - das säugt Nachkommen nicht: Die Biene.

Also merken wir: säugen gehört nicht in Lebewesen. Man kann aber jetzt eine neue Klasse erzeugen: Säugetier. Säugetier erbt von Lebewesen, Hund und Katze erben von Säugetier. Und das säugen wandert von Lebewesen zu Säugetier.

Das nur um es einmal auf die Schnelle zu skizzieren.

ah okay also würdest du die Methode gar nicht überschreiben sondern komplett aus Unit entfernen und nur in den Klassen Frog, Bug1,Bug2 implementieren ?

 

[KonradN]

Nein, nicht ganz.

Zwei Möglichkeiten:
a) Rein mit Vererbung, dann hätte man sowas wie eine ActableUnit, welche von Unit erbt wovon dann Frog, Bug1, ... erben können.
b) Mit Interfaces. Dann gibt es einfach ein Interface Actable welches Frog, Bug1, .... implementieren können.