【2/12】建築イベント2月 #3913

概要

https://docs.google.com/spreadsheets/d/1BC9rQqTUWLrBtUCKwBhu2AEIEBC3kfJkggIcQovuHoE/edit?gid=133142113#gid=133142113

SP報酬のランダムアクセサリ対応は今回初
表示と集金時で異なるため、集金時に直接対応

画像は別途

@wada
レビューお願いいたします

@kouho
開始日時が仕様書と違います。

@kouho

SubFunc.inc#S_GetUserBuildGoldDetail()内の以下処理について

}{
	//ランダムアクセサリ対応は特殊なのでここで
	if($return_data['type'] == PresentItemConfig::ITEM_TYPE_COMPOSITION){
		switch($bonus_ratio){
			case 82://ランダムエメラルドのアクセサリ
				$item_nos = [2004,2009,2014,2019,2024,2029];
				S_error_log($return_data['count']);
				for($i=0; $i < $return_data['count']; ++$i){
					$item_no = $item_nos[array_rand($item_nos)];
					$sql[] = sprintf("(%d,%d,%s,%d,%d,%d,%d)"
						,$user_info['user_no']
						,0
						,S_quote_smart('集金時に手に入れました!',$_dbc[37])
						,2
						,$item_no
						,0
						,1
					);
				}
				break;
		}
		...
}

• ①：該当の処理が中括弧{}で囲まれていますが、なにか意味がありますか？

}{
	//ランダムアクセサリ対応は特殊なのでここで
	if($return_data['type'] == PresentItemConfig::ITEM_TYPE_COMPOSITION){
	...
}

• ②：case 82ですが、アクセの種類ごとに処理が切り替わるようなので、case item_noに変更をお願いします

case 82://ランダムエメラルドのアクセサリ

• ③：アクセのIDはS_GetCompositionRankItem()で取得できます

$item_nos = [2004,2009,2014,2019,2024,2029];

• ④：リリースまでにログ出力の削除をお願いします

S_error_log($return_data['count']);

• ⑤：ただの興味なんですが、for文内のインクリメントが前置なのはなんでなんですか？

for($i=0; $i < $return_data['count']; ++$i){

@kouho
集金履歴のアイテム表示も対応お願いします。
▼アクセを入手したが時計に置き換わっている
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1739250332613YKpd9moqOt" width="300">

@wada
特に意味がないため削除しました
> 該当の処理が中括弧{}で囲まれていますが、なにか意味がありますか？

修正しました
> case 82ですが、アクセの種類ごとに処理が切り替わるようなので、case item_noに変更をお願いします

修正しました
> アクセのIDはS_GetCompositionRankItem()で取得できます

削除しました
> リリースまでにログ出力の削除をお願いします

特に意識しないと前置インクリメントで書く癖があるようです。多分プログラム習いたての時からの癖みたいです。
> ただの興味なんですが、for文内のインクリメントが前置なのはなんでなんですか？

修正しました。ログの入れ方から変えておりますので、集金し直したら治ります。
> 集金履歴のアイテム表示も対応お願いします。
▼アクセを入手したが時計に置き換わっている

@kouho
一括集金時のアイテム画像も時計になっていました。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1739423551528HRvRn0l5uc" width="300">

@kouho
アイテムを取得すると、ログでエメラルドのアクセサリ表記がなくなります。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1739424120334UOg3Dri3zx" width="300">

@wada
適切な修正方法がよく分かりませんがどすうべきかわかりますか？
> 一括集金時のアイテム画像も時計になっていました。

@wada
修正しました
> アイテムを取得すると、ログでエメラルドのアクセサリ表記がなくなります。

@kouho
> 適切な修正方法がよく分かりませんがどすうべきかわかりますか？

item_no=4ではパラメータが他に必要になるので、集金時（単発、一括、~~ログ~~）のアイテム表示を抽選後アイテムにしてみるのはどうですか？

@kouho
仕様書には集金時のアイテム名表示について特に記載がないですが、
プランナー間でどちら（抽選前後）で表示することになっていますか？

@kouho
確認OKです。
SP効果については、次の島テーマ実装までにPR作成します。

@kouho
getメソッドの中で更新してほしくないです。
直近の例でいうと、getUserMission()のようなパターンです。