クイズで同じ問題を出さない方法、次の問題で問〇を1増やす方法、最後に得点を表示する方法

Question

前提・実現したいこと

railsでクイズアプリを作っています。最初の2つ解決することが出来ました。本当にありがとうございます。

• データベースからランダム取得している問題を、一度取得したものは表示しないようにしたい

どうしてもこれだけがわかりません。

発生している問題・エラーメッセージ

-教わった @drone_questions = DroneQuestion.where.not(id:表示したID)order("RANDOM()").first
を使ってみています。
自己解決のソースコードの通り表示したIDをparamsで受け取り@donesに入れ、以下のようにやっていますがうまくいきません。最後の壁と思ってがんばりたいです。少し知恵を頂きたいです。

予想のソースコード

def overtest2
@drone_questions = DroneQuestion.order("RANDOM()").first
if @dones != @drone_questions.id
   @dones.push(@drone_questions.id)
else
@drone_questions = DroneQuestion.where.not(id:@dones)order("RANDOM()").first
end
end

2つ解決したコードは自己解決の欄に書きましたが、これを追加してもうまくいかないので解決欄には書いておりません。

どうか知恵をお貸しください。

Comments

[winterboum]

loginしてやるのですか、login不要ですか
今回始めてからの得点累積、同じ問題出さない なのか
今までの利用実績も含めて 同じ問題を出さない、得点累積なのか
どちらでしょう

[papanchoi0523]

おはようございます。
ログインはせずに使えるようにします。
いままでの実績は関係なく、topページの「試験開始」を押したら0点から+2していきたいです。全部の問題が終わったら、再度やる場合はリセットした状態から？と申せばよいのでしょうか。

出題に関しても同じで、今までの利用実績は関係なく、そのテスト中だけ出題されなくしたいです。

質問内容が足りずもうしわけございません。どうぞよろしくお願い申し上げます。

Answer 1

私が「loginしてやるのですか、login不要ですか」と質問し、
tacsheaven さんが「「画面間の情報は、明示的に引き継ぐ処理をしない限り引き継がれない」ことを憶えておきましょう。」 と解説している
のは同じ問題を指摘しています。tacsheavenさんの回答にたいするあなたのコメントを見ると、そこを全く理解してくれていません。
一度出した問題を記録に残すということを「一度表示したIDのレコードを新たなテーブルに.newで作成し」ということでは解決しません。loginしていないのですから誰に出した問題なのかがそれでは判断出来ない。

loginさせる場合は、user_idとともにしまえば判断できますが、loginさせないとすると、tacsheavenさんの回答にあるように、「「別画面でも必要な情報」は、画面にパラメータで渡すか、あるいはセッションやクッキーを使用して保存するなどして、引き継がせなければなりません。」です

」

Comments

[papanchoi0523]

ご説明ありがとうございます。問の連番、スコア、一度表示した問題は全てparamsでviewからcontrollerに渡してやってみます。また、新たなアクションに変数をつくり、一度表示した問題をその変数に代入していき、「htmlではif文でランダム表示する際に、一度表示した問題を入れた変数だったら表示しない」という方法なのでしょうか。
たとえばdone = @drone_questions で今ランダム表示した問題を代入し、次からはif文で
```
<% if @drone_questions ！= done %>
<%= @drone_questions %>
end
```
のような形?のイメージになっています。とりあえず問の連番の問題から解決していこうとしていますがいまだ未解決です。色々と教えてくださりありがとうございます(#^.^#)

[winterboum]

viewで判断することもできますが、私の好みとしてはviewではあまりプログラムしたくありません。
controllerかmodelでやります。例えば
DroneQuestion.order("RANDOM()").first
↓
DroneQuestion.where.not(id: すでに出した問題のidの配列).order("RANDOM()").first

[papanchoi0523]

出来るだけviewではプログラムしないのですね。そうしてみますありがとうございます。
詳しく教えて頂きありがとうございます。既出を出題しないコードはこれを真似してみます。
丁度今、問の連番についてやっと解決しテンションが上がっておりました(#^^#)　解決方法の部分に書きます。引き続き既出問題を表示しないようにする部分についてがんばります(*^-^*)

Answer 2

まず、Rails に限らずですが、Web アプリケーションは「画面間の情報は、明示的に引き継ぐ処理をしない限り引き継がれない」ことを憶えておきましょう。
別画面に遷移する場合に、「別画面でも必要な情報」は、画面にパラメータで渡すか、あるいはセッションやクッキーを使用して保存するなどして、引き継がせなければなりません。

そしてそれが理解できたら、問の連番が変わらないこと、スコアも増えていかないことが理解できているはずです。

一度表示した問題を表示しないためには、「一度表示した問題のリスト」がないといけません（そのリストに入っている問題は表示対象としない）。これも「画面間で引き継ぐべき情報」です。

Comments

[papanchoi0523]

こんにちは。ご返信ありがとうございます。
問とスコアはparams でやってみます。

問題はDBからランダム取得しているのですが、一度表示したIDのレコードを新たなテーブルに.newで作成し、DB２つをJOINして、どちらにもあてはまるもののみ表示する。where と and　を使ってという事でしょうか？

Answer 3

(結局自己解決できず、プロに教わりながらやりました)

Rails.application.routes.draw do
  
  get "top"=> "home#top"
  # For details on the DSL available within this file, see https://guides.rubyonrails.org/routing.html
  get "/home/exam" => "home#exam"
  get "/home/overtest2" => "home#exam"
  post "/home/overtest2" => "home#overtest2"
end

<% @point = 0 %>
<% @firsttoi = 1 %>
<h1><%= "問#{@firsttoi}" %></h1>
<%= @drone_questions.q %>
<%= button_to @drone_questions.choice1 , "/home/overtest2", { method: :post, params: {choice1: @drone_questions.choice1, answer: @drone_questions.answer, toi: @firsttoi, dones: [@drone_questions.id], score: @point}} %>
<%= button_to @drone_questions.choice2 , "/home/overtest2", { method: :post, params: {choice2: @drone_questions.choice2, answer: @drone_questions.answer, toi: @firsttoi, dones: [@drone_questions.id], score: @point}} %>
<%= button_to @drone_questions.choice3 , "/home/overtest2", { method: :post, params: {choice3: @drone_questions.choice3, answer: @drone_questions.answer, toi: @firsttoi, dones: [@drone_questions.id], score: @point}} %>
<%= button_to @drone_questions.choice4 , "/home/overtest2", { method: :post, params: {choice4: @drone_questions.choice4, answer: @drone_questions.answer, toi: @firsttoi, dones: [@drone_questions.id], score: @point}} %>

<% if @right %>
<%= @right %>
<% else %>
<%= @wrong %>
<% end %>
</p>

<% if @drone_questions.present? %>

<!-- いったん間違っても得点を表示するようにしました -->
<p>
現在の得点は<%= @scores %>点です。
</p>

<h1><%= "問#{@toi}" %></h1>
<%= @drone_questions.q %>
<%= button_to @drone_questions.choice1 , "/home/overtest2", { method: :post, params: {choice1: @drone_questions.choice1, answer: @drone_questions.answer, toi: @toi, dones: @dones, score: @scores}} %>
<%= button_to @drone_questions.choice2 , "/home/overtest2", { method: :post, params: {choice2: @drone_questions.choice2, answer: @drone_questions.answer, toi: @toi, dones: @dones, score: @scores}} %>
<%= button_to @drone_questions.choice3 , "/home/overtest2", { method: :post, params: {choice3: @drone_questions.choice3, answer: @drone_questions.answer, toi: @toi, dones: @dones, score: @scores}} %>
<%= button_to @drone_questions.choice4 , "/home/overtest2", { method: :post, params: {choice4: @drone_questions.choice4, answer: @drone_questions.answer, toi: @toi, dones: @dones, score: @scores}} %>

<% else %>

あなたの得点は<%= @scores %>点でした。
<p>
<%= link_to "戻る", '/home/exam' %>
<% end %>

class HomeController < ApplicationController
  before_action :exam, only: [:overtest2]
  def top

  end

  def exam
    # ランダムに問題を1問抽出
    @drone_questions = DroneQuestion.order("RANDOM()").first
  end

  def overtest2
    # パラメータ受け取り     
    @choice1 = params[:choice1]
    @choice2 = params[:choice2]
    @choice3 = params[:choice3]
    @choice4 = params[:choice4]
    @answer = params[:answer] # 本来なら正解は問題文表示に渡さずDBから取ってくるべき
    @toi = params[:toi]
    @toi = @toi.to_i + 1
    @dones = params[:dones] # これまで回答した問題ID（html側も配列で持つように変更済み）
    @done = @dones.last # 最後に回答した問題ID
    @scores = params[:score]

    if @choice1 == @answer
      @right = "正解です！"
      @scores = @scores.to_i + 2
    else
      @wrong = "間違いです。正解は、「#{@answer}」です。"
      @scores = @scores.to_i + 0
    end
    if @choice2 == @answer
      @right = "正解です！"
      @scores = @scores.to_i + 2
    else
      @wrong = "間違いです。正解は、「#{@answer}」です。"
      @scores = @scores.to_i + 0
    end
    if @choice3 == @answer
      @right = "正解です！"
      @scores = @scores.to_i + 2
    else
      @wrong = "間違いです。正解は、「#{@answer}」です。"
      @scores = @scores.to_i + 0
    end
    if @choice4 == @answer
      @right = "正解です！"
      @scores = @scores.to_i + 2
    else
      @wrong = "間違いです。正解は、「#{@answer}」です。"
      @scores = @scores.to_i + 0
    end
    
    @drone_questions = DroneQuestion.where.not(id: @dones).order("RANDOM()").first
    if @drone_questions.present?
      # 次の問題がある
      @dones.push(@drone_questions.id); # 次の問題IDを履歴に追加
    end
  end
end

Answer 4

正解ごとに得点を２点づつ増やすことに成功しました。教わった通りparamsで受け取りました。正解なら+2だけでなく外れなら+0としないと、一度外れた場合また0点からスタートしてしまいましたのでif文にelseで+0が必要です。

<% @point = 0 %>
<% @firsttoi = 1 %>
<h1><%= "問#{@firsttoi}" %></h1>
<%= @drone_questions.q %>
<%= button_to @drone_questions.choice1 , "/home/overtest2", { method: :get, params: {choice1: @drone_questions.choice1, answer: @drone_questions.answer, toi: @firsttoi, done: @drone_questions.id, score: @point}} %>
<%= button_to @drone_questions.choice2 , "/home/overtest2", { method: :get, params: {choice2: @drone_questions.choice2, answer: @drone_questions.answer, toi: @firsttoi, done: @drone_questions.id, score: @point}} %>
<%= button_to @drone_questions.choice3 , "/home/overtest2", { method: :get, params: {choice3: @drone_questions.choice3, answer: @drone_questions.answer, toi: @firsttoi, done: @drone_questions.id, score: @point}} %>
<%= button_to @drone_questions.choice4 , "/home/overtest2", { method: :get, params: {choice4: @drone_questions.choice4, answer: @drone_questions.answer, toi: @firsttoi, done: @drone_questions.id, score: @point}} %>

<p>
<% if @right %>
<%= @right %>
<br>
<p>
現在の得点は<%= @scores %>点です。
</p>
<% else %>
<%= @wrong %>
<% end %>
</p>
<h1><%= "問#{@toi}" %></h1>
<%= @drone_questions.q %>
<%= button_to @drone_questions.choice1 , "/home/overtest2", { method: :get, params: {choice1: @drone_questions.choice1, answer: @drone_questions.answer, toi: @toi, done: @drone_questions.id, score: @scores}} %>
<%= button_to @drone_questions.choice2 , "/home/overtest2", { method: :get, params: {choice2: @drone_questions.choice2, answer: @drone_questions.answer, toi: @toi, done: @drone_questions.id, score: @scores}} %>
<%= button_to @drone_questions.choice3 , "/home/overtest2", { method: :get, params: {choice3: @drone_questions.choice3, answer: @drone_questions.answer, toi: @toi, done: @drone_questions.id, score: @scores}} %>
<%= button_to @drone_questions.choice4 , "/home/overtest2", { method: :get, params: {choice4: @drone_questions.choice4, answer: @drone_questions.answer, toi: @toi, done: @drone_questions.id, score: @scores}} %>

before_action :exam, only: [:overtest2]
  def top

  end

  def exam
    @drone_questions = DroneQuestion.order("RANDOM()").first
    
  end

  def overtest2
     
    @choice1 = params[:choice1]
    @choice2 = params[:choice2]
    @choice3 = params[:choice3]
    @choice4 = params[:choice4]
    @answer = params[:answer]
    @toi = params[:toi]
    @toi = @toi.to_i + 1
    @dones = params[:done]
    @scores = params[:score]
      
     
    if @choice1 == @answer
      @right = "正解です！"
      @scores = @scores.to_i + 2
    else
      @wrong = "間違いです。正解は、「#{@answer}」です。"
      @scores = @scores.to_i + 0
    end
    if @choice2 == @answer
      @right = "正解です！"
      @scores = @scores.to_i + 2
    else
      @wrong = "間違いです。正解は、「#{@answer}」です。"
      @scores = @scores.to_i + 0
    end
    if @choice3 == @answer
      @right = "正解です！"
      @scores = @scores.to_i + 2
    else
      @wrong = "間違いです。正解は、「#{@answer}」です。"
      @scores = @scores.to_i + 0
    end
    if @choice4 == @answer
      @right = "正解です！"
      @scores = @scores.to_i + 2
    else
      @wrong = "間違いです。正解は、「#{@answer}」です。"
      @scores = @scores.to_i + 0
    end
    DroneQuestion.where.not(id: @dones).order("RANDOM()").first
    
  end
end

Answer 5

問〇を次の問題に行くたびに1ずつ増やすことが出来ました。
button_toに問〇の部分のパラメータを渡してもらい、homecontrollerで受け取り@toiに代入しました。そして、@toiを+1したかったので@toi+=1しか頭になく、どこをどうやってもエラーでしたが、@toi=@toi.to_iにする事で解決しました。【変更点：firsttoiはデータベースから取得せず最初だけHTMLに直接書きました。また、やはり+1が必要だったので@toi = @toi.to_i + 1で解決しました。】

<% @firsttoi = 1 %>
<h1><%= "問#{@firsttoi}" %></h1>
<%= @drone_questions.q %>
<%= button_to @drone_questions.choice1 , "/home/overtest2", { method: :get, params: {choice1: @drone_questions.choice1, answer: @drone_questions.answer, toi: @firsttoi}} %>
<%= button_to @drone_questions.choice2 , "/home/overtest2", { method: :get, params: {choice2: @drone_questions.choice2, answer: @drone_questions.answer, toi: @firsttoi}} %>
<%= button_to @drone_questions.choice3 , "/home/overtest2", { method: :get, params: {choice3: @drone_questions.choice3, answer: @drone_questions.answer, toi: @firsttoi}} %>
<%= button_to @drone_questions.choice4 , "/home/overtest2", { method: :get, params: {choice4: @drone_questions.choice4, answer: @drone_questions.answer, toi: @firsttoi}} %>

def exam
    @drone_questions = DroneQuestion.order("RANDOM()").first
   
  end

  def overtest2
    @choice1 = params[:choice1]
    @choice2 = params[:choice2]
    @choice3 = params[:choice3]
    @choice4 = params[:choice4]
    @answer = params[:answer]
    @toi = params[:toi]
    @toi = @toi.to_i　+　1