元祖若手のプログラミング奮闘記

元祖若手の奮闘記。主に注意されたことをメモがわりに使用するよ!

コントローラー、ヘルパー、ビューにモデル処理を描くんじゃない!

タイトルの通り今回これをやらかしてたので
今日はこれ

当たり前っちゃ当たり前なんだけど
コード書きまくってたら良くやるのもこれ

なんてったって実際そんな書き方してるコードが
あるんだから参考にしちゃうよねw

基本書くのはこちら

app/models/...

気を付けよう!
まず一例でこちら
ヘルパーにモデル処理書いてみたよ。

app/helpers/users_helper.rb

module UsersHelper
  def get_user_names_by_group(group)
    User.where(group: group).pluck(:name)
  end
end

これは突っ込みありまくりコードだけど
気を付けよう実際あったし。

app/controllers/users_controller.rb

  def create
    if request.post?
      @users = set_user_params
      User.transaction do
        @users.each do |user|
          new_user = User.new(name: user[:name])
          raise unless user.save 
        end
      end
    end
    @users = User.all
  end

まぁどんまい!