Railsで個体識別番号を自動採番したい

Question

前回の質問の回答を実装したいので助けて欲しいです。
目的はuser_idごとに”年、月”-”番号”というような個体番号を新規登録ごとにふりたいです。
現在の状況記載します。

class FamiliesController < ApplicationController
  before_action :authenticate_user!, except: [:index]
  def index
    @families =Family.all.order(created_at: "DESC")
  end

  def show
    @family = Family.find(params[:id])
  end

  def new
    @family = Family.new
  end

  def create
    @family = Family.new(family_params)
    @family.user_id = current_user.id
    if @family.save
      redirect_to family_path(@family), notice: "登録完了しました。"
    else
      render :new
    end
  end

  def edit
    @family = Family.find(params[:id])
    if @family.user != current_user
      redirect_to families_path, alert: "不正なアクセスです。"
    end
  end

  def update
    @family = Family.find(params[:id])
    if @family.update(family_params)
      redirect_to family_path(@family), notice: "更新に成功しました。"
    else
      render :edit
    end
  end

  def destroy
    family = Family.find(params[:id])
    family.destroy
    redirect_to families_path
  end
  private
  def family_params
    params.require(:family).permit(:category, :molph, :individualnumber, :image, :fathermolph, :mothermolph, :hatchdate, :sex, :weight, :feature)
  end
end

class CreateFamilies < ActiveRecord::Migration[5.2]
  def change
    create_table :families do |t|
      t.integer :user_id
      t.string :category
      t.string :molph
      t.string :individualnumber
      t.string :image_id
      t.string :fathermolph
      t.string :mothermolph
      t.date :hatchdate
      t.string :sex
      t.integer :weight
      t.text :feature

      t.timestamps
    end
  end
end

Rails.application.routes.draw do
  devise_for :users
  root to: "home#index"
  resources :users
  resources :families
end

<section class="hero is-success">
  <div class="hero-body">
    <div class="container">
      <h1 class="title">
        new family
      </h1>
    </div>
  </div>
</section>

 <% if @family.errors.any? %>
   <div class="notification is-danger">
     <h2><%= pluralize(@family.errors.count, "error") %> prohibited this object from being saved: not successfully</h2>
     <ul>
      <% @family.errors.full_messages.each do |message| %>
      <li><%= message %></li>
      <% end %>
    </ul>
  </div>
 <% end %>

<section class="section">
  <div class="container">
    <div class="columns is-centered">
      <div class="column is-6">
        <%= form_for @family do |f| %>
          <div class="field">
            <%= f.label :category, "カテゴリー", class: "label" %>
            <%= f.text_field :category, class: "input" %>
          </div>
          <div class="field">
            <%= f.label :molph, "モルフ", class: "label" %>
            <%= f.text_field :molph, class: "input" %>
          </div>
          <div class="field">
            <%= f.label :individualnumber, "個体番号", class: "label" %>
            <%= f.month_field :individualnumber, class: "input" %>
          </div>
          <div class="field">
            <%= f.label :image, "写真", class: "label" %>
            <%= f.attachment_field :image, class: "input" %>
          </div>
          <div class="field">
            <%= f.label :fathermolph, "父親モルフ", class: "label" %>
            <%= f.text_field :fathermolph, class: "input" %>
          </div>
          <div class="field">
            <%= f.label :mothermolph, "母親モルフ", class: "label" %>
            <%= f.text_field :mothermolph, class: "input" %>
          </div>
          <div class="field">
            <%= f.label :hatchdate, "ハッチデータ", class: "label" %>
            <%= f.date_field :hatchdate, class: "input" %>
          </div>
          <div class="field">
            <%= f.label :sex, "雌雄", class: "label" %>
            <%= f.select :sex, [["♂", "♂"], ["♀", "♀"],["不明","不明"]], include_blank: "選択して下さい"  %>
          </div>
          <div class="field">
            <%= f.label :weight, "体重", class: "label" %>
            <%= f.number_field :weight, class: "input"  %>
          </div>
          <div class="field">
            <%= f.label :feature, "特徴", class: "label" %>
            <%= f.text_area :feature, class: "textarea" %>
          </div>

            <%= f.submit "登録", class: "button is-success"  %>

        <% end %>
      </div>
    </div>
  </div>
</section>

下記のmew.htmlの個体番号を”年、月”-”番号”で自動採番もしくは、
年、月は選択で番号は自動採番できますでしょうか。

　　<div class="field">
       <%= f.label :individualnumber, "個体番号", class: "label" %>
       <%= f.month_field :individualnumber, class: "input" %>
    </div>

よろしくお願いします。

追記

def create
    @family = Family.new(family_params)
    @family.get_serial
    @family.user_id = current_user.id
    if @family.save
      redirect_to family_path(@family), notice: "登録完了しました。"
    else
      render :new
    end
  end

  def get_serial
    month = Date.parse(individualnumber+"/1").strftime("%Y年%m月-")
    number = where("individualnumber like ?"."#{month}%").
               order(individualnumber: :desc).
               first.individualnumber[/月-(\d+)/,1].succ
     self.individualnumber = month + number
  end

上記のように記載しました。
すると下記のエラーが出ています。

app/controllers/families_controller.rb:28: syntax error, unexpected tSTRING_BEG
app/controllers/families_controller.rb:28: syntax error, unexpected ')', expecting end

number = where("individualnumber like ?"."#{month}%").

です。

追記２

class Family < ApplicationRecord
  belongs_to :user
  attachment :image

  with_options presence: true do
    validates :category
    validates :molph
    validates :image
    validates :fathermolph
    validates :mothermolph
    validates :hatchdate
    validates :sex
    validates :weight
    validates :feature
  end

  def get_serial
    month = Date.parse(individualnumber+"/1").strftime("%y年%m月-")
    number = Family.where("individualnumber like ?","#{month}%").
               order(individualnumber: :desc).
               first.individualnumber[/月-(\d+)/,1].succ
     self.individualnumber = month + number
  end
end

  def create
    @family = Family.new(family_params)
    @family = get_serial
    @family.user_id = current_user.id
    if @family.save
      redirect_to family_path(@family), notice: "登録完了しました。"
    else
      render :new
    end
  end

上記ように変更しました。

となります。
methodが定義されていないのでしょうか。。。

追記③

何を変更していいのやら全くわからないです。
何が違ってますでしょうか。

class Family < ApplicationRecord
  belongs_to :user
  attachment :image

  with_options presence: true do
    validates :category
    validates :molph
    validates :image
    validates :fathermolph
    validates :mothermolph
    validates :hatchdate
    validates :sex
    validates :weight
    validates :feature
  end

  def get_serial
    month = Date.parse(individualnumber+"/1").strftime("%y年%m月-")
    number = Family.where("individualnumber like ?","#{month}%").
               order(individualnumber: :desc).
               first.individualnumber[/月-(\d+)/,1].succ
     self.individualnumber = month + number
  end
end

追記④

def get_serial
    month = Date.parse(individualnumber+"/1").strftime("%y年%m月-")
    number = Family.where("individualnumber like ?","#{month}%").
      latest = where("individualnumber like ?"."#{month}%").
      order(individualnumber: :desc).first
      number = latest ? latest.individualnumber[/月-(\d+)/,1].succ : 1
     self.individualnumber = month + number
  end

これでいいのですか？

以下のエラーになりました。

追記⑤

like ?"."#{month}%"がカンマの部分を直して上のエラーは無くなりました。
次はNo method errorで止まってしましました。

model family.rb

class Family < ApplicationRecord
  belongs_to :user
  attachment :image

  with_options presence: true do
    validates :category
    validates :molph
    validates :image
    validates :fathermolph
    validates :mothermolph
    validates :hatchdate
    validates :sex
    validates :weight
    validates :feature
  end

  def get_serial
    month = Date.parse(individualnumber+"/1").strftime("%y年%m月-")
    number = Family.where("individualnumber like ?","#{month}%").
      latest = where("individualnumber like ?","#{month}%").
      order(individualnumber: :desc).first
      number = latest ? latest.individualnumber[/月-(\d+)/,1].succ : 1
     self.individualnumber = month + number
  end
end

families.controller

class FamiliesController < ApplicationController

省略
  def create
    @family = Family.new(family_params)
    @family.get_serial
    @family.user_id = current_user.id
    if @family.save
      redirect_to family_path(@family), notice: "登録完了しました。"
    else
      render :new
    end
  end

  def edit
    @family = Family.find(params[:id])
    if @family.user != current_user
      redirect_to families_path, alert: "不正なアクセスです。"
    end
  end

  def update
    @family = Family.find(params[:id])
    if @family.update(family_params)
      redirect_to family_path(@family), notice: "更新に成功しました。"
    else
      render :edit
    end
  end

質問ばっかりすいません。

よろしくお願いします。

追記⑥

qiitaやteratailsで探したのですがgemfileの更新などが出てきました。
やっていいものかわからなかったので確認のため質問します。

Answer 1

individualnumber のDBにしまわれる値は "2020年8月-12" としたいが、登録時の入力は 2020/8 として、追い番は自動採番としたい

という意味と取って考えます。
個人で使うもののようですから入力のエラーチェックなどは省いちゃいます。

まず、"2020年8月-12"　ではなく "2020年08月-12"にさせてください。
入力は 2020/08、2020-8　の様に入れる、ということにします。
@family = Family.new(family_params)　のあと
@family.get_serial とでも名付けたmethodを呼びます
class Family の中で

def get_serial
  month = Date.parse(individualnumber+"/1").strftime("%Y年%m月-")
  number = where("individualnumber like ?"."#{month}%").
             order(individualnumber: :desc).
             first.individualnumber[/月-(\d+)/,1].succ
  self.individualnumber = month + number
end

first.individualnumber までで　"2020年08月-12"　が得られ、その後ろの[ ] で "12" が、そして　succ で "13"が得られます。
Date.parseで +"/1" してるのはエラーを起こしにくくするおまじない。

Comments

[ahiru3]

ありがとうございます。
馬鹿で申し訳ないのですが、
def create
@family = Family.new(family_params)
@family.get_serial
@family.user_id = current_user.id
if @family.save
redirect_to family_path(@family), notice: "登録完了しました。"
else
render :new
end
end

def get_serial
month = Date.parse(individualnumber+"/1").strftime("%Y年%m月-")
number = where("individualnumber like ?"."#{month}%").
order(individualnumber: :desc).
first.individualnumber[/月-(\d+)/,1].succ
self.individualnumber = month + number
end

上記で良いのでしょうか。
これだと、
app/controllers/families_controller.rb:28: syntax error, unexpected tSTRING_BEG
app/controllers/families_controller.rb:28: syntax error, unexpected ')', expecting end
とエラーになりました。

聞いてばかりですいません。

[winterboum]

codeはコメンtに書くとベタになって読みにくいので質問欄に追記でおねがいします。
で、２８行目ってどこでしょう

[ahiru3]

質問に追記しました。
すいません。

[winterboum]

like ?"."#{m　のが　カンマですピリオドになってます。
それと
def get_serial　は modelに書いてください。
ミス。　whereの前に Family.　をつけてください

[ahiru3]

調べたのですがどこの何が足りないのかが不明です。
追記で質問しております。
すいませんがお願いします。

[winterboum]

@family = get_serial
でなく
@family.get_serial
です

[ahiru3]

ありがとうございます。
どんどんエラー出てきて流のですが、
controllerにmethodが定義されていないということでしょうか。

[winterboum]

@family = get_serial　だと　controllerのget_serialメソッドを呼んでます。
それだとエラーになるので
@family.get_serial　にしていただいて、@familyのget_serialを実行します。

[ahiru3]

一つ前のコメントで@family = get.serialの指摘いただいたので、
@family.get.serialに変更しているのですが、他にも変更箇所はありますでしょうか。
追記３に変更を追加しておきます。

[winterboum]

ああ、そうかごめんなさい、その月の最初の子牛のときエラーになってしまてました
latest = where("individualnumber like ?"."#{month}%").
order(individualnumber: :desc).first
number = latest ? latest.individualnumber[/月-(\d+)/,1].succ : 1
にしてください

Nilにはindividualnumberがないというエラーなので、firstした結果がnilということです。

[ahiru3]

質問ばかりだとダメだと思って調べてチャレンジしたのですが、
結局ダメでした。
追記⑤の状態で止まってしまっています。
すいませんが助けて下さい。

よろしくお願いします。

[winterboum]

ごめんなさい、そこは私のミスで2020/08/13 08:05にコメントでいれてますように、
where を Family.where にしてください。

[ahiru3]

ありがとうございます。
self.individualnumber = month + numberでTypeErrorが発生してしまいました。
追記しますのでお願いします。

[winterboum]

month + number.to_s
ですね。
このくらいは自力で見つけてほしいな

[ahiru3]

すいませんでした。ありがとうございます。
sqliteのdb見ると年と月しか入っていなかったので、
ちょっと原因調べてみます。