【2/4】仲間集結キャンペーン #4486

概要

仕様書
https://docs.google.com/spreadsheets/d/1S6idyeT1StSmpCciYFDIiY2FNJtY0oi4JMPK6cG9ujY/edit?gid=2088028903#gid=2088028903

実装用シート
https://docs.google.com/spreadsheets/d/1rXTsIT1ROmFYP9WIt-x-RRNFPRYc0_eNLNZOygQbtPE/edit?gid=1458806153#gid=1458806153

SQL

CREATE TABLE `user_team_join_campaign` ( `id` bigint NOT NULL AUTO_INCREMENT COMMENT "管理ID", `campaign_no` int unsigned COMMENT "開催回", `user_no` int unsigned NOT NULL COMMENT "ユーザーID", `start_date` datetime NOT NULL COMMENT "参加確定/判定固定日時", PRIMARY KEY (`id`), UNIQUE KEY `uniq_campaign_user` (`campaign_no`,`user_no`) ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4

CREATE TABLE `user_team_join_campaign_quest` (
  `id` bigint NOT NULL AUTO_INCREMENT COMMENT '管理ID',
  `campaign_no` int unsigned NOT NULL COMMENT '開催回',
  `user_no` int unsigned NOT NULL COMMENT 'ユーザーID',
  `quest_no` int unsigned NOT NULL COMMENT 'クエストID',
  `receive_flg` tinyint unsigned DEFAULT '0' COMMENT '受取済フラグ',
  `border_type` tinyint unsigned NOT NULL COMMENT 'クエスト種別',
  `border_count` int unsigned NOT NULL COMMENT '達成条件数',
  `clear_count` int unsigned NOT NULL COMMENT '進捗カウント',
  `update_time` datetime NOT NULL COMMENT '最終更新日時',
  PRIMARY KEY (`id`),
  UNIQUE KEY `uniq_campaign_user_team_quest` (`campaign_no`,`user_no`,`quest_no`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

TODO

• [x] ログイン判定の妥当性の確認　→　加入時に判定追加

@wada
レビューして頂ける状態でないですが、実装用シートと現状の共有です

@wada
CPの進行と報酬の受け取りまでをプッシュしていますのでレビューをお願いいたします

@kouho
①②はコメントが隠れていたので再度コメントしています。

• ①：lib/models/dao/user_info.phpでget_by_user_no()メソッドは使用していますか？-
• ②：lib/models/dao/user_team_join_campaign.phpのinsert_usersでarray_chunk()を使ってより理解しやすいコードにしたいです。

$total = 0;
foreach (array_chunk($values, $chunk_size) as $chunk) {
    $this->insert_bulk($chunk);
    $total += count($chunk);
}

• ③：lib/exec/tool_team_join_campaign_target.phpで失敗時を想定してトランザクション対応は不要ですか？

@kouho
defineファイルでconfig配列を$_dfに格納していますが、$_dfではなくconfig->toArray()を呼び出してconfig内の変数を参照しています。
こちらはなにか理由がありますか？

#define_{pf}.inc
$_df['team_join_cp'] = TeamJoinCampaignConfig::get()->toArray();

#contoller,model
$config = TeamJoinCampaignConfig::get()->toArray();
$config = $_df['team_join_cp']; #$_df代入ではダメ？

@wada
確認ありがとうございます。以下修正しています。

> lib/models/dao/user_info.phpでget_by_user_no()メソッドは使用していますか？-
使用していないため、削除

> lib/models/dao/user_team_join_campaign.phpのinsert_usersでarray_chunk()を使ってより理解しやすいコードにしたいです。

> lib/exec/tool_team_join_campaign_target.phpで失敗時を想定してトランザクション対応は不要ですか？
トランザクション対応とRollback追加

> defineファイルでconfig配列を$_dfに格納していますが、$_dfではなくconfig->toArray()を呼び出してconfig内の変数を参照しています。こちらはなにか理由がありますか？
特に理由がありませんでしたので、global変数を参照する形に修正しました。

@kouho
Team_join_campaign.phpのupdate_clear_count()で以下の処理が2回呼び出されています。
・新規作成後の再取得であれば、if文内のcreate_quest()直後
・更新後の判定用であれば、取得済みのデータをamountを利用

$user_quest_dao->get_by_campaign_user_team($campaign_no, $user_no);

@kouho
quest_noとquest_id、reward_noとreward_idが混在しているので、どちらかに合わせておいて欲しいです。
タイプミスを誘発します。

@kouho
Banner.phpのバナー情報もTeam_join_campaign.phpのget_banner_info()を使用してください。
設定は1箇所にまとめておきたいです。

@kouho
user_team_join_campaignテーブルのcp_statusは、どこかで更新したり参照していますか？

@wada
確認ありがとうございます。以下対応いたしました。

> quest_noとquest_id、reward_noとreward_idが混在しているので、どちらかに合わせておいて欲しいです。
タイプミスを誘発します。

quest_noに合わせました。reward_noとreward_idはreward_noに統一しております。

> user_team_join_campaign_rewardを使用せず user_team_join_campaign_questのreceive_flgをセット

user_team_join_campaign_rewardの削除と、user_team_join_campaign_questのreceive_flgにカラム追加対応いたしました。

> Banner.phpのバナー情報もTeam_join_campaign.phpのget_banner_info()を使用してください。
設定は1箇所にまとめておきたいです。

Banner.phpにまとめ対応いたしました。

@kouho
初日のログインクエストは、すでにゲームを始めていて当日ログインしているとカウントされませんか？

@kouho
master_team_join_campaign_questのquest_nameカラム追加がpushされていないと思います。

@wada
以下対応いたしました。
> Team_join_campaign.phpのupdate_clear_count()で以下の処理が2回呼び出されています。

1回だけの呼び出しに修正

> master_team_join_campaign_questのquest_nameカラム追加がpushされていないと思います。

master_team_join_campaign_questのquest_nameカラムは最初作りましたが、後で削除したものです。
参照箇所は$this->get_quest_descriptionを参照するように修正しました。

@wada
判定の順番を修正しました
> クエスト進捗を取得してからログイン判定しているので、初回アクセス時にリストの進捗に反映されていません。

@kouho
このメソッドは使用していますか？

@kouho
array_chunkを使ってより理解しやすいコードにしたいです。

$total = 0;
foreach (array_chunk($values, $chunk_size) as $chunk) {
	$this->insert_bulk($chunk);
	$total += count($chunk);
}

@kouho
クエスト進捗を取得してからログイン判定しているので、初回アクセス時にリストの進捗に反映されていません。