ゾートシヴァがモンスター検索で常に検出される (フォーラムメッセージ #86644 からの引用)

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

詳細

フォーラム 変愚蛮怒3.0.0Alpha開発フォーラム [#86644] からの引用

[forum: 86644]

> 文字の説明(/) で名前で検索(^M)すると、ゾートとシヴァの化身の軟革ブーツが常に検索に引っかかります。

配列外アクセスの問題だった
コード中に散らばったマジックナンバーもいい加減どうにかしないといけない

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

2021-01-19 22:32 更新者: hourier

• 新しいチケット "ゾートシヴァがモンスター検索で常に検出される (フォーラムメッセージ #86644 からの引用)" が作成されました

2021-01-20 22:14 更新者: deskull

コメント

ひとまずマージしました、が例の無敵（以下略）も含めて80byte分にとどめてアウトになっている個所はまだいくつもあるはずなのでチケット自体はまだ据え置きにしておきます。

2021-01-20 23:11 更新者: hourier

コメント

feature/Fix-Monster-OutOfArrayAccess にて継続対応しましたが、あまり自信がないです

一応「80」で全文検索して、それっぽいものをMAX_MONSTER_NAME に置換したつもりです

レビューを実施し、問題ありましたらこのチケットで指摘して下さい

・developとの差分において、モンスター名でないものまで置換していないか (←優先度高、今後の保守で担当者を混乱させないため

・上に同じく、モンスター名であるはずの配列サイズを置換し損ねていないか (←優先度低、最悪都度対応でも良い

なお同値の定数マクロにMAX_NLEN というものもありましたが、「オブジェクトの最大サイズ」とちょっとあやふやな表記だったので使用しませんでした

同一目的で運用されているのであればその旨もご連絡下さい、こちらで修正します

2021-01-21 08:14 更新者: deskull

コメント

MAX_NLENはコメの通り、オブジェクト（アイテム）名のつもりで命名したものですね。ネタや機能拡張でアイテムの名前本体や、記述が増えた場合にスムーズに拡張できるようにという観点で加えたことを思い出しました。 モンスター名とば別名、かつ分かりやすい定数にすべきかもしれません。ここまでの内容をマージしつつ、まだもう少しこの観点の問題があると考えチケットは維持しましょう。

2021-01-21 08:19 更新者: deskull

コメント

ブランチ自体も維持します。

2021-01-21 09:19 更新者: hourier

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

コメント

レビューありがとうございました、モンスター名でないものは置換されていないようなのでこのチケットはクローズします

別チケット#41296を作成済なので以後はそちらで対応します

(チケットの寿命を伸ばすのは問題自体が長期化する原因になる＋ブランチの寿命も伸びて差分が増えるので好ましくない)

2021-01-21 22:31 更新者: deskull

コメント

了解しました。まずはお疲れ様です。