You are shadowing your instance variables with duplicate method local variable definition:
int finalbalance = balance-amount;
just do the assignment instead of re-decalaration
finalbalance = balance-amount;
Also using keyword this
will make sure that you are setting the instance variable and not the local variable in case you have two such variables with the same name. So this is much safer:
this.finalbalance = balance-amount;