in the chop method you should find the tree and check for nulls and stuff
RS2Object tree = this.closestObjectForName(area,"Tree"); //dont really know the method w.e. it is you get what i am saying
if(tree != null)
if(tree.isVisible() && tree.exsists())
tree.interact("Chop Down");
Something along those lines
also you should make a getState() method in which returns the current state instead of having one global state that you are going off of. and do
switch(getState())
{
case CHOP:
}