Hey guys, I am a student and I got the memo (without explanation) that
my following code can be much better. How would you improve my
controller/reviewname method? Thank you for your time!

class ReviewController < ApplicationController

 def index
    @reviews = Review.all.order(created_at: :desc)
 end

 def reviewname
    @review = Review.find(params[:id])
    if @review.update_attribute(:title, sanitize(params[:title]))
     format.json { render json: { status: 200 } }
    else
     format.json { render json: { status: 500 } }
    end
 end
end

-- 
Posted via http://www.ruby-forum.com/.

-- 
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to rubyonrails-talk+unsubscr...@googlegroups.com.
To post to this group, send email to rubyonrails-talk@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/rubyonrails-talk/46daa34efc4f826cca394d079bae9590%40ruby-forum.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to