ビューの中身を変える部分が分かりません

Question

###前提・実現したいこと
該当のソースコードにあるAを指定してビューを読み込むと正しく名前やコンテンツの出力ができません。Bを指定して実行すると正しく動きます。だけどBを指定すると会員でもない誰でも追加や編集ができて困っています。
Aを指定してビューを変える部分を探していますが、分かりません。

###該当のソースコード

A.
@user = User.find(current_user)
@site = @user.sites.build

B.
@site = Site.new(site_params)
 
views _form
<%= form_for @site do |f| %>
  <% if @site.errors.any? %>
    <% @site.errors.full_messages.each do |msg| %>
      <%= msg %><br />
    <% end %>
  <% end %>
<%= f.text_field :name %><br />
<%= f.text_field :content %><br />
<%= f.submit %>
<% end %>

index.
<% @site.each do |site| %>
<ul class="linkBox">
  <li><a href= <%= site.content%> target="_blank"><%= site.name %></a></li>
</ul>

Answer 1

もう少し、何をしたいのかを明確に意識して、何をすべきなのか、何をしているのかを一行一行考えながら進めてみる事をお勧めします。
特に、hoge.fuga.buildみたいなものは、内部でどのような処理がなされているのか(歴史的に見ると、いくつかの処理に分かれていたものを一行出かけるように改良されたメソッドです)を意識するべきでしょう。

# app/controllers/sites_controller.rb
class SitesController < ApplicationController
  # このコントローラでのアクションでは、全て最初にログインしているユーザーをインスタンス化する
  before_filter :signin_user

  private
  # サインインしているユーザーをインスタンス化する
  def signin_user
    # 前提として、サインインしているユーザーのIDはsession[:signin_id]に入っているものとする
    @user = User.find(session[:signin_id])
  end

  public
  # サインイン中のユーザーのsitesを一覧表示する
  def index
    @sites = @user.sites
  end

  def new
    @site = Site.new
  end

  def create
    @site = Site.new  # 空のsiteを生成
    @site.attributes = params[:site] # ビューで入力された値をセットする
    @site.user = @user # 新しいsiteと関連するのはログイン中のユーザー
    if @site.save
      flash[:success] = "新しいSiteを追加しました"
      redirect_to action: :index # 成功したので一覧ページにリダイレクト
    else
      render action: :new # 失敗したので、newページに戻る
    end
  end
  
  # 以下略
end

# app/views/sites/index.html.erb
<h1><%= @user.name %>のSite</h1>
<ul>
<% @sites.each do |site| %>
  <li>
    <%= link_to site.content, site.name, target: "_blank" %><!-- href=site.name, text=site.content target="_blank"のaタグを生成する -->
  </li>
<% end %>
</ul>

# app/views/sites/new.html.erb
<%= render :partial 'form' %>

# app/views/sites/_form.html.erb
<%= form_for @site do |f| %>
  <% if @site.errors.any? %>
    <% @site.errors.full_messages.each do |msg| %>
      <%= msg %><br />
    <% end %>
  <% end %>
  <%= f.text_field :name %><br />
  <%= f.text_field :content %><br />
  <%= f.submit %>
<% end %>

Comments

[退会済みユーザー]

そのままコードを写してもうまくいかず、1行1行コードをよく見て、理解し変更していけば思い通りに行きました！大変ありがとうございました。