扩展抽象类的类如何使用自己的变量？

Question

我有两个类，它们使用了几个相等的方法，所以我想我可以使用一个抽象类而不是一个接口来允许抽象类有方法体，并减少其他类中代码的重复。但是在运行我的主函数时，我得到一个错误，说x是“null”，我认为这是因为它使用了抽象类中的空变量。但是对于抽象类中的方法，我需要声明方法使用的变量，那么如何让它使用子类中的变量呢？具体来说，错误出现在底部的agamefughts setPosition（）中。x=x

这是我的代码：

abstract class AGameFigures {
public Rectangle hitbox;
int action;
ICollision collision;
float tileHeight, tileWidth;
float spriteHeight, spriteWidth;
float velocityY;

public AGameFigures(){
    this.hitbox = hitbox;
    this.action = action;
    this.collision = collision;
    this.tileHeight = tileHeight;
    this.tileWidth =tileWidth;
    this.spriteHeight = spriteHeight;
    this.spriteWidth = spriteWidth;
    this.velocityY = 0;
}

public void update(float delta) {
    float oldY = hitbox.y;
    hitbox.y += velocityY;
    velocityY -= (25 * delta);

    boolean collisionDown = false;
    boolean collisionUpwards = false;

    if(velocityY < 0) {
        collisionDown = collision.collidesDownwards(hitbox.x, hitbox.y);
    } else if(velocityY > 0) {
        collisionUpwards = collision.collidesUpwards(hitbox.x, hitbox.y);
    }
    if(collisionDown) {
        hitbox.y = (int) (oldY / tileHeight) * tileHeight;
        velocityY = 0;
    }
    if(collisionUpwards) {
        hitbox.y = (int) ((hitbox.y+spriteHeight/2) / tileHeight) * tileHeight;
        if(velocityY > 0)
            velocityY = 0;
        velocityY -= (25 * delta);
    }
}
public void setPosition(float x, float y) {
    hitbox.x = x;
    hitbox.y = y;
}

abstract void moveLeft(float delta);

abstract void moveRight(float delta);

public void setVelocityY(float newVelY) {
    velocityY = newVelY;
}
public void jump() {
    if (velocityY == 0)
        velocityY = 7;

}
public int hits(Rectangle r) {
    if(hitbox.overlaps(r))
        return 1;
    return -1;
}

}

public class GamePlayer extends AGameFigures{

public Rectangle hitbox;
int action;
float tileHeight, tileWidth;
float spriteHeight, spriteWidth;
ICollision collision;
float velocityY;


public GamePlayer(float spriteHeight, float spriteWidth, ICollision collision) {
    hitbox = new Rectangle(0.0f, 0.0f, 128.0f, 128.0f);
    velocityY = 0;
    hitbox.x = 0; hitbox.y = 0;
    tileWidth = collision.getTileWidth(); tileHeight = collision.getTileHeight();
    this.spriteHeight = spriteHeight; this.spriteWidth = spriteWidth;
    this.collision = collision;
}

public void action(int type, float x, float y) {
    if(type == 1) {
        velocityY = 0;
        setPosition(hitbox.x, y);
    }
}

@Override
public void moveLeft(float delta) {
    hitbox.x -= (200 * delta);  // delta == "change of time" "for fluid change om motion"
    if(collision.collidesLeftwards(hitbox.x, hitbox.y)) {
        hitbox.x = (int) (hitbox.x / tileWidth) * tileWidth + spriteWidth;
    }
}
@Override
public void moveRight(float delta) {
    hitbox.x += (200 * delta);  // (value) * delta, endre value for å endre movementspeed
    if(collision.collidesRightwards(hitbox.x, hitbox.y)) {
        hitbox.x = (int) (hitbox.x / tileWidth) * tileWidth;
    }
}

}




Answer 1

我认为你的问题在于抽象类的构造函数。变量将始终为空。

您需要在构造函数中提供参数：

public AGameFigures(Hitbox hitbox, int action, ICollision collision, float tileHeight, float tileWidth, float spriteHeight, float spriteWith, float velocityY){
    this.hitbox = hitbox;
    this.action = action;
    this.collision = collision;
    this.tileHeight = tileHeight;
    this.tileWidth =tileWidth;
    this.spriteHeight = spriteHeight;
    this.spriteWidth = spriteWidth;
    this.velocityY = 0;
}

而且，您不需要重复继承的变量。

Answer 2

为什么你要把sames归入超类和子类

Answer 3

从AGameFIG构造函数：

public AGameFigures(){
    this.hitbox = hitbox;
    ... 
}

没有传递给构造函数的参数，因此，hitbox引用类实例字段，该字段为空。构造函数中的这行代码：

this.hitbox = hitbox;

实际上：

this.hitbox = null;

所以后来遇到了一个空端口。需要更新构造函数以传递参数：

public AGameFigures(Rectangle hitbox, ...){
    this.hitbox = hitbox;
    ... 
}

子类需要提供构造函数参数值。

GamePlayer定义了自己的hitbox，这可能会引起一些混乱hitbox应该从GamePlayer中删除，只需依赖基类即可。

检查编译器警告，不要（通常）忽略它们！