If you carefully read your code, and look at the changes I am suggesting, you will find they are logically equivalent, and make more sense from a hardware standpoint. Right now, you are reading a pin set to OUTPUT, not INPUT, which may work, but doesn't fully make sense, depending on how you are thinking about it...