Rails 4，简单形式：自定义网址无法按ID正确找到用户

Question

我有一个基本的问题来补充我拥有的simple_form

这是表格

  <%= simple_form_for(current_user, :url =>  nation_user_path(current_user), :method => :post) do |f| %>
        <%= f.input :selected_country, collection: @user_country_array, label: "Select a country to view data on (only countries represented are listed)", class: 'form-control' %>
        <%= f.submit "Submit", class: "btn btn-primary" %></h3>
  <% end %>

routes.rb

resources :users do
   member do
     get :network
     post :nation
   end
end

user_controller

before_action :set_user, only: [:show, :edit, :update, :destroy, :nation]


 def nation
    @country = User.find(params[:user][:selected_country])
    @user_country_array = Array.new
    @user_country_array = User.all.map {|user| user.country }.reject{ |country| country.nil? }.uniq.sort
    @user_count = User.all.select { |u| u.user_complete == true }.count
    @user_json = User.all.group_by(&:iso).map{|k,v| [k, v.count, k.to_s.downcase]}.map {|c, v | ["code" => c, "value" => v, "flag" => c.to_s.downcase]}.flatten.to_json
    @sectiona =  User.all.group('practice').count.map { |k,v| [ "name" => k, "y" => v] unless k.nil?}.reject { |a| a.blank? }.flatten.to_json
    render 'country'
 end


def set_user
  @user = User.find(params[:id])
end


# Never trust parameters from the scary internet, only allow the white list through.
def user_params
  params.require(:user).permit(:experience,
                               :kind,
                               :selected_country,
                             ])
end

和user.rb

  attr_accessor :selected_country

当我向一个国家提交表格时，我得到了...

  Couldn't find User with 'id'=American Samoa

我正在努力查看为什么before操作不能正确设置suer并将默认（似乎）设置为选择框中的值的原因

关于这个的任何建议。

Answer 1

您遇到的问题是，您正在尝试使用selected_country属性find User ：

@country = User.find(params[:user][:selected_country])

由于您将其设置为纯字符串，因此它将尝试用您选择的国家/地区查找User 。

如果要使用字符串查找值，最好使用find_by方法：

@country = User.find_by country: params[:user][:selected_country]

-

老实说，您的整个nation方法确实很糟糕（多个繁重的数据库调用等）。 我将执行以下操作：

 def nation
    @country = User.find_by country_params[:selected_country]
    @user_country_array = User.select(:country).distinct
    @user_count = User.where(user_complete: true).count
    @user_json = User.all.group_by(&:iso).map{|k,v| [k, v.count, k.to_s.downcase]}.map {|c, v | ["code" => c, "value" => v, "flag" => c.to_s.downcase]}.flatten.to_json #-> this needs to improve
    @sectiona =  User.all.group('practice').count.map { |k,v| [ "name" => k, "y" => v] unless k.nil?}.reject { |a| a.blank? }.flatten.to_json #-> this also needs to improve
    render :country
 end

 private

 def country_params
    params.require(:user).permit(:selected_country)
 end