teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

1

表現間違い修正

2019/05/08 02:54

投稿

moredeep
moredeep

スコア1507

answer CHANGED
@@ -1,5 +1,5 @@
1
1
  * maxの最初の`for(i=0;i<100;i++){`は98~100回同じことを繰り返しているだけなので不要です。
2
- * `for(j=0;j=i;j++){`この処理の意味が全てのmax_time、max_scoreを表示するというものであれば、jは* max_time/scoreの添え字なので、この配列の最大値未満という条件であるべきだと思います。そうでない場合、恐らくfor文である必要はありません。どちらにせよ、場所を移動するか実行するかどうかの条件を付けるべきです。
2
+ * `for(j=0;j=i;j++){`この処理の意味が全てのmax_time、max_scoreを表示するというものであれば、jは* max_time/scoreの添え字なので、この配列の要素数(100)未満という条件であるべきだと思います。そうでない場合、恐らくfor文である必要はありません。どちらにせよ、場所を移動するか実行するかどうかの条件を付けるべきです。
3
3
  * `max(head->next);`は最終的にnullになるためnullチェックが必要です。
4
4
  * `struct lv *lv;`使用していないため不要です。
5
5
  * headおよびhead->nextはmallocされていますが、freeされていません。