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

• 报告人: hourier
• 属主: deskull
• 类型: リファクタリング
• 状态: 关闭
• 组件: (无)
• 里程碑: 変愚蛮怒3.0.Xリリース
• 优先: 2
• 严重性: 2
• 处理结果: Fixed
• 文件: 无

Details

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

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

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

任务单历史 (3/6 Histories)

2020-11-10 19:08 Updated by: hourier

• New Ticket "player-status-resist のforループ変数スコープを狭める" created

2020-11-11 08:18 Updated by: deskull

• 处理结果 Update from 无 to Accepted

2020-11-11 08:22 Updated by: deskull

评论

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

2020-11-25 08:15 Updated by: deskull

评论

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

2020-12-08 08:24 Updated by: deskull

• 里程碑 Update from 変愚蛮怒3.0.0正式リリース (关闭) to 変愚蛮怒3.0.Xリリース
• 严重性 Update from 3 to 2
• 优先 Update from 3 to 2

评论

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

2021-10-29 19:43 Updated by: hourier

• 状态 Update from 开启 to 关闭
• 处理结果 Update from Accepted to Fixed

评论

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