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

• Reporter: hourier
• Owner: deskull
• Type: リファクタリング
• Status: Closed
• Component: (None)
• MileStone: 変愚蛮怒3.0.Xリリース
• Priority: 2
• Severity: 2
• Resolution: Fixed
• File: None

Details

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

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

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

Ticket History (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

• Resolution Update from None to Accepted

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

Comment

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

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

Comment

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

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

• Milestone Update from 変愚蛮怒3.0.0正式リリース (closed) to 変愚蛮怒3.0.Xリリース
• Severity Update from 3 to 2
• Priority Update from 3 to 2

Comment

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

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

• Status Update from Open to Closed
• Resolution Update from Accepted to Fixed

Comment

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