質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.34%

  • Ruby

    8234questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails 5

    2193questions

Railsのモデルにバリデーションを記述しても効かず、困っています。。。

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 53

shohei0718

score 2

はじめての質問です!
お忙しいなか、恐れいりますが、宜しくお願い致します

※質問内容にクレジットカードと記載がありますが、今回のコードはサンプルで作成しているだけなので、
自身のPCのローカル環境でのみ動かしています。

開発環境

Rails 5.0.1
Ruby 2.3.1

前提・実現したいこと

バリデーションを効かせることで、入力フォームが空欄のときに
エラーメッセージが出るようにしたい。

発生している問題

modelにバリデーションの記載をしてもバリデーションが効かない

class PaymentInformation < ApplicationRecord
  belongs_to :user

  validates :card_number,     presence: true
  validates :valid_year,      presence: true
  validates :valid_month,     presence: true
  validates :cvc,             numericality: true

end
%section.payment_information-container
  .content-title
    %h2 クレジットカード情報入力
    .form-content
      .form#credit-information
        = form_for @payment_information,url: payment_informations_path, method: :post do |f|
          = f.label 'カード番号', class: 'label'
          %span 必須
          %br/
          = f.number_field :card_number,class: 'number',id: 'card_num'
          = f.label '有効期限', class: 'label'
          %span 必須
          .selectbox
            = f.select :valid_month,[*(1..12)],prompt:"選択してください",class: 'valid_month'
            %p 月

          .select-valid
            = f.select :valid_year,[*18..23],prompt:"選択してください",class: 'valid_month'
            %p 年

          = f.label 'セキュリティコード', class: 'label'
          %span 必須
          = f.text_field :cvc, placeholder:' カード背面4桁もしくは3桁の番号',class: 'number',id: 'security_num',minumum:3, maximum:4
          .submit-box
            =f.submit '追加する', class: 'submit'

試したこと

マイグレーションファイルのカラムにnull false制約を加えてみました。
validateをvalidatesにしてみました。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • maisumakun

    2018/12/06 16:54

    あくまでサンプルだと信じたいですが、カード番号を保存するにはPCI DSSのルールに則った仕組みの構築が必要ですし、セキュリティコードは保存が一切禁止されています。

    キャンセル

  • shohei0718

    2018/12/06 17:02

    質問内容に不備があり、申し訳ございません。今回のコードはサンプルで作成しているだけなので、自身のPCのローカル環境でのみ動かしています

    キャンセル

  • maisumakun

    2018/12/06 17:21

    「バリデーションが効かない」というのはどのような状況でしょうか。「そのまま保存されてしまう」「エラーにはなるけどエラーメッセージがわからない」など、不適当な値を入れるとどうなるのか教えていただけますでしょうか。

    キャンセル

  • shohei0718

    2018/12/06 17:35

    現在の状況ですが、フォームが空欄の状態でSubmitボタンを押してもブラウザの表示が全く変わりません。モデルにpresence:trueを記述しているので、何らかのエラーメッセージが出るはずなのですが、エラーメッセージも出ません。Submitした空欄の情報はDBには保存されていない状態です。

    キャンセル

回答 3

check解決した方法

0

下記の記述で解決することができました。
ありがとうございます!!

##Gemfile
gem 'rails-i18n'

##config/locales/ja.yamlを作成
ja:
  attributes:
    card_number:  カード番号
    valid_year :  有効期限の年
    valid_month:  有効期限の月
    cvc        :  セキュリティコード

##html.haml
-if @payment_informations.error.any?
    =render 'layouts/error_messages', model: f.object

 ##views/layout/_error_messages
 - if model.errors.any?
     .alert.alert-warning
     %ul
         - model.errors.full_messages.each do |message|
            %li = message

##controller

def create
    @payment_information = PaymentInformation.create(payment_information_params)

    if @payment_information.save
        redirect_to action: :index
    else
        render :new
    end
end

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

現在の状況ですが、フォームが空欄の状態でSubmitボタンを押してもブラウザの表示が全く変わりません。

おそらく、現状は「バリデーションエラーとなっているけれど、エラーメッセージを表示していないために反応しているのかも定かではない」状態になっているのではないかと思われます。

とりあえず、<%= @payment_information.errors.inspect %>とすれば、モデルがどのようなエラーメッセージを生成しているか確認できます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/12/06 19:59

    ありがとうございます。
    HTMLにご教授頂いた
    <%= @payment_information.errors.inspect %>を入れると、
    下記のようなエラー文が出てきました。
    #<ActiveModel::Errors:0x007fbffeafcb00 @base=#<PaymentInformation id: nil, card_number: nil, valid_year: nil, valid_month: nil, cvc: nil, created_at: nil, updated_at: nil, user_id: nil>, @messages={:カード番号=>[], :card_number=>[]}, @details={}>

    もう少し自力での解決を試みます!

    キャンセル

  • 2018/12/06 21:15 編集

    エラー文をビューで表示させるために下記記述をし、フォームが空欄の状態でSubmitボタンを押しても何もビューに表示されずに困っています。。。

    - if @payment_information.errors.any?
    %p エラーです。

    先程ご教授頂いたコードを記述して、#<ActiveModel::Errors:0x007fbffeafcb00 〜というエラー文が出たので、エラーは検知しているとは思うのですが、なぜエラー文が表示されないのでしょうか

    キャンセル

0

Controller 側の実装はどうなっているでしょうか?  
validateエラー時は render でフォームを描画する必要があります

もし redirect_to でフォーム画面に戻っていたら、 新しく読み込み直した @payment_information を使って描画しているため、エラー情報が消えます

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/12/07 20:35 編集

    ご回答頂きまして、ありがとうございます!
    ただ、下記の記述で問題を解決することができました。
    お知恵を頂いたにも関わらず、申し訳ございません。。。

    ```
    ##Gemfile
    gem 'rails-i18n'

    ##config/locales/ja.yamlを作成
    ja:
    attributes:
    card_number: カード番号
    valid_year : 有効期限の年
    valid_month: 有効期限の月
    cvc : セキュリティコード

    ##html.haml
    -if @payment_informations.error.any?
    =render 'layouts/error_messages', model: f.object

    ##views/layout/_error_messages
    - if model.errors.any?
    .alert.alert-warning
    %ul
    - model.errors.full_messages.each do |message|
    %li = message

    ##controller

    def create
    @payment_information = PaymentInformation.create(payment_information_params)

    if @payment_information.save
    redirect_to action: :index
    else
    render :new
    end
    end
    ```

    キャンセル

同じタグがついた質問を見る

  • Ruby

    8234questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails 5

    2193questions