メインコンテンツまでスキップ

ruby review

配列

分解の仕方

制御構文

if else節がなければ後置ifに変更させる

unless 否定の部分 if !now みたいのがあればunlessへ切り替えるようにする。 ※unlessのelseは否定の否定はかえって理解しづらいのでやめる。

for Rubyでのスタイルガイドでforは原則使わないとなっている。

例外処理

  • rescue節がからではないかor出力しているだけか(console.logみたいな) rescue節を空にしてはいけない(発生した例外を無視する=握りつぶすと同一) 最低でも例外情報を出力し、例外の発生を確認できるようにする(そもそも処理すべき方法を持たないのであれば無理して処理せず、rubyがエラーを通知するに任せるべき)

オブジェクトアクセス

nilか確認してアクセスしているところは &.(Safe Navigation Operator)でオブジェクトへアクセスできる。

helperやutilsについて

ディレクトリorクラスでも

Javaの様に、クラスの外に直接関数を定義できない言語では、関数を定義するためのプレースホルダーとしてクラスが必要になることもある。 HelperやUtilityといった名前はその様な「静的メソッドしかないクラス」「インスタンス化する意味がないクラス」にだけ使う。

もしそのクラスをインスタンス化することに意味があるのなら、そのクラスのインスタンスは何らかのモノを表していることになる。

長くなりがちなmodelを綺麗に分割する

参考URL

rails review

参考URL