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

improve: [0810] シャッフルグループ数の上限を見直し、ミラー譜面のハイスコア保存条件を変更 #1672

Merged
merged 3 commits into from
Jun 8, 2024

Conversation

cwtickle
Copy link
Owner

@cwtickle cwtickle commented Jun 8, 2024

🔨 変更内容 / Details of Changes

1. シャッフルグループ数の上限変更

  • シャッフルグループ数の上限を(対応キー - 1)になるように修正しました。
  • 7keyであれば6、12keyであれば11が上限となります。

2. Mirror/X-Mirrorのハイスコア保存条件の変更

  • キーコンフィグ画面右上の ShuffleGr: 1 と同じシャッフルグループで無い場合、
    Mirror/X-Mirrorのハイスコアを更新しないよう変更しました。
  • 更新条件に合致しない Mirror / X-Mirror 譜面の場合、プレイ開始時にメッセージが表示されるようになります。

3. シャッフル利用時の譜面名の表記変更

  • キーコンフィグ画面右上の ShuffleGr: 1 と同じシャッフルグループで無い場合、
    譜面名のシャッフル名に(S)を付加するようにしました。
    OFF/Random+/S-Random+/Scatter+のようにグループに関係のないシャッフルの場合は付加されません。

🔖 関連Issue, 変更理由 / Related Issues, Reason for Changes

  1. 対応キー数を超えて設定を可能にすると、正規と同じ譜面をミラーとしてプレイ可能となってしまうため。
    最低でも1組は2キー以上属するように変更しています。
    ※対応キーの半分まで、といった閾値も考えましたが多様性を考慮して正規と合致しない上限としました。
  2. 1.関連ですが、これを許してしまうと何の譜面に対するハイスコアかが特定できなくなるため。
  3. 2.関連の修正です。これを適用するとShuffleGr: 2, 3の場合も(S)が表示されますが、
    あえて区別する必要性が薄いため今回は一律同じ記号にしました。

📷 スクリーンショット / Screenshot

📝 その他コメント / Other Comments

  • (S) は (Self) の略名です。

Copy link

codeclimate bot commented Jun 8, 2024

Code Climate has analyzed commit aef1615 and detected 0 issues on this pull request.

View more on Code Climate.

@cwtickle cwtickle merged commit 135c365 into develop Jun 8, 2024
5 checks passed
@cwtickle cwtickle deleted the feature/0810-shuffle branch June 8, 2024 07:59
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.

None yet

1 participant