createとupdateで値が入らない&一覧表示ができない

Question

http://qiita.com/jacoyutorius/items/ea0673fe30cfe7cdac8f#%E6%9B%B4%E6%96%B0%E7%B7%A8
この記事を参考にindexページで作っていますがうまくいきません。
現在ののコードで実行すると、
NoMethodError in CallHists#createまたはNoMethodError in CallHists#update
undefined method `each' for nil:NilClass

Extracted source (around line #75):
75 <% @call_hists.each do |call_hist| %>
76  <p><%= call_hist.created_at %></p>
77  <p><%= call_hist.cmp_name %></p>
78  <p><%= call_hist.contact %></p>

となってしまいます。

call_hists_controller.rb

class CallHistsController < ApplicationController
before_action :set_call_hist, only: [:show, :edit, :update, :destroy]


  def create
    @call_hist = CallHist.new(call_hist_params)
    @call_hist.save
    respond_to do |format|
      if @call_hist.save
        format.html { redirect_to call_hists_path, notice: 'Item was successfully created.' }
        format.json { render action: 'index', status: :created, location: @call_hist }
      else
        format.html { render action: 'index' }
        format.json { render json: @call_hist.errors, status: :unprocessable_entity }
      end
    end
  end

  def index
    @call_hists = CallHist.all.order(created_at: :desc)
      if params[:id].present?
        set_call_hist
      else
        @call_hist = CallHist.new
      end
  end

  def update
    respond_to do |format|
      if @call_hist.update(call_hist_params)
        format.html { redirect_to request.referer, notice: 'Item was successfully updated.' }
        format.json { head :no_content }
      else
        format.html { render action: 'index' }
        format.json { render json: @call_hist.errors, status: :unprocessable_entity }
      end
    end
  end

  def destroy
    set_call_hist
    @call_hist.destroy
    respond_to do |format|
      format.html { redirect_to "/call_hists" }
      format.json { head :no_content }
    end
  end

  private

    def set_call_hist
      @call_hist = CallHist.find_by(id: params[:id])
    end

    def call_hist_params
      params.require(:call_hist).permit( :date, :time, :tel,
                                        :operator,
                                        :cmp_name, :call_id, :tanyu_id,
                                        :fig1, :fig2, :fig3, :fig4, :fig5,
                                        :csi_no, :memo, :csi_data,
                                        :tanyu_ahd,
                                        :cosmo_id, :contact)
    end

end

index.html.erb


<%= link_to "マイページ",call_hists_path %>
<div>
  <input type="button" value="履歴作成・編集" class="write_call_hist">
  <input type="button" value="閉じる" class="close_form">
</div>

<div>
  <%= session[:operator_id] %>
  <%= link_to("ログアウト", "/logout", {method: :post}) %>
</div>

<div class="form">
  <%= form_for (@call_hist) do |f| %>

  <div class="field">
    <%= f.label :cmp_name, "ユーザー" %>
    <%= f.text_field :cmp_name %>
  </div>

  <div class="field">
    <%= f.label :contact, "連絡者" %>
    <%= f.text_field :contact %>
  </div>

  <div class="field">
    <%= f.label :tel, "電話番号" %>
    <%= f.text_field :tel %>
  </div>

  <div class="field">
    <%= f.label :csi_no, "CSI NO." %>
    <%= f.text_field :csi_no %>
  </div>

  <div class="field">
    <%= f.label :fig4, "受電方法" %>
    <%= f.collection_select :fig4, CallHist.all, :id, :fig4 %>
  </div>

  <div class="field">
    <%= f.label :fig5, "転送先" %>
    <%= f.collection_select :fig5, CallHist.all, :id, :fig5 %>
  </div>

  <div class="field">
    <%= f.label :memo %>
    <%= f.text_area :memo %>
  </div>

  <div class="field">
    <%= f.label :tanyu_id, "リモート依頼" %> <%= f.check_box :tanyu_id %>
    <%= f.label :tanyu_ahd, "依頼先" %>
    <%= f.collection_select :tanyu_ahd, CallHist.all, :id, :tanyu_ahd %>
  </div>

  <div class="field">
    <%= f.label :csi_data %>
    <%= f.text_area :csi_data %>
  </div>

  <div class="field">
    <%= f.label :fig1, "1." %> <%= f.check_box :fig1 %>
    <%= f.label :fig1, "2." %> <%= f.check_box :fig2 %>
    <%= f.label :fig1, "3." %> <%= f.check_box :fig3 %>
  </div>

  <div class="btn">
    <%= f.submit %>
  </div>

  <% end %>
</div>


<% @call_hists.each do |call_hist| %>
  <p><%= call_hist.created_at %></p>
  <p><%= call_hist.cmp_name %></p>
  <p><%= call_hist.contact %></p>
  <p><%= call_hist.tel %></p>
  <p><%= link_to 'Edit', call_hists_path(id: call_hist.id) %></p>
  <p><%= link_to 'Destroy', call_hist_path(id: call_hist.id), method: :delete, data: { confirm: 'Are you sure?' } %></p>
<% end %>

ご教授いただけると幸いです。
よろしくお願いいたします。

Answer 1

createやupdateの時にバリデーションエラー等で更新に失敗する場合には、下記コードによりindexのテンプレートをレンダリングしようとしています。

format.html { render action: 'index' }

この時、index.html.erbでは@call_histsを参照していますが、createやupdateメソッドでは@call_histsに値を代入していないためnilとなっていてエラーになっています。

解決するためには、createとupdateのformat.html { render action: 'index' }の前にて、@call_histsにCallHist.allなど表示したいCallHistの一覧を設定しておけば良いかと思います。

Comments

[Reo23]

うまくいきました。誠にありがとうございます。