Why are you using instance variables here?