public class Fighter extends Actor
{
    // todo: would it be better to put these in each character instead
    int weight;
    int xVelocity;
    int yVelocity;
    int moveSpeed;
    int hp = 100;
    int jumpHeight;
    int xPos = 150;
    int yPos = 150;
    
    /**
     * Act - do whatever the Fighter wants to do. This method is called whenever
     * the 'Act' or 'Run' button gets pressed in the environment.
     */
    public void act()
    {
        
    }
    public boolean isGrounded(){
        // if the players position is below the arenas limit, the player is
        // considered grounded
        if (yPos >= Arena.ground){
            yPos = Arena.ground;
            return true;
        }
        return false;
    }
    public void fall(){
        // moves down if above the ground, if below, doesnt move at all
        if (isGrounded() == false){
            yVelocity = yVelocity + Arena.gravity + weight;
            yPos = yPos + yVelocity;
        } else {
            yVelocity = 0;
        }
    }
    public void jump(String up){
        if (Greenfoot.isKeyDown(up)){
            //can only jump when grounded
            if (isGrounded()){
                yVelocity = jumpHeight;
                //yPos = yPos - 1;
            }
        }
    }
    public boolean moveLeft(String left){
        if (Greenfoot.isKeyDown(left)){
            //can only change speed when on the ground,
            //same for moveright
            if (isGrounded()){
                xVelocity = -moveSpeed;
                return true;
            }
        }
        return false;
    }
    public boolean moveRight(String right){
        if (Greenfoot.isKeyDown(right)){
            if (isGrounded()){
                xVelocity = moveSpeed;
                return true;
            }
        }
        return false;
    }
}

public class Jotaro extends Fighter
{
    GifImage gifImage = new GifImage("jotarokujopixel.gif");
    /**
     * Act - do whatever the Jotaro wants to do. This method is called whenever
     * the 'Act' or 'Run' button gets pressed in the environment.
     */
    public void act()
    {
        weight = 1/2;
        jumpHeight = -10;
        moveSpeed = 3;
        
        setLocation(xPos, yPos);
        xPos = getX();
        yPos = getY();
        xPos = xPos + xVelocity;
        yPos = yPos + yVelocity;
        isGrounded();
        fall();
        jump("W");
        //todo this code is HORRID please put this into fighter somehow later
        if (moveLeft("A") && moveRight("D")){
            // this is supposed to make the player not move if both buttons
            // are held but it doesnt work for some reason lol
            xVelocity = 0;
            setImage("jotarokujopixel21.png");
        } else if (moveLeft("A")){
            moveLeft("A");
            setImage(gifImage.getCurrentImage());
        } else if (moveRight("D")){
            moveRight("D");
            setImage(gifImage.getCurrentImage());
        }
        if (moveLeft("A") == false && moveRight("D") == false){
            xVelocity = 0;
            setImage("jotarokujopixel21.png");
        }
    }
}