use local variables not fields