player-status-resist のforループ変数スコープを狭める

• 報告者: hourier
• 担当者: deskull
• チケットの種類: リファクタリング
• 状況: 完了
• コンポーネント: (未割り当て)
• マイルストーン: 変愚蛮怒3.0.Xリリース
• 優先度: 2
• 重要度: 2
• 解決法: 修正済み
• ファイル: なし

詳細

#40514 の関連で、forループの外側で「int i;」が宣言されています

しかしループの外側でiの値を使っていないので、forループ内で宣言するように修正願います

※ ファイル中に4箇所同様の宣言があります

チケットの履歴 (6 件中 3 件表示)

2020-11-10 19:08 更新者: hourier

• 新しいチケット "player-status-resist のforループ変数スコープを狭める" が作成されました

2020-11-11 08:18 更新者: deskull

• 解決法 が なし から 受領 に更新されました

2020-11-11 08:22 更新者: deskull

コメント

今更検索しましたが、japanese.cなど少数の例外を除いてほぼ大多数はそういう方針になっていますね。 コーディング規約、とまではいかんでもこれを推奨の慣例としておくつもりでやっておきます。

2020-11-25 08:15 更新者: deskull

コメント

話が遅れたがjapanese.cなども含めて自分の手を付けてない部分も今日中位に直しておく。

2020-12-08 08:24 更新者: deskull

• マイルストーン が 変愚蛮怒3.0.0正式リリース (完了済み) から 変愚蛮怒3.0.Xリリース に更新されました
• 重要度 が 3 から 2 に更新されました
• 優先度 が 3 から 2 に更新されました

コメント

今日中どころではないが見直し中。随所に問題の箇所は多数見られるが、中には複雑な分岐をforで絡めているところもあり迂闊に直すとエンバグになる可能性は高い。 検索の上方から順にいくつか手を加えてのを除き後バージョンに回し優先度も下げる。

2021-10-29 19:43 更新者: hourier

• 状況 が オープン から 完了 に更新されました
• 解決法 が 受領 から 修正済み に更新されました

コメント

最終コメントから10ヶ月が経過し、既に相当量のリファクタリングがなされている
他ファイルも含めて十分に無実化したと思われるのでクローズ