回答編集履歴

2

指摘に対する修正

2015/04/04 10:17

投稿

yuragumo
yuragumo

スコア39

test CHANGED
@@ -14,8 +14,6 @@
14
14
 
15
15
  後、とりあえず気になった点を。
16
16
 
17
- ・ban配列の初期化ですが、[0]番目しか0クリアされないかと。(処理内でmemsetとかでクリアのほうがよさげ)
18
-
19
17
  ・while(1)をdo~whileにして、終了条件をkey=='\n'にしたほうがいいかと。(無限ループは多用しないほうがいい。)
20
18
 
21
19
  ・xを位置を求める際に使用している方法「-」なのに、右方向が「+」なのは逆ではないですか?

1

必ずではなかったですね、すいません。

2015/04/04 10:17

投稿

yuragumo
yuragumo

スコア39

test CHANGED
@@ -1,6 +1,6 @@
1
1
  このコードだと毎ループx,yに0クリアとmoveかけてるから、0,0にカーソルが戻るのは正常?ですね。
2
2
 
3
- となると、問題はbanに書き込みが行われないか反映されていないという事になりますが、banへの書き込みはcheck関数内で必ず通っています。
3
+ となると、問題はbanに書き込みが行われないか反映されていないという事になりますが、正常値を与えればbanへの書き込みはcheck関数内で通るようになっています。
4
4
 
5
5
  なので、反映されないというのが問題点になるのですが、そこに焦点を当てて追ってみると「banを使ってprintfしているのが、ループ突入前しかない」事がわかります。
6
6