Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

モブへの攻撃時処理を追加 #677

Merged
merged 10 commits into from
Feb 19, 2025

Conversation

NePonpon
Copy link
Member

@NePonpon NePonpon commented Jan 17, 2025

チケット URL

NO-ISSUE

対応内容・対応背景・妥協点

プレイヤーがモブへ攻撃を与えた時の処理を実装しました。

やったこと

  • functions -> function

やってないこと

投擲物でのダメージについて、ダメージの受け渡しが他方の変更で不十分になっているので、完全に動作しません。
その他、ダメージに関係するトリガーを全部見直してからこの実装についてのデバッグを行います。

チェックリスト (ガイドライン)

  • Pull Request

    • PR のタイトルはわかりやすい名前が設定されていますか?(日本語でも可)
    • PR の必須項目はすべて記載していますか?
    • PR の必要ない項目は削除していますか?
    • PR の内容と変更内容は一致していますか?
      • 1 機能=1PR であるべきです。1 機能の途中でも問題ありません
  • Commit

    • Commit メッセージはルール通りですか?
      • コミットメッセージの先頭に[Add|Delete|Modify|Fix|Refactor|Move]
        等の動詞の原形を追加してください。
        (参考)
        (参考 2)
      • コミットメッセージの説明にはGH-〇〇でチケット番号をつけてください
        (チケットが存在しない場合はNO-ISSUEにしてください)
      • マージコミットの場合はこの限りではありません
    • コミットの粒度は適切ですか?
      • 1 コミット=1 要素(1 ロジック)の変更であるべきです
  • Branch

    • ブランチの名前は正しいですか?
      • ブランチ名先頭はfeature/[簡単な説明] または fix/[簡単な説明] であるべきです
      • 簡単な説明は英語であるべきです(不具合発生するので)
    • ブランチの向き、切り先は正しいですか?

@NePonpon
Copy link
Member Author

NePonpon commented Feb 4, 2025

基本はplayer:trigger/hurt_entity/enemy:damage/update_healthでダメージを付与します。つまり、物理ダメージのみの計算となります。

  • melee_attack
    AbsorptionAmountの差が物理ダメージになります。entity: damageへ物理ダメージ値をコピーしますが、ほとんどの場合無視されます。スキルなどによる攻撃力補正があるときに役立つと思います。
    スキルにMelee:trueがある場合、武器で与えたダメージは無視され、スキルに登録されている物理ダメージのみを与えます。

  • hit_projectile
    投擲物に保存されたダメージを与えます。また、弓の引き加減によるダメージも加味されるため、与えられるダメージは、
    矢のダメージ + 矢に保存された物理ダメージ となります。

魔法、理外は今のところ想定しておらず、物理ダメージのみが与えられます。

@NePonpon NePonpon marked this pull request as ready for review February 4, 2025 14:57
@NePonpon NePonpon requested a review from a team as a code owner February 4, 2025 14:57
Copy link
Contributor

@yabane yabane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

剣・弓・弓(近接)・トライデントで正常にダメージが付与できること・死亡処理が行われることを確認しました。

@NePonpon NePonpon merged commit cdf3cee into 1-21-1-dev Feb 19, 2025
3 checks passed
@NePonpon NePonpon deleted the feature/player-trigger-hurt_entity branch February 19, 2025 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants