while you are at it .. could you move the assignment out of the if? getting hard to read