PR

【早め】oxクイズ #1688

wada opened this at 2022-03-29T15:13:00; closed

Description

opened by wada at 2022-03-29T15:13:00; updated 2022-03-31T10:09:52

概要

【4/1】エイプリルフール #1670
全n問のoxクイズ
全問正解でアイテム配布

テーブル

CREATE TABLE `log_yes_no_quiz` (
  `event_no` int(11) unsigned NOT NULL,
  `question_id` tinyint(4) unsigned NOT NULL,
  `yes` int(11) unsigned NOT NULL,
  `no` int(11) unsigned NOT NULL,
  PRIMARY KEY (`event_no`,`question_id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

CREATE TABLE `user_yes_no_quiz` (
  `user_no` int(11) unsigned NOT NULL,
  `event_no` int(11) unsigned NOT NULL,
  `question_id` tinyint(4) unsigned NOT NULL,
  `clear_num` tinyint(4) unsigned NOT NULL DEFAULT '0',
  `reward_flg` tinyint(4) unsigned NOT NULL DEFAULT '0',
  PRIMARY KEY (`user_no`,`event_no`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

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

遷移する方法がまだないので、http://127.0.0.1:20080/?M=YesNoQuizで直接アクセスしてください。

TODO

  • [ ] 告知
  • [x] 遷移方法(バナー)
  • [ ] 画像(クイズ吹き出し、バナー、告知もあれば)
  • [x] クイズ設定
  • [ ] ページ内文言

Comment

#18112 by wada at 2022-03-29T15:15:13; updated 2022-03-29T15:15:13

@kouho
開発で確認しつつ進めたいので、先に処理のレビューをお願いします。
TODOの内容は後ほど実装します。

wada add_label at 2022-03-29T15:16:35: レビュー待ち

Comment

#18114 by kouho at 2022-03-29T15:42:00; updated 2022-03-29T15:42:00

@wada
log_yes_no_quizが誰が何と回答したかを保存するならuser_noもテーブルに入れている方がいいと思います。

Comment

#18115 by wada at 2022-03-29T15:45:39; updated 2022-03-29T15:45:39

@kouho
問題の正答率を見るだけなので user_no は外しています。

Comment

#18116 by kouho at 2022-03-29T15:46:45; updated 2022-03-29T15:46:45

@wada
console.logが残っています。レビュー時には確かめやすいですがマージ時にはコメントアウトしておいていただきたいです。

Comment

#18117 by kouho at 2022-03-29T15:48:42; updated 2022-03-29T15:48:42

@wada
正答率を見るだけなら大丈夫です。かしこまりました。

Comment

#18118 by kouho at 2022-03-29T16:17:09; updated 2022-03-29T16:17:09

@wada
基本的な仕組みは問題ないと思いますがログについて、
問題の正答率を見る想定のlog_yes_no_quizにはどういうデータの入り方をすると正しいでしょうか?
現状初回のデータの入り方は問題ないですが2回目以降のデータの入り方がyesもnoも+1されておかしくなっていないでしょうか。

Comment

#18123 by wada at 2022-03-29T17:33:43; updated 2022-03-29T17:33:43

@kouho
ログデータ更新修正しました。

Comment

#18124 by kouho at 2022-03-29T17:53:22; updated 2022-03-29T17:53:22

@wada
ありがとうございます。TODO以外の確認OKです。

Comment

#18125 by wada at 2022-03-29T18:03:53; updated 2022-03-29T18:03:53

@kouho
確認ありがとうございます。
一旦developマージします。

wada merge at 2022-03-29T18:03:57: 【早め】oxクイズ
wada close at 2022-03-29T18:03:57: Close

Commit range

base7fe0a5cc2040cbd30b15dec25dbec5dfce53c0c2
heada92bbf3085f1a9bc1887679b375701b00a313efd
base branchjolly/jr-web:develop
head branchjolly/jr-web:april_quiz

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed9 filesView patch
statusfile
addedlib/class/controller/JrControllerYesNoQuiz.class.php
addedlib/conf/class/YesNoQuizConfig.class.php
addedlib/models/Yes_no_quiz.php
addedlib/models/dao/log_yes_no_quiz.php
addedlib/models/dao/user_yes_no_quiz.php
modifiedpublic_html/index.php
modifiedpublic_html/template/Event.tpl
addedpublic_html/template/YesNoQuiz-Default.tpl
addedpublic_html/template/YesNoQuiz-Result.tpl
Patch body is stored as a separate file to keep this page lightweight.