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

[Refactor] グローバル変数 cur_store_num を削除する #3416

Merged
merged 1 commit into from
Jul 4, 2023

Conversation

sikabane-works
Copy link
Owner

基本的に cur_store_num は現在プレイヤーが居る店舗を持つグローバル変数のようだが、
すべて引数で渡せばグローバル変数にする必要が見当たらない。
例外としてプレイヤーが店舗に居ない時に呼ばれる store_shuffle() で cur_store_num に店主入れ替えを行った店舗の番号を入れているようだが、これにしても次にプレイヤーが店舗に
入った時に更新されるので意味がなさそう。
したがってグローバル変数 cur_store_num を完全に削除する。

副作用として、グローバル変数とローカル変数の取り違いが原因で発生しているバグ #2442
修正される。

基本的に cur_store_num は現在プレイヤーが居る店舗を持つグローバル変数のようだが、
すべて引数で渡せばグローバル変数にする必要が見当たらない。
例外としてプレイヤーが店舗に居ない時に呼ばれる store_shuffle() で cur_store_num
に店主入れ替えを行った店舗の番号を入れているようだが、これにしても次にプレイヤーが店舗に
入った時に更新されるので意味がなさそう。
したがってグローバル変数 cur_store_num を完全に削除する。

副作用として、グローバル変数とローカル変数の取り違いが原因で発生しているバグ #2442 も
修正される。
@sikabane-works sikabane-works added the enhancement New feature or request label Jul 4, 2023
@sikabane-works sikabane-works self-assigned this Jul 4, 2023
@sikabane-works sikabane-works merged commit ff2af09 into develop Jul 4, 2023
@sikabane-works sikabane-works deleted the merge/heng#2446 branch July 4, 2023 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

変愚「[Refactor] グローバル変数 cur_store_num を削除する #2446」のマージ
2 participants