[Rails]バリデーションをかけているのですが、効きません。理由と解決方法を知りたいです。

Question

前提・実現したいこと

Railsで簡単なイベント閲覧アプリを作成しています。
イベントを登録する際にバリデーションをかけて、以下のエラーが発生したので解決したいのですが、できません。

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

イベントを登録する際に、以下のバリデーションをかけています。

・イベント名(event_name)が存在すること
・開催地の郵便番号(postcode)が北海道であること
・開催地の住所が存在すること
・登録するイベントの開始日が今日以降であること
・登録するイベントの終了日が開始日以降であること

問題は、

・イベント名(event_name)が存在すること

というバリデーションは有効であるのに対して

・開催地の住所が存在すること

というバリデーションは無効であることです。

undefined method `[]' for nil:NilClass

このエラーメッセージは開催地の住所が空であるにも関わらず、緯度経度を取得しようとしているがゆえに起きるメッセージです。
「開催地の住所が存在すること」というバリデーションが無効という判断は、上記エラーより推測しました。

該当のソースコード

以下はモデル（event.rb）のコードです。

class Event < ApplicationRecord
  has_many :interests
  has_many :interested_users, through: :interests, source: :user
  has_many :likes
  has_many :liked_users, through: :likes, source: :user
  belongs_to :category

  with_options presence: true do
    validates :event_name
    validates :place
  end
  validates :postcode, format: {with: /\A[0-9]{5,6}\z/}
  validates :postcode, format: {without: /\A[1-3][0-9]{5}\z/}
  validate :not_before_today
  validate :not_before_start_on
  after_validation :geocode

  def not_before_today
    errors.add(:start_on, 'Please set today or after today') if start_on.nil? || start_on < Date.today
  end

  def not_before_start_on
    errors.add(:end_on, 'Please set after start day') if end_on.nil? || end_on < start_on
  end

  # ↓↓住所から緯度経度を取得してDBに保存するための記述↓↓
  private
  def geocode
    address = (place + place_building)
    uri = URI.escape("https://maps.googleapis.com/maps/api/geocode/json?address="+address.gsub(" ", "")+"&key=AIzaSyCuZz7CQo9VlX7NCAFAPiNevqd7NTWpXO8")
    res = HTTP.get(uri).to_s
    response = JSON.parse(res)
    self.latitude = response["results"][0]["geometry"]["location"]["lat"]
    self.longitude = response["results"][0]["geometry"]["location"]["lng"]
  end

end

関連するコントローラ(events_controller.rb)は以下のコードです。

class EventsController < ApplicationController
  before_action :authenticate_user!, only: [:show, :new, :create]

  def index
    @search = Event.ransack(params[:q]) #ransackメソッド推奨
    @events = @search.result.includes(:category).order(:start_on)
    @allevents = Event.where('start_on >= ?', Date.today).order(:start_on)
  end

  def new
    @event = Event.new
  end

  def create
    @event = Event.new(event_params)

    if @event.save
      redirect_to action: 'index'
    else
      render "new"
    end
  
  end

  def show
    @event = Event.find(params[:id])
    @interest = Interest.new
    @like = Like.new
  end

  private
  def event_params
    params.require(:event).permit(:event_name, :url, :start_on, :end_on, :category_id, :postcode, :place, :place_building, :latitude, :longitude)
  end
end

投稿フォームのビュー(new.html.erb)は以下の通りです

<div class="main-wrapper">

  <h3>イベント登録する</h3>

  <div class="form-wrapper">

    <%= form_for @event do |f| %>

      <%= render 'layouts/error_messages', model: f.object %>

      <div class="event_name">
        <%= f.label :event_name, "イベント名 (必須)" %>
        <%= f.text_field :event_name %>
      </div>

      <div class="postcode">
        <%= f.label :postcode, "郵便番号 (必須)" %>
        <%= f.text_field :postcode, size: "10", maxlength: "8", onKeyUp: "AjaxZip3.zip2addr(this,'','addr11','addr11');", placeholder: "ハイフンなし7桁" %>  <%# name属性のzip11は無くてもいい %>
      </div>

      <div class="place">
        <div class="place__address">
          <%= f.label :place, "住所" %>
          <%= f.text_field :place, name: "addr11", size: "60", class: "false-place", placeholder: "郵便番号から自動入力されます" %>
        </div>
        <div class="place__building">
          <%= f.label :place_building, "建物名等" %>
          <%= f.text_field :place_building, placeholder: "あったら助かります" %>
        </div>
      </div>

      <div class="url">
        <%= f.label :url, "イベントURL" %>
        <%= f.text_field :url, class:'form-control', placeholder: "例: https://hokkaido.com" %>
      </div>

      <div class="start_on">
        <div class="start_on__label">
          <%= f.label :start_on, "開始日" %>
        </div>
        <%= f.date_field :start_on, value: Time.now.strftime("%Y-%m-%d"), class: 'form-control' %>
      </div>

      <div class="end_on">
        <div class="end_on__label">
          <%= f.label :end_on, "終了日" %>
        </div>
        <%= f.date_field :end_on, value: Time.now.strftime("%Y-%m-%d"), class: 'form-control' %>
      </div>

      <div class="category clearfix">
        <div class="category__label">
          <%= f.label :category_id, "カテゴリー"%>
        </div>
        <%= f.select :category_id, [["グルメ",1], ["お祭り",2], ["花火",3], ["よさこい",4], ["その他",5]] %>
      </div>

      <%= f.submit '登録', class:'btn btn-primary add-btn' %>
    
    <% end %>

  </div>

</div>

投稿フォームのビューにかけているjs(new.js)は以下の通りです

$(function(){
  $('.false-place').keyup(function(){
    $('input[name="addr11"]').attr('name', 'event[place]');
  });
});

試したこと

①バリデーションの順番を変えましたが、効果はありませんでした。
「イベント名(event_name)が存在すること」はバリデーションが効いて「イベント名を入力してください」というエラーメッセージが表示されます。

②mysqlのplaceカラムのヌル許可にチェックを入れデフォルトを「NULL」にしましたが、効果はありませんでした。
画像はテーブルの構造です。

補足情報（FW/ツールのバージョンなど）

プログラミングを始めて1ヶ月半です。初めての質問でわかりづらい部分もあると思いますが、ご回答いただけると幸いです。
Rails 5.2.3

Answer 1

こんばんは。

該当のエラーは、def geocode .... end のブロックの中で発生しているので、コントローラーからの入力のバリデーションは通っているのでは、と思います。

それよりも、この処理の中で maps.googleapis.com にアクセスしてその結果をJSONに入れていますが、ここの値が想定どおりになっていないのでは？という気がします。
組み立てているURLが正しいか、そのURLでアクセスした結果のJSONが想定どおりのフォーマットになっているかを確認すると良いのでは。

デバッグやputsでプリントさせてみるといかがでしょう。

  def geocode
    # .... 省略
    puts uri

    res = HTTP.get(uri).to_s
    response = JSON.parse(res)

    puts response

    self.latitude = response["results"][0]["geometry"]["location"]["lat"]
    self.longitude = response["results"][0]["geometry"]["location"]["lng"]
  end

追記

with_options のブロックのところが効いてないのではと思います。
いったん、宣言部分と対応する end の部分をコメントアウトしてチェックするといいのでは。

  # with_options presence: true do
    validates :event_name
    validates :place
  # end

追記2

geocodeに入ってきた直後に、errorsの中身をチェックするといいかもしれません。

def geocode
    puts errors.messages
    address = (place + place_building)
    # 本来の’処理
end

Comments

[mball]

ご指摘ありがとうございます。

頂いたアドバイスを基に、フォーム上でpriceの値が空の時と空でない時で試しました。ターミナル上で確認したところ、puts uriの時点で

①price="北海道札幌市北区北二十二条西3-2-15"の場合
https://maps.googleapis.com/maps/api/geocode/json?address=%E5%8C%97%E6%B5%B7%E9%81%93%E6%9C%AD%E5%B9%8C%E5%B8%82%E5%8C%97%E5%8C%BA%E5%8C%97%E4%BA%8C%E5%8D%81%E4%BA%8C%E6%9D%A1%E8%A5%BF3-2-15&key=[APIのキー]
{"results"=>[{"address_components"=>[{"long_name"=>"Kita 22 Jonishi", "short_name"=>"Kita 22 Jonishi", "types"=>["political", "sublocality", "sublocality_level_2"]}, {"long_name"=>"Kita Ward", "short_name"=>"Kita Ward", "types"=>["political", "sublocality", "sublocality_level_1"]}, {"long_name"=>"Sapporo", "short_name"=>"Sapporo", "types"=>["locality", "political"]}, {"long_name"=>"Hokkaido", "short_name"=>"Hokkaido", "types"=>["administrative_area_level_1", "political"]}, {"long_name"=>"Japan", "short_name"=>"JP", "types"=>["country", "political"]}, {"long_name"=>"001-0022", "short_name"=>"001-0022", "types"=>["postal_code"]}], "formatted_address"=>"Kita 22 Jonishi, Kita Ward, Sapporo, Hokkaido 001-0022, Japan", "geometry"=>{"bounds"=>{"northeast"=>{"lat"=>43.08919299999999, "lng"=>141.3485081}, "southwest"=>{"lat"=>43.085417, "lng"=>141.3280919}}, "location"=>{"lat"=>43.0874237, "lng"=>141.3404892}, "location_type"=>"APPROXIMATE", "viewport"=>{"northeast"=>{"lat"=>43.08919299999999, "lng"=>141.3485081}, "southwest"=>{"lat"=>43.085417, "lng"=>141.3280919}}}, "place_id"=>"ChIJD8-1AuMoC18RrQYFY4MZqZI", "types"=>["political", "sublocality", "sublocality_level_2"]}], "status"=>"OK"}

②price=""の場合
https://maps.googleapis.com/maps/api/geocode/json?address=&key=[APIのキー]
{"error_message"=>"Invalid request. Missing the 'address', 'components', 'latlng' or 'place_id' parameter.", "results"=>[], "status"=>"INVALID_REQUEST"}

と出力されました。

考えてみたのですが、
price=""でも after_validation :geocodeが呼び出されているので、
with_options presence: true do
validates :event_name
validates :place
end
でplaceに対するバリデーションがうまくいっていないような気がして、、、

お手数おかけしますが、再度ご意見いただければ幸いです、、

[mball]

[訂正] {}のデータはputs responseによって出力されたものでした。

[suama]

こんにちは！失礼しました。

placeがブランクになってしまった上でGoogleのAPIへのリクエストが送信されている。
placeがブランクなら、GoogleのAPIへのリクエスト処理を出す前に、バリデーションエラーになってほしいのにそうならない、ということですね。

with_options のあとの条件式を指定を確認してみてはどうでしょう。
with_optionsの後ろは、if とか unless が来るのではと。

いったん、with_options のブロックに入れない形で、バリデーションをかけて同じリクエストを投げてみては？

単純に、こんなかんじで。

validates :place, presence: true

[mball]

再度アドバイスありがとうございます。

with_optionsのブロックに入れない形でバリデーションをかけました。
validates :event_name, presence: true
validates :place, presence: true

がしかし、またしても同じエラーが起きてしまいました。。。

with_optionsのブロックに入れても入れなくても
event_nameにはバリデーションがかかり「イベント名を入力してください」というエラーメッセージを吐き出すのですが、placeにだけバリデーションがかからず画像を載せている通りエラー画面へ遷移してしまうみたいです。バリデーションがかかっていない理由がどうしてもわかりません。。

何度も申し訳ございませんが、ご意見いただければ幸いです。

[suama]

フォームから’送信したときの、リクエストはどうなっていますか？
ログに出ていると思いますので。

[mball]

place=""(空の時)かつevent_name="よさこい"(空でない時)は
https://i.gyazo.com/de35deedc876738ed0089821c37a0941.png

place="北海道札幌市中央区南四条西6丁目"(空でない時)かつevent_name=""(空の時)は
https://i.gyazo.com/883fc0ee715a022bac829981876fbddc.png

のようにログが出ます。

[suama]

ありがとうございます！
ちなみに、place はテーブルに存在するカラムでしょうか？

[mball]

placeは存在します！
https://gyazo.com/baa125859a4ddc6b27d4d842e19196a6

[suama]

バリデーションはどちらの場合でもチェックされていると思います。

標準のバリデーションは、errorsに値が入るところまでで、
実際には「追加更新削除」、する直前に、errorsに値があればロールバックするという流れになります。

実際に、event_name、placeどちらをブランクにした場合でも、geocode のメソッドが呼ばれていますよね。

ここでGoogle APIにリクエストを出していますが、この箇所では event_nameは使っていませんよね。

placeが”” の時は、Goggle APIにリクエストを出しても想定した形のJSONが返ってこないので、undefined method `[]' for nil:NilClass が発生し、「saveが走る前」に例外でエラーになってしまいます。

このため、バリデーションエラーがあっても、View側に結果を返せずに50xで画面が赤くなる、と考えられます。

placeがある場合は、Google APIから正しくJSONが取得できて、メソッドが正常に処理されるので save に進みますが、結局ここでevent_nameが無いのでActiveRecordでRollbackが走ります。

このケースなら、コントローラとView側が処理できるので、画面にエラーを返すことができているのだと思います。

def geocode のあとに、 puts errors とか、puts errors.messages と差し込んでみると、多分バリデーションがかかっているかどうか確認できるのでは。

def geocode
puts errors.messages
address = (place + place_building)
.....
end

[mball]

ありがとうございます！！
確かにバリデーションはかかっていたのですが、after_validationでgeocodeメソッドを呼び出していたのでエラー画面に遷移してしまっていたようです！！

親身に答えてくださってありがとうございました！本当に助かりました！！

[suama]

よかったですね!
何往復かしてしまってすみませんが、自分でも書いてて「あ、そうか」とおもってしまいました。

Answer 2

原因がわかりました！

after_validation :geocode

この一行があったせいで、place=""(空の時)でもgeocode内のメソッドが呼び出されていました。place=""でも住所の緯度経度を取得しようとしたために出たエラー画面でした。

この一行を消し、event.rb(モデル)内のgeocodeメソッドを定義している前に記述されたprivateを消した後、event_controller.rbへ次の記述をしました。

def create
 @event = Event.new(event_params)
 if @event.place != ""
   @event.geocode
 end
 if @event.save
   @event.geocode
   redirect_to action: 'index'
 else
   render "new"
 end
end

after_validationの意味や、バリデーションについて理解を深められたエラーでした！
ご協力いただきありがとうございました！！

Comments

[suama]

よかったですね！
if @event.place != "" のところは、placeがあるかどうかだけのチェックなので、if event.valid? だと、その他のフィールドもふくめたバリデーションの状態を返してくれます。
よろしければ。

[mball]

なるほど！その記述も後で試します！
初めてteratailで投稿してドキドキしてましたが、またひとつ成長できて楽しいです。
今回は助けてくださってありがとうございました！

Answer 3

どれかのオブジェクトがnilになってるのでそのエラーが出てます
response
response["results"]
response["results"][0]
response["results"][0]["geometry"]
response["results"][0]["geometry"]["location"]

このどれかがnilなのでがんばって調べましょう

Comments

[mball]

responseの時点で
{"error_message"=>"Invalid request. Missing the 'address', 'components', 'latlng' or 'place_id' parameter.", "results"=>[], "status"=>"INVALID_REQUEST"}
でした。。