On UNO and similar Arduinos, pin 1 is used for Serial transmission. Your use of pin 1 for the first row of the keypad will conflict with this and probably explains why you can't get the 1, 2, 3 keys to work.
Whether or not that is the problem, you should read How to post code properly and then use the </> icon to create
[code]...[/code]
tags around your code so that it is easier to read.
Pete