if文が作動せず常にelseの処理が実行されてしまいます

Question

現在、「勤務アプリ」を作成しており、画像のようなuserが持つshowページにて「出勤・退勤」が操作できる動きを作成しております。実際の日付にボタンを表示し、
出勤を押すと押した時刻が「出社の時と分」に表記され、退勤ボタンに切り替わるという内容です。
出勤時間を(started_at)退勤時間を(finished_at)と定義しました。

本来であれば、ボタンを押すとupdateアクションの内容が評価されるはずですが、if文のtureの条件を満たしているのに
elseのエラー文が表記されてしまいます。beybugも入れて調べてみましたが、解決策がわかりません。

お手数おかけしますが、解決策を教えていただけると幸いです。
宜しくお願い致します。

＊users_showの画面＊

＊user_show_html.erb＊

<tr>
         <td><%= l(day.worked_on, format: :short) %></td>
          <td class="<%= css_class %>"><%= $days_of_the_week[day.worked_on.wday] %></td>
          <td><%= l(day.started_at,format: :hour) if day.started_at.present? %></td>
           <td><%= l(day.started_at,format: :min) if day.started_at.present? %></td>
          <td> <!--繰り返し処理で表示している時間と本日が同じかつ出勤時間が空だったら、出勤ボタンを表示-->
            <% if btn_text = attendance_state(day) %>
             <%= link_to "#{btn_text}登録", user_attendance_path(@user, day), method: :patch, class: "btn btn-primary btn-attendance" %>
            <% end %>
          </td>
          <td><%= l(day.finished_at, format: :hour) if day.finished_at.present? %></td>
          <td><%= l(day.finished_at, format: :min) if day.finished_at.present? %></td>
          <td></td>
          <td>
            <% if day.started_at.present? && day.finished_at.present? %>
            <%= working_times(day.started_at, day.finished_at) %>
            <% end %>
          </td>
          <td><%= day.note %></td>
      </tr>
      <% end %>
      </tr>

＊controller＊

*ボタンを押すとこちらのupdateアクションが評価されますが、
elseの評価しかされません。

class AttendancesController < ApplicationController
 
  before_action :set_user, only:[:edit_one_month, :update_one_month]
  before_action :logged_in_user, only: [:update, :edit_one_month]
  before_action :set_one_month, only: :edit_one_month
  
  UPDATE_ERROR_MSG = "勤怠登録に失敗しました。やり直してください。"
  
  
  def update
    @user = User.find(params[:user_id])
    @attendance = Attendance.find(params[:id])
    # 出勤時間が未登録であることを判定します。
    if @attendance.started_at.nil?
      if @attendance.update_attributes(started_at: Time.current.change(sec: 0))
        flash[:info] = "おはようございます!"
      else
        flash[:danger] = UPDATE_ERROR_MSG
      end  
    elsif @attendance.finished_at.nil?
      if @attendance.update_attributes(finished_at: Time.current.change(sec: 0))
        flash[:info] = "お疲れ様でした。"
      else
        flash[:danger] = UPDATE_ERROR_MSG
      end
    end  
    redirect_to @user
  end  

  def edit_one_month
  end
end  

＊スキーマです＊

# This file is auto-generated from the current state of the database. Instead
# of editing this file, please use the migrations feature of Active Record to
# incrementally modify your database, and then regenerate this schema definition.
#
# Note that this schema.rb definition is the authoritative source for your
# database schema. If you need to create the application database on another
# system, you should be using db:schema:load, not running all the migrations
# from scratch. The latter is a flawed and unsustainable approach (the more migrations
# you'll amass, the slower it'll run and the greater likelihood for issues).
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema.define(version: 20200803060313) do

  create_table "attendances", force: :cascade do |t|
    t.date "worked_on"
    t.datetime "started_at"
    t.datetime "finished_at"
    t.string "note"
    t.integer "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
![イメージ説明](09e48a586e80f934055712e2fcb9a49f.png)  end

  create_table "users", force: :cascade do |t|
    t.string "name"
    t.string "email"
    t.string "department"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.string "password_digest"
    t.string "remember_digest"
    t.datetime "basic_time", default: "2020-08-10 23:00:00"
    t.datetime "work_time", default: "2020-08-10 22:30:00"
    t.boolean "admin", default: false
    t.index ["email"], name: "index_users_on_email", unique: true
  end

end

＊byebug画面です＊

Comments

[maisumakun]

Railsのバージョンはいくつですか？

[RThe]

Rails 5.1.7です。
ボタンを作った段階では、正常に動いていました。
データを整理使用と、rails db:migrate:resetをし、rails:db:seedを行いました。
それが要因となっているのですかね・・・

Answer 1

if文のtureの条件を満たしているのにelseのエラー文が表記されてしまいます。

きちんと理解できていなければ恐縮なのですが・・・
byebugの結果を見るとelseが実行されて良いのではないでしょうか？

if @attendance.started_at.nil? # @attendanceがnil
  if @attendance.update_attributes(started_at: ....) # @attendanceがnilなのでelseが実行される
    flash[:info] = "おはようございます!"
  else
    flash[:danger] = UPDATE_ERROR_MSG
  end  

Comments

[RThe]

ご回答ありがとうございます。
そうなんですね！しかし、elseでない場合の条件がでないので、バリデーションなどで引っかかっているのでしょうかね・・・

[no1knows]

flashしたら、再度描画（render）させるか、リダイレクト（redirect_to）させないとメッセージは反映されないはずなのでお試しください。

[RThe]

承知いたしました！
検討してみます。ありがとうございました