Sorry, my fault.
I didn't mean that the constructor as such is wrong but the wrong pins being used in the constructor.
Nevertheless we should see how the wiring looks like as the OP doesn't seem to know the relationship of the constructor and the adequate Arduino pins.