なぜ動かないのか自分でも確認する癖をつけるといいですよ。 「PRINT X,Y」 と変数の内容を確認するとわかりやすくなると思います