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

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

ただいまの
回答率

89.52%

Railsの三項演算子の書き方について

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,226

ssk

score 286

現状のコード

#コントローラー
params[:coordinator][:section_id] = params[:coordinator][:section_id].blank? ? -1 : params[:coordinator][:section_id]

やりたいこと

params[:coordinator][:section_id]の値は""です。
""の場合は.blank?で判定して-1を代入するようにしています。

http://qiita.com/quattro_4/items/88578a2044c423cab3f0
の方法でシンプルに書けるかと思ったのですが、nilまたは空白ではないため、うまくいきません。

シンプルに書くにはどのように記述すれば良いでしょうか?
何卒、よろしくお願い致します。

追記

coordinators_controller.rb

class CoordinatorsController < ApplicationController
  before_action :set_coordinator, only: [:show, :edit, :update, :destroy]

  # GET /coordinators
  # GET /coordinators.json
  def index
    @coordinators = Coordinator.all
  end

  # GET /coordinators/1
  # GET /coordinators/1.json
  def show
  end

  # GET /coordinators/regist
  def regist
    @coordinator = Coordinator.new
  end

  # GET /coordinators/1/edit
  def edit
  end

  # POST /coordinators
  # POST /coordinators.json
  def create
    @coordinator = Coordinator.new(coordinator_params)
    respond_to do |format|
      if @coordinator.save
        format.html { redirect_to @coordinator, notice: 'コーディネーター登録完了' }
        #format.json { render :show, status: :created, location: @coordinator }
      else
        format.html { render :regist }
        #format.json { render json: @coordinator.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /coordinators/1
  # PATCH/PUT /coordinators/1.json
  def update
      #params[:coordinator][:section_id] = params[:coordinator][:section_id].blank? ? -1 : params[:coordinator][:section_id]
    respond_to do |format|
      if @coordinator.update(coordinator_params)
        format.html { redirect_to @coordinator, notice: 'Coordinator was successfully updated.' }
        format.json { render :show, status: :ok, location: @coordinator }
      else
        format.html { render :edit }
        format.json { render json: @coordinator.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /coordinators/1
  # DELETE /coordinators/1.json
  def destroy
    @coordinator.destroy
    respond_to do |format|
      format.html { redirect_to coordinators_url, notice: 'Coordinator was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_coordinator
      @coordinator = Coordinator.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def coordinator_params
      params.require(:coordinator).permit(:name, :name2, :coordinator_id, :password, :mail, :section_id, :department_id, :division_id, :position_id, :profimg, :authority_id, :status, :token, :token_limit, {:office_ids => []})
    end
end

coordinator.rb

class Coordinator < ApplicationRecord
    after_initialize :set_default
    private
    def set_default
        self.section_id ||= -1
    end
end

さらに追記

regist.html.erb

<h1>New Coordinator</h1>

<%= render 'form', coordinator: @coordinator %>

<%= link_to 'Back', coordinators_path %>

_form.html.erb(一部:section_idのみ)

<div class="field">      
       <%= f.label :section, '事業課' %>
    <%= f.collection_select( :section_id, Section.all, :id, :name, :include_blank => true) %>
  </div>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

これは、こう書くしかありません
あえて書くなら

params[:coordinator][:section_id]=-1 if params[:coordinator][:section_id].blank?


こうでしょうか、しかし

前の質問に答えてあげれれば,
よかったのですが。

 before_save :set_default 
private
  def set_default
    self.section_id = -1 if self.section_id==nil
  end


このように新たなレコードがイニシャライズ(初期化)された時に
特に指定がなければ、-1が代入されるというように
モデル層に記述すれば、sskさんの悩みは全て解決されると思います。

てっきりcontroller層に書いたというから

def new
 Coordinator.new(section_id: -1)
end


みたいにしていると思ったのですが。
違ったみたいですね。

あと、DBにdefault_valueを設定するという手もありますね。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/08 19:30

    以下のコードで:include_blank => trueを指定してもエラーが出なくなりました。問題はなさそうですか?

    before_save :set_default

    private
    def set_default
    self.section_id = -1 if self.section_id == nil
    end

    キャンセル

  • 2017/05/08 19:48 編集

    :include_blank => trueを使わないことを考えていたのですが
    そのコードでいいですね。
    回答も変更しておきます。

    キャンセル

  • 2017/05/08 19:49

    ありがとうございます。
    本当に助かりました。

    キャンセル

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

  • ただいまの回答率 89.52%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる