You have made the pins into input pins, but they need to be output pins. What is a little confusing about this is that you can digitalWrite() to an input pin, and you can digitalRead() from an output pin. So you're not getting an error message even though your code is not actually doing what you want it to.
If you digitalWrite() to an input pin, it either enables or disables the internal pullup resistor, depending on whether you wrote HIGH or LOW. If you digitalRead() from an output pin, it returns the pin's current value (HIGH or LOW).