CASru-GAME / TeamGameDevBootcamp

チームで大規模Unity開発をするための、予行練習リポジトリ
0 stars 0 forks source link

20 能力値クラス実装 #24

Closed momomo546 closed 1 month ago

momomo546 commented 1 month ago

案として

しようと思ってます。
一つ目のvalueはメソッドの計算で使います。

三つめの理由は

Elise3993 commented 1 month ago

↑のコミットでHPに関してテストスクリプトの実装・バグ修正を行いました。MPに関してはノータッチなので修正よろしくお願いします。あとテスト用スクリプトの↑の方にお気持ちコメント入っちゃったのでそれも修正してもらえると助かります(すいません)

あと、多分改行の方法の環境による差が混じってしまったみたいです。めっちゃ申し訳ないんですがこの辺り調べて揃えてもらえるとありがたいです。(自分も調べます) image

Elise3993 commented 1 month ago

HPのテストスクリプトを書いている途中で感じたのですが、関数実行のために新しくHPインスタンスを作るのがすごく冗長に感じました。確かに値オブジェクトとしてはこの書き方が正しいとは思うのですが、少々変な感じがしました。(感想ですが)

https://github.com/CASru-GAME/TeamGameDevBootcamp/blob/994e41817197131fc0f81b8f48669da97e1b6e69/Assets/App/Data/Parameters/TestHp.cs#L63-L84

Elise3993 commented 1 month ago

あと、private Hp(int maxHp, int currentHp)の引数の位置を逆にしたく思いました。 表記としては currentHp / maxHp (50 / 80) みたいなイメージがあるので不自然に思いました

https://github.com/CASru-GAME/TeamGameDevBootcamp/blob/994e41817197131fc0f81b8f48669da97e1b6e69/Assets/App/Data/Parameters/Hp.cs#L23-L24