【新人注目】マジックナンバーは使わないで・・・。

マジックナンバーとは

実際に僕が書いたコード・・・

 prefectures = Prefecture.where(area_id: 3)
 cities = City.where(prefecture_id: 13)

こんなコード。
ようするに自分以外の人が見たら何の数字のことかわからない番号のこと。

自分の頭の中では,,,
area_idの3は関東
prefecture_idの13は東京都
という感じで当たり前のようにコードを書いていた。

変更後のコード

  kantou = Area.find_by(name: '関東')
  prefectures = Prefecture.where(area_id: kantou)
  tokyo = Prefecture.find_by(name: '東京都')
  cities = City.where(prefecture_id: tokyo)

番号を使うのではなく、誰が見てもすぐにわかる変数に格納した。