抽象类和方法出错

Question

I have an abstract class Player and i want to create two kind of players. Fast and Technical Players.So i created this abtsract class:

public abstract class Player
{
    private int speed;
    private int technical;
    private Cell playerCell;

    public abstract void computeAbility();
    public abstract void clonePlayer();

    public Player(){
        Random rand = new Random();
        speed = rand.nextInt(50);
        technical = rand.nextInt(50);
    }

    //i also have accesor and mutator methods
}




  public class FastPlayer extends Player
{
    private int overallAtrributes;

    public FastPlayer(){
        super();
        setSpeed(2*getSpeed());
    }

    public void computeOverallAtrributes(){
                overallAtrributes = getSpeed() + getTechnical();
    }

    public void clonePlayer(){
        FastPlayer cloneFastPlayer = new FastPlayer();
        cloneFastPlayer.setTechnical(getTechnical());
        cloneFastPlayer.setSpeed(getSpeed());
    }
}



public class TechnicalPlayer extends Player
{
    private int overallAtrributes;

    public TechnicalPlayer(){
        setTechnical(2*getTechnical());
    }

    public void computeOverallAtrributes(){
        overallAtrributes = getSpeed() + getTechnical();
    }

    public void clonePlayer(){
        TechnicalPlayer cloneTechnicalPlayer = new TechnicalPlayer();
            cloneTechnicalPlayer.setTechnical(getTechnical());
            cloneTechnicalPlayer.setSpeed(getSpeed());
        }
}

Now i want create a reproducePlayer method in Player class to produce a Player of the same type as current Player and i want to put the new person in the same Cell. So i created this method in Player class:

public void reproduce(){
        this.clonePlayer();
        //how can i put the new Player in the same Cell with current Player?
    }

I also want to create a method movePlayer which moves the Player in a Random neighbor Cell of my array. i dont have create the class Cell yet.

Answer 1

You need to make the clone in the common class.. And than override it or call it.

Something like that:

Under player:

public abstract Player clonePlayer();
protected Player clonePlayer(Player newPlayer){
    newPlayer.setTechnical(getTechnical());
    newPlayer.setSpeed(getSpeed());
    return newPlayer;
}

Under the player extended classes:

Class TechnicalPlayer extends Player
public Player clonePlayer() {
   return super.clonePlayer(new TechnicalPlayer ());
}

This should be something like that.. Hope that helps }

Answer 2

Because all your implementing classes have no-args constructors, you could define just one method on the abstract class that uses reflection to create another instance of the same class as the current one:

public Player reproducePlayer() {
    Player p;
    try {
        p = getClass().newInstance();
    catch (InstantiationException | IllegalAccessException ignore) {}
    p.speed = speed;
    p.technical = technical;
    p.playerCell = playerCell;
    return p;
}