[Rails5]1対１のモデルについて

Question

現在、簡単なSNSアプリを作ろうと思っているのですが、モデルの取り扱いについてわからなかったため質問いたします。

ユーザー1人はUserモデルとMyintroモデルを持っております.

Userモデルはdevise認証のGemで作ったものです。
1人のユーザーは1つの自分の詳細モデルを持つ、というイメージで考えており、
Userモデル１つにつき、Myintroモデルを1つ持っているものになります。

ここで、ユーザーを一人作った後、自分の詳細モデルを作る際
Myintroコントローラーのnewアクションを行うのですが
その中で、Userが持っているMyintroが一つであればeditページへ飛ばす、としたいのです。

質問

ユーザーがMyintroを一つ以上持っているならという条件を
if current_user.Myintro.length >=1
というようなコードで書けたらいいのですが、何かいい方法はありませんでしょうか？

class MyintrosController < ApplicationController
  before_action :authenticate_user!
  def index

  end
  def show
  end
  def new
    # 現在のユーザーの紹介を作る
    @length = current_user.myintro.length
    # 現在のユーザーが詳細モデルをすでに持っていれば編集ページに遷移する
      redirect_to   edit_myintro_path(@myintro)
    end

    @myintro = current_user.build_myintro
  end

  def create
    @myintro = current_user.build_myintro(myintro_params)
    if @myintro.save
    redirect_to   edit_myintro_path(@myintro),notice:"プロフィール詳細を作成・保存しました"
    else
      redirect_to new_myintro_path,notice:"プロフィール詳細を作成できませんでした"
    end
  end
  def edit
  end

  private
  def myintro_params
    params.require(:myintro).permit(:sex, :love_target)
  end
end

class Myintro < ApplicationRecord
  belongs_to :user

  #必須項目
  validates :sex, presence: true
  
end

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :trackable, :validatable
  has_one :myintro,dependent: :destroy
end

Answer 1

解決おめでとうございます。表記を正しく直しました。こちらが勉強になりました（笑）。

こんにちは。詳しくはないのですが、似た問題を解決しました。書籍を参考にしました。
devise のテーブル（Userモデルに該当）
ユーザ情報入力用のテーブル（Myintroモデルに該当）----user_id 列で、user.id を参照する。

書籍によって、devise 側のテーブル、変数を user としている場合と、ユーザの情報入力用のテーブル、変数を user としている場合があり、ややこしいです。プログラムの読み違えが起きそうです。

問題は、二つ。（以下）
問題は、切り分けると、ストレスが減ります（笑）。

(1)--ユーザの情報入力用のテーブルに、まだ、レコードがない場合---を、どのように if 文に書くか？

(2)--devise 側のテーブルから id を取得して、ここで、ユーザ情報入力用のテーブルに保存しないと、参照できない。
@ユーザ情報入力用インスタンス.new
@ユーザ情報入力用インスタンス.save
どちらも、ここで行うことになる。

書かれている if current_user.Myintro.length >=1 と同じだと思いますが、私が動作確認できた方法は以下です。

# ユーザ情報入力のテーブル---deviseモデル名_id 列--に、
# 今、ログインしている人の devise の id があるか確認する。
# where を使うので、配列で受け取り。
# @はいれつ[0]---最初の要素

@はいれつ = ユーザ情報入力モデル.where('deviseモデル名_id == ?', current_deviseモデル名.id)

if @はいれつ[0] == nil then
  # まだ、ユーザ情報入力モデルのレコードがない。

  # ここで、新規入力を行う準備。new,create の url 使わない。
  @ひと = ユーザ情報入力.new 

  # 参照される、devise のテーブルの id を代入。
  @ひと.deviseモデル名_id = current_deviseモデル名.id

  # ここで、ユーザ情報入力モデルのテーブルに保存。
  @ひと.save

  # 今、保存したレコードを where の検索。
  @はいれつ = ユーザの情報入力モデル.where('deviseモデル名_id == ?', current_deviseモデル名.id)

else
  # すでにユーザの情報入力テーブルにレコードがある。
  # 過去にログインしたとき保存された。
end

# 配列の最初の１個を代入
@ユーザの情報入力用インスタンス = @はいれつ[0]

コントローラ、if 文の中で、ユーザ情報入力テーブルに保存。つまり、ユーザ情報入力テーブルへの入力は、edit, update で行う。new, create ではない。
ややこしいです。理解するのに、かなり時間がかかりました。
今、読み返すと、自分で、忘れている部分があります。（あー）

Comments

[marokatsu]

nekoyama7さん
返信ありがとうございます！

where文はまだ使ったことがなかったのですが、
newアクションで
Myintroモデルを参照し、nilであれば作成する、という流れでしょうか。。。
MyintroコントローラーのnewアクションでdeviseのUserに紐付いたMyintroを作成し、
updateアクションで作ったMyintroを更新していく、ということだと理解しました。
ありがとうございました！

[nekoyama7]

pekapekatailさんの書いていただいた、before_action を使う方法（まだ、じっくりコードを読んでませんが）で、new アクションを使うのが、流れが自然で良いと思います。参考にした本では、ユーザ情報入力のテーブル（Myintroモデル、テーブル）の１件のレコード作成を、index アクション（本来、複数のレコードを扱う）で行うという不自然なものでした。
save で保存した瞬間から、新規レコードではなくなるので、edit, update を使うというのも不自然。（new, create ではなく。）

[marokatsu]

nekoyama7さん
ありがとうございます。
myintroのnewアクションでidを引数で受け取っているわけではなかったため

if User.find(params[:id]).myintro.all.length == 1
この:idでエラーが発生してしまい、色々試した結果下記の方法で何とか動きました。

if current_user.myintro != nil

1対1のモデルの扱いがまだはっきりとわかっていないため、また質問するかもしれません。
お時間がありましたらまたよろしくお願い致します。

[nekoyama7]

お疲れ様です。プログラムで、大きなトラブルを減らせるコツとして、私が心がけているのは----
シンプルで、小さいサンプルを先に書く。---です。
いったん、devise のモデル、テーブルからはなれて、より、シンプルな物を考えた方が、急がば回れ---で、早いかも。

[marokatsu]

ありがとうございます。
自分が作っているシステムをより簡単に。。。
わかりました。小さな機能から作って勉強します！

Answer 2

ご質問拝見しました！
routes.rbのファイルも見せて頂けますでしょうか？

基本的には以下のようにnewアクションの前にbefore_actionで処理を行うのが良いのではないかと思います！

before_action :myintro_counter, only: [:new]

~~~
private
def myintro_counter
  if User.find(params[:id]).myintro.all.length == 1
    redirect_to honyarara #UserがMyintroをちょうど一つ持っている時に飛ばしたい場所を指定

  else
    #そうでない場合の処理
  end
end

ちなみに、ご質問に「UserがMyintroを一つ以上持っているならという条件を」と書かれていましたが、has_oneを使ってアソシエーションされているので、もしご質問に書かれているように1:nの対応関係にするならhas_manyにする必要があります！

私も初心者ですが以上ご回答です！

Comments

[marokatsu]

pekapekatailさん
返信ありがとうございます！
コードも拝見しました。ひとつのユーザーに紐付いた自己紹介のモデルの数を取得する、ということですね。
気づきませんでした！
Userは自分のプロフィールであるMyintroを一つだけ持つため1対1のモデルです。
ありがとうございました!

routes.rb

Rails.application.routes.draw do
devise_for :users
#get 'pages/index'
root 'pages#index'
resources :users, only: [:show]
resources :myintros
get 'myintros/:id/life' => 'myintros#life', as: 'manage_myintro_life'
get 'myintros/:id/job' => 'myintros#job', as: 'manage_myintro_job'
get 'myintros/:id/etc' => 'myintros#etc', as: 'manage_myintro_etc'
# For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
end