Description
概要
フックモンスターが別々の人に同時に出ないようにする
テスト方法、確認した内容
- フックモンスター討伐時に問題なくレアモンスターモードに入るか
TODO
- 開発反映時に複数人で同時に動かして検証したい
- ~~可能なら5/10の新大陸開催時に反映したい~~
Comment
@kouho
イベント期間中だと、どのステージでもロックしているので、イベントステージのみロックするように変更をお願いします。check_pop_monster()内でget_lock()の前にif( $user_mission['stage_type'] >= 10 && $user_mission['stage_type'] <= 12 )のイベントステージ判定
このタイミングで、ロック周りのtime()をDateTimeUtil::getCurrentTIme()に変更しても良いと思います。
Comment
@wada
確認ありがとうございます。以下修正しました。
- イベントステージのみロックをする
- ロック周りのtime()をDateTimeUtil::getCurrentTIme()に
- イベントステージ判定が何回も行われていたのでまとめる
Comment
@kouho
修正確認しました。動作問題ありません。
ロック周り意識するのが久しぶりなので、以下の点が気になりました。
・team_event_new_island_mode_limitのselect頻度が多いならキャッシュした方が良いか
・team_event_new_island_mode_limitのcreate()前に再度get_lock()があるが問題ないか
・主キー以外でロックしているので、他のteamに影響は出ないか
Comment
今のテーブルをログテーブルにロック用のテーブルを新たに作成
kouho change_title at 2022-05-20T17:41:13: 【4/28】新大陸のフックモンスター出現処理にロック追加
[WIP]【4/28】新大陸のフックモンスター出現処理にロック追加
kouho delete_label at 2022-05-20T17:45:14: レビュー待ち
kouho change_title at 2022-06-17T17:01:36: [WIP]【4/28】新大陸のフックモンスター出現処理にロック追加
[WIP]新大陸のフックモンスター出現処理にロック追加
kouho change_title at 2022-07-26T18:35:17: [WIP]新大陸のフックモンスター出現処理にロック追加
新大陸のフックモンスター出現処理にロック追加
Comment
@kouho
イベント管理テーブルの前にロック
#S_GetTeamLock($this->user_info['taem_no']);
$event_rare_monster_limit = S_LoadModelClass('dao/team_event_new_island_mode_limit')->get_mode_limit($this->user_info['team_no']);Comment
@wada
確認ありがとうございます。
イベント管理テーブルの前にロックに修正しました。
Comment
@wada
確かにイベントステージだけの方がいいですね修正します
Comment
@wada
イベントステージのみロックに修正しました
wada close_comment at 2023-02-22T12:04:07: 対応が難しいためクローズ
commit comment
@kouho
修正前のコードを確認していたところ、
以前は、普通のステージでロックしないようにしていたのですが、全ステージでロックした方が良いですか？