you should avoid one use assigment