Ticket #37353

機能拡張のためのリファクタリング

Eröffnet am: 2017-07-10 23:38 Letztes Update: 2020-12-28 08:19

Auswertung:
Verantwortlicher:
Status:
Geschlossen
Komponente:
(Keine)
Priorität:
5 - Mittel
Schweregrad:
5 - Mittel
Lösung:
Gefixt
Datei:
Keine

Details

現在の変愚蛮怒実装の各要素、特にモンスター種族フラグなどこれ以上の機能追加にためには数が足りない、分かりづらいなどの問題がある部分を本チケットで処理する。 一部セーブデータにも関わる細心な部分も多いので、変更において留意すると共に、開発版であることを念頭に置いてもらった上でのテストを希望する。

以上のつもりでやっていたが、実際は結構雑多なリファクタリングのコミットをこれに充ててしまったのでなし崩しに今後もそのつもりで。

Ticket-Verlauf (3/11 Historien)

2017-07-10 23:38 Aktualisiert von: deskull
  • New Ticket "機能拡張のためのリファクタリング" created
2017-07-11 00:09 Aktualisiert von: deskull
Kommentar

モンスターの特殊攻撃(発動型技能と改名したい)のデータ枠をu32b×2の64から×4の128に拡張。

2017-07-11 23:26 Aktualisiert von: deskull
Kommentar

現在乗馬、マーシャルアーツ、二刀流の3種が定義されている技能枠について、実際のデータ枠が10あることを確認したが、この最大数がマジックナンバーのままになっていたので、分かる限りでGINOU_MAXに差し替えた。ゆくゆくセーブデータの更新も留意しながら枠を広げる。

2017-07-12 20:29 Aktualisiert von: deskull
Kommentar
place_monster_okay()関数はESCORTによる護衛生成判定にしか使われていないので、place_monster_can_escort()に改名。
2018-09-04 21:33 Aktualisiert von: deskull
  • Details Updated
Kommentar

spells2.cに散見される主にボルト処理の2行のみのシンタックスシュガー的な関数群が、却って処理の検索にしづらかったり、半端にしか使われていないきらいがあるので一通り整理してみる。

2018-11-24 09:16 Aktualisiert von: deskull
Kommentar

このチケット付けのコミット増やし過ぎたので可能な限り分化したい。

2019-01-06 13:41 Aktualisiert von: deskull
  • Details Updated
2020-02-11 15:30 Aktualisiert von: deskull
Kommentar

スペル効果で得た「状態」を参照したり、付与/解除する処理に当たる関数やマクロについては各領域でなく、player-status.cに置く方向性で。これで #39912 のインクルード整理も捗り、大半の別処理が一々スペルの実行処理のみを定義するヘッダにインクルードしてまでアクセスしなくて済む。

2020-08-19 03:11 Aktualisiert von: deskull
  • Status Update from Offen to Geschlossen
Kommentar

今まで幅広く対象にし過ぎたが、分割した各チケットですべてカバーしていると判断しチケット完了とする。

2020-12-28 08:17 Aktualisiert von: deskull
2020-12-28 08:19 Aktualisiert von: deskull
Kommentar

後半Hourier氏のおかげで大分助かった。多謝。

Dateianhangliste

Keine Anhänge

Bearbeiten

Please login to add comment to this ticket » Anmelden