関係ないけどさ、

入力チェックしたら？
SELECT 文一つにまとめたら？
IF文 一つにまとめたら？
"3"とか使うのやめたら？
変数定義してから使ったら？

少しはバグ減ると思うよ。