【10/22】コインガチャチケット使用時ログ記録 #1237

概要

コインガチャチケット利用時と獲得時の情報をログを残すようにするPRです。
コインガチャチケットの情報が不明瞭過ぎるため対応しました。

テスト方法、確認した内容

CREATE TABLE `log_gacha_ticket` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `user_no` int(11) NOT NULL,
  `before_count` smallint(11) NOT NULL,
  `after_count` smallint(11) NOT NULL,
  `type` tinyint(11) NOT NULL COMMENT '1:獲得 2:消費',
  `status` smallint(11) NOT NULL COMMENT 'type1の場合は1がプレゼントから獲得,2がその他type2の場合はgacha_no',
  `reg_date` datetime NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

• プレゼントからガチャチケット受け取り時にログに記録されるか
• ステップアップガチャでガチャチケット利用時にログに記録されるか

TODO

• プレゼントからの獲得以外でチケットが直接増減する箇所がないかの確認
• アイテムセットにガチャチケットが含まれる場合に耐えるようにする必要があるかどうか。現状は耐えない？

@wada @iwamoto
確認お願いいたします。

@kouho
ある程度レコード数が増えると思うので、検索用のインデックスを追加して欲しいです。

idにnullを入れるのを初めて見たのですが、これでも大丈夫ですか？

#S_LogGachaTicket
$sqlSt = sprintf( "
	INSERT INTO
		`log_gacha_ticket`
			VALUES
		(NULL,%d,%d,%d,%d,%d,'%s')
;"

@wada
確認ありがとうございます。検索用のインデックス了解です。
以下のインデックスを追加します。

ALTER TABLE log_gacha_ticket ADD INDEX user_no_type_status_reg_date (user_no,type,status,reg_date);

idはAUTO_INCREMENTでNOT NULLなのでidにNULLを指定した場合、自動的にシーケンス番号が割り当てられるので大丈夫ですが、確かにジョリーで同じような書き方の箇所が見つかりませんでしたのでINSERT文の書き方を変更しました。
https://dev.mysql.com/doc/refman/5.7/en/example-auto-increment.html

@kouho
確認しました。OKです
用途が分からないのでインデックスに関しては明言できませんが、
期間ごとやガチャごとのようなuser_noに紐付かない集計をとる場合はありませんか？

@wada
目的がユーザーごとの所持ガチャチケットが何枚から何枚になったかを明確にすることなので、
user_noに紐付かない集計をすることは考えなくて大丈夫だと思います。

もしも特定期間内のガチャチケット使用枚数やガチャごとのチケット使用枚数を知りたい場合は
log_user_gacha_statusから見ればわかりますので。

@kouho
了解です。用途としては、個人のガチャチケット増減を確認するということですね。
管理画面上でチケットの日ごとの獲得数や、消費数などを表示したりする可能性があるのか気になったので確認しました。

@kouho
上記指摘のコード以外は問題ないと思います。

@iwamoto
確認ありがとうございます。指摘箇所を修正しました。

@kouho
確認しました。
問題ないと思います。

こちらコードの修正をお願い致します。
$user_info['gacha_ticket'-1