I think this will cause problems. Those variable names are being used as parameters in your methods, so you don't need to put them there. You be using the ones in your constructor instead
eg.
...