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

Keep buying Halloween season repeatedly #77

Open
hitsub opened this issue Oct 4, 2021 · 4 comments · Fixed by #92 · May be fixed by #85
Open

Keep buying Halloween season repeatedly #77

hitsub opened this issue Oct 4, 2021 · 4 comments · Fixed by #92 · May be fixed by #85
Labels
bug Something isn't working
Milestone

Comments

@hitsub
Copy link
Owner

hitsub commented Oct 4, 2021

  • 全シーズンのうち、チョコレートエッグのみ未購入
  • シーズン完了後の切り替え先を「ハロウィン」にしている

とき、ハロウィンのシーズン開始アップグレードを購入し続ける。
買うSEがするだけで問題はない(はず)

@hitsub hitsub added the bug Something isn't working label Oct 4, 2021
@hitsub hitsub added this to the 0.7.1 milestone Oct 4, 2021
@hitsub
Copy link
Owner Author

hitsub commented Oct 5, 2021

ハロウィンのスイッチを購入し続けていたわけではなかった。
もともとこの機能はアップグレード完了後も動作することを想定していなかったため、
ハロウィンのアップグレード完了後に以下の処理を踏むことになる。

else if (Game.season == "halloween")
{
    /// 省略 ///
    //エルダー誓約の撤回が出来る場合はする(Wrinklerをスポーンさせる必要があるため)
    if (Game.UpgradesInStore.indexOf(Game.Upgrades["Revoke Elder Covenant"]) != -1)
    {
        Game.Upgrades["Revoke Elder Covenant"].buy();
    }
    if (halloweenRate >= 1)
    {
        //エルダー誓約を購入してババアポカリプスを終了させてから次に行く
        Game.Upgrades["Elder Covenant"].buy(1);
        CookieAssistant.SwitchNextSeason();
    }
}

これにより、エルダー宣誓&エルダー宣誓の撤回を同時に買い続ける現象が発生した。

@hitsub
Copy link
Owner Author

hitsub commented Oct 5, 2021

どうしようこれ…

@hitsub
Copy link
Owner Author

hitsub commented Oct 5, 2021

ハロウィンに固定する目的はGC確率+2%だけだと思うが、エルダー宣誓や撤回は勝手に買ってほしくないと思うので完了後は止めたい。
フラグ持って転生ごとにフラグリセットをかけるしかない…?

@hitsub
Copy link
Owner Author

hitsub commented Dec 20, 2021

修正方針

そもそも、
シーズンが全て完了している && 完了後のシーズンと現在のシーズンが一緒 => autoSwitchSeasonの実行をせずEarly Return
とすればよい。

全シーズン終えていて、完了後のシーズンを切り替える時に1回無駄な処理が入ってしまうのは許容でいい。

(それはそれとしてシーズン切り替え処理が汚すぎるのでリファクタしたい…)

@hitsub hitsub modified the milestones: 0.7.2, 0.7.1 Dec 21, 2021
hitsub added a commit that referenced this issue Dec 21, 2021
シーズン完了後はauotoSwitchSeasonの処理が走らないように修正
@hitsub hitsub linked a pull request Dec 21, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
1 participant