im very confused by what youre doing
theres a lot about what you're doing that i dont understand, but maybe this will help.
public class ClayMine extends Node{
final int clay = 7454;
final String ClayOre = "Clay";
private String status = "";
private Script sA;
public ClayMine(Script sA) {
super(sA);
this.sA = sA; //you'll only need this if the parent class has
//Script sA as private
}
@[member='Override']
public boolean validate() throws InterruptedException {
if(sA.inventory.isEmpty() && Data.ClayAreaN.contains(s.myPlayer()) {
status = "We're at north mining area";
return true;
}
return false;
}
@
Override
public boolean execute() throws InterruptedException {
Entity ore = sA.objects.closest(data.clay);
if(ore != null && Data.ClayAreaN.contains(sA.myPlayer())){
if(!sA.myPlayer().isAnimating()){
ore.interact("Mine");
status = "Mining ore";
}
return random(250,400);
}
break;
}
}
also, you're aware that you just keep defining a new string status, but never use it, right? this code needs to be seriously restructured and heavily changed, would recommend starting over tbh. also i would recommend making any data from 'Data' public and static instead of defining an object 'Data' without apparent purpose...