【Rails5】find_byのnil挙動について

Question

###前提・実現したいこと
has_oneとhas_manyの関係を持つユーザ情報を
CSVファイルのアップロードでDBに反映させたいです。

■リレーション

#user.rb
has_one :section, primary_key: "section_id", foreign_key: 'id'
has_one :position, primary_key: "position_id", foreign_key: 'id'
has_one :division, primary_key: "division_id", foreign_key: 'id'
has_one :department, primary_key: "department_id", foreign_key: 'id'
has_one :authority, primary_key: "authority_id", foreign_key: 'id'

has_many :user_offices
has_many :offices, through: :user_offices

//インポートするCSVファイルのサンプルです。
id,section_id,department_id,division_id,position_id,office_ids
1,飲食事業,東京部署,新宿支店,店長,新宿支店；渋谷支店

###発生している問題・エラーメッセージ
■has_oneの関係
idに変換するのでfind_byで検索しています。

table['section_id']     = Section.find_by(name: table['section_id']).id
table['department_id']  = Department.find_by(name: table['department_id']).id
table['division_id']    = Division.find_by(name: table['division_id']).id
table['position_id']    = Position.find_by(name: table['position_id']).id

しかし、該当するnameがない場合
find_byの結果がnilとなり、そこで止まってしまいます。

undefined method `id' for nil:NilClass

■has_manyの関係
has_oneとあまり変わりませんが、配列を生成しています。
一応、動いてはいますが、他に良い方法がありましたら、助言をしていただきたいです。

if table['office_ids'].present?
    a = table['office_ids'].split('；')
    table['office_ids'] = []

    a.each do |b|
      table['office_ids'] << Office.find_by(name: b).id
    end
end

###補足情報(言語/FW/ツール等のバージョンなど)
・Rails (5.1.2)
・参考サイト

has_oneの情報量って少ないですね、、
何卒、よろしくお願いいたします。

Answer 1

Section なら Section の，name と id の対照表が欲しいわけですよね。
CSV の 1 行ごとに検索するのも無駄なので，

section_name_to_id = Section.pluck(:name, :id).to_h

というハッシュを作っておくといいのではないでしょうか。

Comments

[ssk]

scivola様

引き続き、ありがとうございます。
仰る通り、ハッシュを作成することで対応できました。

find_byで取得できなかった場合、nilが入るので
そのままnilが代入されると考えていました。

find_byで該当レコードがない場合はnilを返しでエラーになり
ハッシュで該当キーがない場合はnilを返しエラーにはならない

という理解でよろしいでしょうか？

[scivola]

```rb
Section.find_by(name: table['section_id']).id
```

の場合，当該レコードが存在しなければ `find_by` が `nil` を返し，`nil` の `id` を呼ぼうとして `NoMethodError` になります。

一方，ハッシュのデフォルト値（つまり存在しないキーに対する値）は，とくに指定しなければ `nil` です。

[ssk]

ありがとうございます。

>の場合，当該レコードが存在しなければ `find_by` が `nil` を返し，`nil` の `id` を呼ぼうとして >`NoMethodError` になります。

理解することができました。
ご説明いただき、ありがとうございます。