バリデーション が機能しない

Question

前提・実現したいこと

投稿にバリデーション をかけてるのですが、上手くいきません、、、

コーヒーに関する投稿の名前、値段、タグ、コーヒーの説明、画像を
必須にすべくバリデーション をかけてるのですが、何も入力しないで投稿ボタンを押すと
空のままsaveされてroot_pathにリダイレクトされます。

空のままsaveされてるのかと思いましたが、例えば、名前だけ入力したら
データベースに保存されません。

全部のデータを過不足なくしっかり入力して投稿ボタン押したら
保存されました。

まだ入力されないなら、エラーメッセージを表示したいのですが、エラーハンドリングされずに
root_pathにリダイレクトされます。

該当のソースコード

drinks/new.html.erb

<div class="items-sell-contents">

  <div class="items-sell-main">
    <h2 class="items-sell-title">商品の情報を入力</h2>
    <%= form_with model: @drink,url: drinks_path, local: true do |f| %>


    <%= render 'shared/error_messages', object: f.object %>
    
    <div class="tag-field" , id='tag-field'>
        <%= f.label :tag_name %>
        <%= f.text_field :tag_name %>
    </div>
    <div id="search-result">
    </div>
    <div class="img-upload">
      <div class="weight-bold-text">
        出品画像
        <span class="indispensable">必須</span>
      </div>
      <div class="click-upload">
        <p>
          クリックしてファイルをアップロード
        </p>
        <%= f.file_field :image, id:"item-image" %>
      </div>
    </div>
    
    <div class="new-items">
      <div class="weight-bold-text">
        商品名
        <span class="indispensable">必須</span>
      </div>
      <%= f.text_area :name, class:"items-text", id:"item-name", placeholder:"商品名（必須 40文字まで)", maxlength:"40" %>
      <div class="items-explain">
        <div class="weight-bold-text">
          商品の説明
          <span class="indispensable">必須</span>
        </div>
        <%= f.text_area :explain, class:"items-text", id:"item-info", placeholder:"コーヒーの感想（必須 1,000文字まで）",rows:"7" ,maxlength:"1000" %>
      </div>
    </div>

    <div class="price-content">
      <div class="price-text">
        <span>価格</span>
        <span class="indispensable">必須</span>
      </div>
        <span class="sell-yen">¥</span>
      <%= f.text_field :price, class:"price-input", id:"item-price", placeholder:"例） 1000" %>
    </div>

    



    <div class="sell-btn-contents">
      <%= f.submit "投稿する" ,class:"sell-btn" %>
      <%=link_to 'もどる', root_path, class:"back-btn" %>
    </div>

  </div>
  <% end %>

drinks_controller

class DrinksController < ApplicationController
  include SessionsHelper
  
  before_action :logged_in_user, only: [:index,:destroy]
  def index
    @user = current_user
    @drinks = Drink.all.order("created_at DESC")
  end

  def show
    @drink = Drink.find(params[:id])
    @user = @drink.user
  end

  def new
    @drink = DrinkTag.new
  end

  def create
    @drink = DrinkTag.new(drink_params)
    if @drink.save
      redirect_to drinks_path
    else
      render 'new'
    end
  end

  def destroy
    Drink.find(params[:id]).destroy
    redirect_to root_path
  end

  # GET search
  def search
    return nil if params[:keyword] == ""
    # 何も入れず検索したらエラーが起こるので、
    # 明示的にnilを返す
    tag = Tag.where(['tag_name LIKE ?',"%#{params[:keyword]}%"])
    # 第二引数が第一引数の?に入っていく,検索した
    # タグがデータベースにあれば返す
    render json:{ keyword: tag}
    # const tagName = XHR.response.keyword
    # の.keywordが使えてる

  end


  private
  def drink_params
    params.require(:drink_tag).permit(:name,:price,:explain,:image,:tag_name).merge(user_id: current_user.id)
  end
end

drink_tag.rb

class DrinkTag

  include ActiveModel::Model

  attr_accessor :name,:price,:explain,:image,:user_id,:tag_name

  with_options presence: true do
    validates :explain
    validates :price
    validates :name
    validates :image 
    validates :tag_name
    # tagとdrinkでnameってカラム名かかってるけど
    # どっちも今のところpresense: trueでいいけど
    # タグ付けしなくても投稿できるけど
    # drinkの名前が必要ってなったらどうする
    # drink[:name]とかで、drinkのnameにだけ
    # バリデーション かけることは可能？
  end

  def save
    drink = Drink.create(name: name,price: price,explain: explain,image: image,user_id: user_id)
    tag = Tag.where(tag_name: tag_name).first_or_initialize
    # .first_or_initializeはwhereで検索した条件の
    # レコードがあれば、そのレコードのインスタンスを返し、
    # なければ新しくインスタンスを作るメソッドです。

    # tag.rbでtag_nameで一意性の制約をしていて
    # 被った時に弾いちゃうから.first_or_initialize
    # が必要

    tag.save
    DrinkTagRelation.create(drink_id: drink.id, tag_id: tag.id)
  end

end

shared/error_messages

<% if object.errors.any? %>
  <div id="error_explanation">
    <div class="alert alert-danger">
      The form contains <%= pluralize(object.errors.count, "error") %>.
    </div>
    <ul>
    <% object.errors.full_messages.each do |msg| %>
      <li><%= msg %></li>
    <% end %>
    </ul>
  </div>
<% end %>

tag.js

if (location.pathname.match("drinks/new")){
 
  document.addEventListener("DOMContentLoaded",()=>{

    const inputElement = document.getElementById("drink_tag_tag_name")

    inputElement.addEventListener("keyup",()=>{

      const keyword = document.getElementById("drink_tag_tag_name").value;

      const XHR = new XMLHttpRequest();

      XHR.open("GET",`search/?keyword=${keyword}`,true);

      XHR.responseType = "json";

      XHR.send();

      XHR.onload = () => {


        const searchResult = document.getElementById("search-result");
        // 1.タグを表示させる場所である,search-resultを取得
        searchResult.innerHTML = "";

        if (XHR.response){
          // イベントに指定したkeyupは、バックスペースキー
          // などの押しても文字入力されないキーでも発火してしまう

          // 存在しないものをtagNameに定義するとエラーが起こる

          // レスポンスにデータがある場合のみタグを表示させる処理を行おう

          const tagName = XHR.response.keyword;

          tagName.forEach((tag) => {

          const childElement = document.createElement("div");
  
          childElement.setAttribute("class", "child");
          childElement.setAttribute("id", tag.id);

          childElement.innerHTML = tag.tag_name;

          const clickElement = document.getElementById(tag.id);

          clickElement.addEventListener("click",()=>{

            document.getElementById("drink_tag_tag_name").value = clickElement.textContent;
            
          });
          });
        };
      };
    });
  });
};

試したこと

rails cで

    19: def create
    20:   @drink = DrinkTag.new(drink_params)
 => 21:   binding.pry
    22:   if @drink.save
    23:     redirect_to drinks_path
    24:   else
    25:     render 'new'
    26:   end
    27: end

[1] pry(#<DrinksController>)> @drink
=> #<DrinkTag:0x00007fb58c414080
 @explain="",
 @name="",
 @price="",
 @tag_name="",
 @user_id=6>
[2] pry(#<DrinksController>)> @drink.save
  CACHE User Load (0.0ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 6 LIMIT 1  [["id", 6], ["LIMIT", 1]]
  ↳ app/models/drink_tag.rb:22:in `save'
  Tag Load (0.2ms)  SELECT `tags`.* FROM `tags` WHERE `tags`.`tag_name` = '' ORDER BY `tags`.`id` ASC LIMIT 1
  ↳ app/models/drink_tag.rb:23:in `save'
DEPRECATION WARNING: Uniqueness validator will no longer enforce case sensitive comparison in Rails 6.1. To continue case sensitive comparison on the :tag_name attribute in Tag model, pass `case_sensitive: true` option explicitly to the uniqueness validator. (called from save at /Users/soichirohara/coffee_passport/app/models/drink_tag.rb:32)
   (0.1ms)  BEGIN
  ↳ app/models/drink_tag.rb:32:in `save'
  Tag Exists? (0.2ms)  SELECT 1 AS one FROM `tags` WHERE `tags`.`tag_name` = BINARY '' AND `tags`.`id` != 8 LIMIT 1
  ↳ app/models/drink_tag.rb:32:in `save'
   (0.1ms)  COMMIT
  ↳ app/models/drink_tag.rb:32:in `save'
   (0.1ms)  BEGIN
  ↳ app/models/drink_tag.rb:33:in `save'
  Tag Load (0.2ms)  SELECT `tags`.* FROM `tags` WHERE `tags`.`id` = 8 LIMIT 1
  ↳ app/models/drink_tag.rb:33:in `save'
   (0.1ms)  ROLLBACK
  ↳ app/models/drink_tag.rb:33:in `save'
=> #<DrinkTagRelation:0x00007fb56d0297c8

と、saveメソッドが起動してしまうのが確認できました

Comments

[m.ts10806]

どういうバリデーションをしているつもりなのか、要件を説明してください。

[okuramasafumi]

`save`メソッドの先頭に`validate`メソッドの呼び出しを書けば動作はするかもしれません。

しかし、全体的に不明点が多いです。
例えば、「saveメソッドが起動してしまう」というのは意味を成していません、起動後の結果が重要なので。
「うまくいきません」ではなんの説明にもなりませんので、「何をしたくて何を期待していて何が期待と違うのか」をできるだけ明瞭に記述するように努力してください。

[divclass123]

バリデーションが掛かってないのを掛かるようにしたいです。

フォームに入力しなくても投稿ボタンを押したら、保存されてしまいます。

投稿ボタンを押した時に、しっかりバリデーションがかかって、エラーハンドリングがされるって感じの事を期待してますが、そうはなりません。

すいません、saveメソッドの先頭にvalidateというのは具体的にどう言う基準でしょうか？

[divclass123]

saveメソッドの起動後は恐らく、binding.pryの実行結果にあると思うのですが、不十分でしたでしょうか？

Answer 1

drinks_controller

で　

  def create
    @drink = DrinkTag.new(drink_params)
    if @drink.save
      redirect_to drinks_path
    else
      render 'new'
    end
  end

を

  def create
    @drink = DrinkTag.new(drink_params)
    if @drink.valid?
      @drink.save
      redirect_to drinks_path
    else
      render 'new'
    end
  end

と記述すべきだった