いろいろとだめだろ。
&がついてないし、入力後に確認すればいいものをループの上で判定するし
たとえば変数xに数字が入っていたら代入すればいいし。