Question

I am just getting started with Ruby on Rails and i am not sure how i should do the queries to database in order to get data, add data or edit data…

Is it better to find for example a user from the controller or its better to add the queries into the model?

currently, my user homepage controller looks like this with some simple functionality

class HomeController < ApplicationController

  #get current_user variable
  helper_method :current_user

  def index
    if user_signed_in?
        @user = User.find_by_id(current_user.id)
    else
        render_404
    end
  end
end

it simply checks if the user is logged in and finds the user…

Should i move the db calls to a model for best practice or using the above method is also fine?

Answer 1

The way you did it is fine in my opinion. The controller’s job is to set the data for the view and part of setting the data is fetching it from the DB.

For more complex DB query you could use scope which are located in the model. This way the controller does not hold too much logic of DB queries on the model.