rails4 の redirect_to とかparams とか

Question

現在、railsアプリケーションの作成中なのですが、進捗が滞っているので…

目的としては、
・new.html.erb でフォームに必要な値を入力してユーザ新規作成
・コントローラでDBにinsert
・insert後、show.html.erb で新規に作成したユーザの情報を表示
したいのですが、show.html.erbでの検索結果の表示がうまくいきません。

insert は問題なくできているので、whereとかfindの中の構文がおかしいのだと思うのですが…

環境
windows8.1
rails4
sqlite3

new.html.erb↓

<%= form_for User.new, :url => {:action => 'create'} do |u| %>
	<%= u.label :name, :class => "name" %><br>
	<%= u.text_field :name, :class => "text" %>
	<br>
	<%= u.label :username %><br>
	<%= u.text_field :uname, :maxlength => 20, :class => "text" %>
	<br>
	<%= u.label :location %><br>
	<%= u.text_field :loc, :class => "text" %>
	<br>
	<%= u.label :bio %><br>
	<%= u.text_area :bio, :maxlength => 60 %><br>
	<br>
	<%= u.submit :class => "btn btn-primary" %>
<% end %>

コントローラ↓

def create
  	@user = User.create(
:name => params[:user][:name],
:uname => params[:user][:uname],
:loc => params[:user][:loc],
:bio => params[:user][:bio])
  	redirect_to :controller => 'users', :action => 'show', :uname => @user.uname
  	@user.save

show.html.erb↓

<%= User.where(params[:uname]) do |u| %>
<h1><%= u.name %></h1>
<p><%= u.uname %></p>
<ul>
	<li>Location : <%= u.loc %></li>
	<li>About    : <%= u.bio %></li>
</ul>
<% end %>

のように、ユーザネームでの検索を行いたいです。(一意キーなので)

whereメソッドやfindメソッドの条件を書き換えて色々試しているのですが、吐くエラーも様々なので、どうしたらいいものか…

検索は、コントローラ側で行うべきなのでしょうか？
また、何度やってもDBの先頭レコードしか取得できないのには何か理由があるのでしょうか？
うまく検索する方法を教えていただけるとありがたいです。。。

Answer 1

やりたいことはこうではないですか？

class UsersController
  # 他は省略

  def show
    @user = User.find_by(uname: params[:uname])
  end
end

<h1><%= @user.name %></h1>
<p><%= @user.uname %></p>
<ul>
    <li>Location : <%= @user.loc %></li>
    <li>About    : <%= @user.bio %></li>
</ul>

Comments

[zvub1123]

ありがとうございます。
あっさりできてしまいました！
@user do |u|
u.name
u.uname ...
のようにしていたのがいけなかったみたいですね^^;

Answer 2

エラー内容が不明なのでなんとも言えませんが
<%= User.where(params[:uname]) do |u| %>はeachではないのですか？
whereをこういう風に書いてるのは見たことありません
僕も初心者なので断言できませんが・・

通常、検索ロジックはコントローラに書くと思います。
コントローラーに検索ロジックを書きviewはu.nameなどをeachで回せば良いと思います。

経験談からですが、入門書は迷わず買った方が良いです。
躓いて２時間使うことはざらにありますが、入門書を見れば劇的にその回数を減らせます。

Comments

[zvub1123]

ご回答ありがとうございます！
入門書を持っていないので手探りで作っていて躓きました。
ロジックによってコントローラ、ビューなどの操作を変えるのは、確かにいいですね。
入門書も近いうちに購入しようと思います。