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

[0491/sc-group] シャッフルグループ、カラーグループが初期化されない問題を修正 #1176

Merged
merged 1 commit into from Dec 12, 2021

Conversation

cwtickle
Copy link
Owner

🔨 変更内容 / Details of Changes

  1. シャッフルグループ、カラーグループが単一のとき、初期化されない問題を修正しました。

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

  1. 以下の問題が発生し、うまく初期化できていませんでした。
    • 設定画面に入ったときsetDifficulty関数を通過しますが、
      このときに保存済みのキー設定を読み出す処理があります(getKeyCtrl関数)。
      この中にg_keyObjの「shuffle」や「color」の番号配列が含まれていたため、
      一時的にこれらの番号を変更しても、元に戻る挙動になっていました。
      ⇒ 前にプレイした状態が「KeyPattern : Self」の場合は、g_keyObjの「shuffle」「color」「chara」「pos」「stepRtn」を変更しないようにしました。
    • キーコンフィグ画面に入ったとき、setColorType関数を通過する処理があります。
      このとき、「color」の番号配列の1番目(ColorGroup: 1)を適用するようになっていました。
      これだと、ColorGroup: 2 などを指定してプレイし、元に戻った際にColorGroup: 1が呼び出されてしまう問題があります。
      ⇒ カラーグループが複数ある際は、現在のカラーグループに対するg_keyObjの「color」の番号配列を取得するように変更しました。

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

📝 その他コメント / Other Comments

  • これらの修正を行っても、以下の挙動になりますがこれは仕様とします。
    • シャッフルグループやカラーグループが単一のキーパターンの場合、
      キーを変えると設定が元に戻る。

理由補足

  • これは、シャッフルグループやカラーグループが複数保持しているパターンでは
    グループ内で個別に設定を保持している(shuffleX_Y_Z:Zはグループ個別番号)のに対して、
    グループが単一の場合、1つしか設定を持っていないためである(shuffleX_Y)。
  • 厳密にはすべて「shuffleX_Y_Z」のような形式に揃えるべきかもしれないが、
    現状カスタムキーではシャッフルグループやカラーグループに対応していないため、
    現在の状況下ではこの仕様に留める。

@cwtickle cwtickle added bug Something isn't working func:KeyConfig labels Dec 12, 2021
@codeclimate
Copy link

codeclimate bot commented Dec 12, 2021

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

View more on Code Climate.

@cwtickle cwtickle added the need-to-fix-supportVer. サポートバージョンへの修正が必要な場合にラベル付け label Dec 12, 2021
@cwtickle cwtickle merged commit 1098bfc into develop Dec 12, 2021
@cwtickle
Copy link
Owner Author

過去バージョンへの反映についてはv23のみ反映予定です。
シャッフルグループ・カラーグループの実装はv22からですが、
v22では実装方法が一部異なり、直近3つの変更を加えると一時的に変更したシャッフルグループ・カラーグループがタイトルに戻って再度プレイした際に反映されない問題が出ます。

v23以降の実装を行うのは修正範囲が多く、微調整とは行かないためv22の修正は見送ります。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working func:KeyConfig need-to-fix-supportVer. サポートバージョンへの修正が必要な場合にラベル付け
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant