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

• Auswertung: hourier
• Verantwortlicher: deskull
• Typ: リファクタリング
• Status: Geschlossen
• Komponente: (Keine)
• Meilenstein: 変愚蛮怒3.0.Xリリース
• Priorität: 2
• Schweregrad: 2
• Lösung: Gefixt
• Datei: Keine

Details

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

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

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

Ticket-Verlauf (3/6 Historien)

2020-11-10 19:08 Aktualisiert von: hourier

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

2020-11-11 08:18 Aktualisiert von: deskull

• Lösung Update from Keine to Accepted

2020-11-11 08:22 Aktualisiert von: deskull

Kommentar

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

2020-11-25 08:15 Aktualisiert von: deskull

Kommentar

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

2020-12-08 08:24 Aktualisiert von: deskull

• Meilenstein Update from 変愚蛮怒3.0.0正式リリース (closed) to 変愚蛮怒3.0.Xリリース
• Schweregrad Update from 3 to 2
• Priorität Update from 3 to 2

Kommentar

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

2021-10-29 19:43 Aktualisiert von: hourier

• Status Update from Offen to Geschlossen
• Lösung Update from Accepted to Gefixt

Kommentar

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