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

Question

###現状のコード

#コントローラー
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>

Answer 1

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

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を設定するという手もありますね。

Comments

[ssk]

ありがとうございます。
早速試してみました。しかし
section_idにはNULLが入るSQL文を作成されているようです。。

Mysql2::Error: Column 'section_id' cannot be null: UPDATE `coordinators` SET `section_id` = NULL, `updated_at` = '2017-05-08 09:19:18' WHERE `coordinators`.`id` = 1

追記
DBにdefault_value-1は設定済みです。しかしNullを許可しないようにしているため、プログラミング側で””の場合は-1を入れるようにしなければいけません。

[moke]

, if: :new_record?を外してみてください、上のコードだと更新の時は-1を入れません
もし、それでも問題があるなら、そうするとあと考えられるのは。
strong_paramaterぐらいですかね、section_idを許可していますか?
とりあえず、CoordinatorControllerを全部見せてください

[ssk]

コントローラーとモデルのコード全文を追記致しました。

[moke]

, if: :new_record?を外してみてください、上のコードだと更新の時は-1を入れません

[ssk]

, if: :new_record?を外して、after_initialize :set_defaultとしたのですが、更新時-1ははいりませんでした。

[moke]

うーん、問題ないですね。
after_initialize :set_default
を
before_save :set_default
にすれば、とりあえず表面上は大丈夫でしょうが...
なぜ、def new ではなくdef registにしているかもわかりませんが、
regist.html.erbとその中でrenderしているviewに本質的な問題があるんでしょうね
サーバーで作る、-1を入れる→ブラウザで描画して変更後Post→戻ってきたデータから-1が消えている
という状態なので

[ssk]

情報を追記致しました。

なぜ、def new ではなくdef registにしているか
↑/newではなく/registで利用したいからです。

[ssk]

before_save :set_default
↑こちらでは-1を入れることができました。

[moke]

:include_blank => true
これですね
not_nullと:include_blank => trueは共存させない方がいいですね。
active_recordが""をnilに変換しますので
あとこう書くと軽くて見通しがいいかもしれません
<%= f.select( :section_id, Section.pluck(:name,:id)) %>

[ssk]

ありがとうございます。最後に、、、

<%= f.select( :section_id, Section.pluck(:name, :id)) %>
とすると未選択がなくなってしまいます。include_blank => trueで未選択を実現していたのですが、どのように記述すれば良いでしょうか。。

[moke]

section_idのnot_nullを消します。
DBのnullは未定、不能、未確認を表す値です。
または
Sectionの中に未選択を追加してそのIDを
after_initializeで設定して
before_saveで未選択だったら-1を設定する
ですね、nullを恐れる気持ちもわかりますが
Railsはそこも吸収してくれます。

[ssk]

Sectionの中に未選択を追加してそのIDをafter_initializeで設定して
before_saveで未選択だったら-1を設定
↑度々、申し訳ございません。。こちらはどのように記述すると実現できますか？かれこれ2日ほど悩んでいまして、、

section_idのnot_nullを消すのも視野にいれます。

[moke]

after_initialize :set_not_selected
before_save :set_default　
private
def set_not_selected
self.section_id ||= 0#未選択のid(仮)
end
def set_default
self.section_id = -1 if self.section_id==0
end
まあ、ほんとそのまんまです。

[ssk]

ありがとうございます＞＜

:include_blank => true
で空の要素を追加して、ご提示いただいたコードを貼り付けたのですが、SQL文はsection_id` = NULLとなってしまいました。

[ssk]

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

before_save :set_default

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

[moke]

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

[ssk]

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