Associations between two models [closed] - ruby-on-rails

Question

Closed. This question needs details or clarity. It is not currently accepting answers.
Want to improve this question? Add details and clarify the problem by editing this post.
Closed 6 years ago.
Improve this question
Thank you for visiting. I'm currently building a blogging app which will have 3 models. User, Article and Comments that will go into comments. I hope someone can help explain to me where I'm going wrong and how I can associated models better. Current error I get is undefined method `users' in Articles#new controller.
My current code is:
Schema
create_table "articles", force: :cascade do |t|
t.string "title"
t.text "body"
t.integer "user_id"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end
create_table "comments", force: :cascade do |t|
t.text "body"
t.integer "post_id"
t.integer "user_id"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
t.index ["post_id"], name: "index_comments_on_post_id"
t.index ["user_id"], name: "index_comments_on_user_id"
end
create_table "users", force: :cascade do |t|
t.string "username"
t.string "email"
t.string "password_digest"
t.text "about_me"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
end
Models
class User < ApplicationRecord
has_secure_password
has_many :articles
has_many :comments
end
class Article < ApplicationRecord
belongs_to :user
has_many :comments
validates :user_id, presence: true
end
class Comment < ApplicationRecord
belongs_to :post
belongs_to :user
end
Controller
class ArticlesController < ApplicationController
before_action :find_article, only: [:show, :edit, :update, :destroy]
def index
#articles = Article.all
end
def new
#article = Article.new
end
def create
#user = User.find(params[:user_id])
#article = #user.articles.create(article_params)
if #article.save
redirect_to articles_path
else
redirect_to new_article_path
end
end
def show
end
def edit
end
def update
if #article.update(article_params)
redirect_to :back
else
end
end
def destroy
if #article.destroy
flash[:notice] = "Successfully destroy the article"
redirect_to #article
else
flash[:alert] = "Couldnt delete article"
redirect_to :back
end
end
private
def find_article
#article = Article.find(params[:id])
end
def article_params
params.require(:article).permit(:title, :body)
end
end
View
<%= form_for(#article, #article.users.build) do |f| %>
<%= f.label :title %>
<%= f.text_field :title %>
<%= f.label :body %>
<%= f.text_field :body %>
<%= f.submit %>
<% end %>
Routes
root 'users#index'
resources :users
resources :articles do
resources :comments
end
If there is something else I might be missing in order to help explain what I'm doing wrong and how to improve/understanding association and building/creating. Please let me know and I'll add the additional information.
For everyone who teaches me/helps me, seriously thank you.

Answer 1

Your form should be
<%= form_for #article do |f| %>
Then in the controller
def create
#article = current_user.articles.build(article_params)
if #article.save
redirect_to articles_path
else
redirect_to new_article_path
end
end