Generally it is more helpful to describe the difference between what you expect code to do and what it actually does in terms of behavior or output than it is to say: "Something is wrong with this. What is it?"
One thing jumps out at me: = is assignment. == is comparison. See line 24.