index

coupons_controller.rb

  def index 
    if params(:state) == used
      @coupons = Coupon.all(params[:state])
    else
      @coupons = Coupon.all
    end
    
    respond_to do |format|
      format.html # index.html.erb
      format.json { render json: @coupons }
    end
  end

def index
if params.has_key?(:state) && params[:state] == "used"
@coupons = Coupon.where(:state => params[:state])
else
@coupons = Coupon.all
end

respond_to do |format|
  format.html # index.html.erb
  format.json { render json: @coupons }
end

end

  def index 
    if params[:state] == "used"
      @coupons = Coupon.where(:state => params[:state])
    else
      @coupons = Coupon.all
    end

    respond_to do |format|
      format.html # index.html.erb
      format.json { render json: @coupons }
    end
  end

You don't actually need the has_key? check in this case

lgtm