incorrectly assigning variables