モデルのメソッドに関して

メソッドの内に書く変数として、属性値なら、name、関連先が絡んでいるなら、self.mailsと
明示したほうがいい気がする

同様の理由で、メソッド名事態も、関連なら@user.get_diariesとかにする方がいいかもしれない。
@user.diariesだと、直接関連しているように見えたり、Railsによる機能のように見えて、
混乱してしまう可能性がある。