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

新クトゥルフ(パルプ・クトゥルフ):連射コマンドの機能追加、連射結果表示の改善 #239

Merged
merged 5 commits into from
Jul 11, 2020

Conversation

AdmiralNyar
Copy link
Contributor

連射コマンドの機能追加および連射結果表示の改善

連射コマンドの機能追加

変更箇所 Before After
コマンド FAR(w,x,y,z,d)
FAR(w,x,y,z,d,v)
v:ボレーの弾丸の数を変更する。省略可。
出力例 FAR(10,60,99,-1,h)
Cthulhu7th : ボーナス・ペナルティダイス[-1]
1回目: → 20, 30 → ハード成功
2回目: → 10, 10, 10 → イクストリーム成功
→ 5発が命中、2発が貫通、残弾0発
FAR(10,60,99,-1,h,3)
Cthulhu7th : ボーナス・ペナルティダイス[-1]
1回目: → 20, 30 → ハード成功     (2発が命中、0発が貫通)
2回目: → 10, 10, 10 → イクストリーム成功 (2発が命中、1発が貫通)
【難易度がハードに変更】
3回目: → 1, 1, 1 → クリティカル    (2発が命中、1発が貫通)
【指定の難易度となったので、処理を終了します。】
→ 6発が命中、2発が貫通、残弾1発

連射結果表示の改善

  • 連射の各ロールごとに命中数と貫通数を整列して表示(それぞれのボレーで別の対象に攻撃している時のため)

  • 連射の各ロールごとの成功度表示を整列

  • 連射ロールの難易度変更表示を括弧で強調

テストデータの更新

  • 結果表示変更に伴う連射ロールテストデータの修正

  • ボレーの弾丸の数の指定時のエラー表示のテストを追加

  • ボレーの弾丸の数の閾値での処理の違いが反映されているかを確かめるテストを追加

@ysakasin
Copy link
Member

@AdmiralNyar
\t が悪さしてる気がします。レイアウトを合わせるために \t を使ったのだと思いますが、 \t もレイアウトに対して万能ではないので、使わない方が良いと思います。

日本語のテキストのみ環境で横のレイアウトを合わせるのは、フォント等の環境依存があまりにも大きいので、レイアウト合わせをしない方が良いです。

@AdmiralNyar
Copy link
Contributor Author

ありがとうございます!
\tの部分を抜いて再度試してみます。

レイアウトについては、最低限にしてみます。

@ysakasin
Copy link
Member

@AdmiralNyar 詳しくみてみたら、コードで\tに置き換えたところについて、テストケースのExpectedを \t に置き換えきれてないですね。

スクリーンショット 2020-07-11 13 40 55

@AdmiralNyar
Copy link
Contributor Author

エラーの原因は、ljustによるマルチバイト文字の数え方が、Ruby 1.8.7と他では違っていたからでした(ですので、マルチバイト文字に対するljustの使用を止めました)

レイアウトについては、\tを使うとひどく崩れそうなので、半角全角スペースのみでの整列にしました

お手すきの際にご確認いただけましたら幸いです

@ysakasin ysakasin merged commit e9c8bb9 into bcdice:master Jul 11, 2020
@ysakasin
Copy link
Member

@AdmiralNyar 改善ありがとうございます。マージしました

@AdmiralNyar
Copy link
Contributor Author

爆速でマージいただき、ありがとうございます!

@AdmiralNyar AdmiralNyar deleted the Cthulhu7th_FAR_Volley_Cap_add branch July 11, 2020 09:44
ysakasin pushed a commit that referenced this pull request Aug 30, 2020
* ボレーの弾丸の数を指定できるようにし、結果表示を改善。

* インフォメーション・テキストの例)を更新

* Ruby1.8.7で、マルチバイト文字をljustが他のバージョンと違う数え方をしていたことへの対応

* 環境による影響がひどそうなので、タブ文字を削除

* スペースの削除
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants