Railsのモデル名をすべて変更した話 - SmartHR Tech Blog

SmartHRでは開発にRuby on Railsを広く採用しています。 今日は負債解消のために、開発しているサービスでRailsのモデル名をすべて変更した話を紹介します。 既存のモデル構造のつらみ 私達が開発しているサービスでは、モデルの親子構造が分かりやすいということで、モデルをネストした構造にしていました。 例えば、 User に紐づくプロフィール画像 User::ProfileImage は、 app/models/user/profile_image.rb に配置する具合です。 パッと見の構造が分かりやすいのですが、時が経つにつれて次のようなつらさが顕在化してきました。 Railsの規約(推奨ルールのようなもの)に則っていないので、関連定義が冗長になる テーブル名が長くなる。 外部キーや関連名が長くなる。 関連名と外部キー名が一致せず、カラムを呼び出したいときにDB定義を見ないと

[nyamadori]

ここまでシステマチックにやり切れるのはすごいし気持ちもわかるけど、ネームスペースのメリットを享受できなくなるのはしんどくないか。ネームスペースがあるからこそスコープが閉じて脳の容量を抑えられるはず。

[peller]

衝突せずに名前変更できるなら、もともと table_name_prefixを空指定にして、関連定義で名前空間無し& class_name指定しすれば良かった話じゃない？

[june29]

偉大だ〜〜〜 / 書き手の個人名を出さない方針なら「私は」より「私たちは」等の方が自然になりそう

[daichirata]

変えたいとみんなが一度は思うもののやり遂げるのはすごく困難な奴。偉業だ。

[leiqunni]

前も見たことある。前のはパーサを書いて名前変更してたな。ワイも文字列置換でやって実行してエラー出して。あけおめことよろ→あけまんことよろ

[YusukeIwaki]

これは頑張ったなぁ

[paulownia]

namespaceを削除って感じなのかな？名前被りはどう回避したんだろうか、そもそも被ってなかったとか？

[ducktoon]

ネームスペース名に問題があるんじゃ？ User モデルがあるのにネームスペース名を User にしちゃおかしくなるよ

[zakinco]

偉いなー

[tomoakinagahara]

RoRは簡単なものはすぐ作れるけど、複雑なものは苦手。レールに沿う分には苦労しないけど、レールから外れると途端に詰んでしまう。

[hdampty7]

なんか、血みどろになってるのに、勝利宣言しているようなブログもあるなか、システマチックにやり切ってるのはすげーな。

[irasally]

やり切ったのすごい

[yuuAn]

名前空間が使いづらいフレームワークはつらい。

[kompiro]

偉業だ

[daira4000]

「なお、このバグは社内勉強会で修正されました」 偉業

[nicht-sein]

うわぁ……この規模のリファクタリングはやると決めるのも大変だ

[gfx]

yaba

[udzura]

トレポ便利

[yamadadadada2]

お疲れ様でした。/名前重要。名前を間違っただけで仕事が倍になる。その時は良くても、未来の自分を苦しめるんや…

[bopperjp]

こういうことができる体力と技術力はうらやましい。。

[iga_k]

モデル名変更作業をシステマティックに実行。力こそパワー素晴らしい。

[seiunsky]

めちゃ良すぎる……!!