[choboko]

I'm trying to make my boolean checker check if my "character" is dead or not.
Designing an RPG character class where they have certain attributes.

import java.util.Random;
public class Character
{
	private String name;
	private int strength;
	private int dexterity;
	private int intelligence;
	private int hp;
	private Random randomiser;
	private int stat = 16;
	
	public Character(String name)
	{
	this.name = name;
	randomiser = new Random();
//random stats between 3 and 18
	strength = 3 + randomiser.nextInt(stat);
	dexterity = 3 + randomiser.nextInt(stat);
	intelligence = 3 + randomiser.nextInt(stat);
	
	hp = strength;
	
	}

	public String getName()
	{
	return name;
	}
...
//Does damage to the character's hp
	public void gotHit(int dmg)
	{
	 hp = hp - dmg;
	}
//checks if hp is less than or equal to 0, returns true if dead; false if not dead.
//resets hp back to 0
	public boolean isDead()
	{
	if(hp < 0)
	{
	System.out.print("Character is dead");
	hp = 0;
	return true;
	}
	else
	{
	return false;
	}
	}
	public int getHp()
	{
	return hp;
	}
	
	public String toString()
	{
		String info = (name + ", Strength: " + strength + ", Dexterity: " + dexterity 
		+ ", Intelligence: " + intelligence + ", Hit Points: " + hp );
		return info;
	}

driver
...
		Character Chicken = new Character("Chicken");
		Chicken.gotHit(100);
		System.out.println(Chicken.toString());
...

The problem is that when I do "damage" it subtracts it from hp, but it's not checked by the isDead() method when it's printed in the toString() method. So I get a negative number instead of 0 being printed.

Hope you can help.

[g00se]

I'd do

public void gotHit(int dmg)
{
	hp = Math.max(hp - dmg, 0);
}

public boolean isDead()
{
	return hp == 0;
}

[choboko]

g00se, on 29 May, 2009 - 06:29 AM, said:

I'd do

public void gotHit(int dmg)
{
	hp = Math.max(hp - dmg, 0);
}

public boolean isDead()
{
	return hp == 0;
}

I thought that in boolean methods I needed to return either true or false?
Edit:
Thanks it works, but I want to print the statement as well. Or do I need to put that somewhere else, because it's not printing in that method.

This post has been edited by choboko: 29 May 2009 - 07:18 PM

[333OnlyHalfEvil]

I'm assuming that when you say you want to print the statement you mean a message saying "You Died" or something similar. In your gotHit() method add the print line.

public void gotHit(int dmg) {
	 hp = Math.max(hp-dmg, 0);
	 if (isDead()) {
		  System.out.println("You Died");
	 }
}

[pbl]

choboko, on 29 May, 2009 - 06:13 PM, said:

I thought that in boolean methods I needed to return either true or false?

return hp == 0;

That will return true if hp == 0 and false otherwise

[choboko]

Hey I'm back again with another problem @.@
I'm having trouble with overriding methods.

public class Barbarian extends Character
{
	protected String name;
	protected boolean rage = false;
	public Barbarian(String name)
	{
	super(name);
	}
	
	public String getName()
	{
	return name;
	}
	
	public void setRage()
	{
			if(rage == false)
		{
			rage = true;
			System.out.println("Rage activated.");
			strength = strength + 6;
		}	
	}
	
	public int getStrength()
	{
		return strength;
	}
	
	public String toString()
		{
			String info = (name + ", Strength: " + strength + ", Dexterity: " + dexterity 
			+ ", Intelligence: " + intelligence + ", Hit Points: " + hp +", Rage: " + rage);
			return info;
		}

Which is extended from the character class above. Only I changed from the Character class was
private -> protected.

Anyway my problem is that when I print the toString method it prints out name = null instead of the name of my character.

Any ideas?

[gl3thr0]

private variables are not inherited by a subclass, thus all your variables
private String name;
private int strength;
private int dexterity;
private int intelligence;
private int hp;
private Random randomiser;
private int stat = 16;
are not accessible by the barbarian class. To fix this and keep your program secure use protected.
so for name you would
remove the line
protected String name;
from the barbarian class
and change private String name in the character class to
protected String name;

what happens in your program is you pass the string name to the constructor and then call super(name). This sets the name in the super class, not in the subclass. thus none of the methods you create in the new class will have access to this variable, only the methods in the superclass. Towards the end of you program you over rid the method toString(). This created a new method in the Barbarian class that did not have access to the private variable name. Only the protected one that you had created that has not been set yet. Thus another way to fix this would be to remove the new toString() method. However, it would make more sense to allow you subclasses to simply inherit variables like name and dexterity.

This post has been edited by gl3thr0: 01 June 2009 - 02:52 AM

[choboko]

Cool thanks it's working now. I keep making small errors