This is a very strange use of the not operator with an integer value. Very irregular. Is it intentional? If so, explain, if not explain. Edit - a pretty strange use of constrain() as well.
The reverse question marks are usually just caused by the wrong baud rate selection. Let's focus on that now, so we can see what the program is printing...