2

Just wondering what the accepted convention is for Rails controller design. Currently, every controller in my app that I've written it set up to send a JSON response when necessary. Thing is, I only ever utilse HTML responses in my app. So is it a good idea to have them defined?

For example:

def show
  @dog = Dog.find(params[:id])

  respond_to do |format|
    format.html
    format.json { render json: @dog } # needed?
  end
end

It makes the controller code less readable (because more LOC) and it also means I have to think deeply about what a good JSON response should be when HTML is not being is used, so it decreases the speed of my controller development. This is especially true when you have conditional responses.

For example:

def create
  @dog = Dog.new(params[:dog])

  respond_to do |format|
    if @dog.save
      format.html { redirect_to @dog }
      format.json { render json: @dog, status: :created, location: @dog }
    else
      format.html { render action: "new" }
      format.json { render json: @dog.errors, status: :unprocessable_entity }
    end
  end
end

The only positive I can see is that it "future-proofs" the controllers (i.e. if I need JSON responses later on then they are already written).

But if I'm writing JSON responses just because then by that same logic I might as well write XML responses (i.e. format.xml) too...

1 Answer 1

3

as it is trivial to add json (or xml, etc) responses later on, I would recommend against implementing them before.
your reasons you give are valid:

  • it is less readable
  • you more LOC and thus more code to maintain, more potential bugs
  • you spend time writing code that you don't need

plus:

  • you risk exposing attributes that are not meant to be openly accessible

there is no RoR convention to have all actions respond to json.

1
  • 1
    I am new to Rails and I was also wondering about this. Why do something that I don't need now and may never need. YAGNI is the word that comes to mind.
    – Lee
    Apr 21, 2013 at 12:21

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.