編集履歴

質問編集履歴

message_tag.rbの変更

2021/01/31 12:42

投稿

shunxile

スコア26

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -268,13 +268,13 @@
   def update
-    @message = Message.create(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
+    @message = Message.update(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
     tag = Tag.find_or_create_by(name: name)
-    @message_tag = MessageTagForm.create(message_id: message.id, tag_id: tag.id)
+    @message_tag = MessageTagForm.update(message_id: message.id, tag_id: tag.id)
   end

コントローラー・ビュー再編集

2021/01/31 12:42

投稿

shunxile

スコア26

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -150,9 +150,9 @@
   def edit
-    @message_tag = Message.find(params[:id])
+    @message = Message.find(params[:id])
-    if current_user.id != @message_tag.user.id
+    if current_user.id != @message.user.id
       redirect_to root_path
@@ -432,7 +432,7 @@
   <h1>残したい想い</h1>
-  <%= form_with model: @message_tag, multipart: true, local: true do |f| %>
+  <%= form_with model: @message, multipart: true, local: true do |f| %>
   <%= render 'shared/error_messages', model: f.object %>

コントローラー内new、create、edit、update・edit.html.erb変更

2021/01/31 09:15

投稿

shunxile

スコア26

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -114,7 +114,7 @@
   def new
-    @message = MessageTag.new
+    @message_tag = MessageTag.new
   end
@@ -122,11 +122,11 @@
   def create
-    @message = MessageTag.new(message_params)
+    @message_tag = MessageTag.new(message_params)
-    if @message.valid?
+    if @message_tag.valid?
-      @message.save
+      @message_tag.save
       return redirect_to root_path
@@ -150,9 +150,43 @@
   def edit
+    @message_tag = Message.find(params[:id])
+    if current_user.id != @message_tag.user.id
+      redirect_to root_path
+    end
+  end
+  def update
     @message = Message.find(params[:id])
+    if @message_tag = MessageTag.new.update
+      return redirect_to root_path
+    else
+      render :edit
+    end
+  end
+  def destroy
+    @message = Message.find(params[:id])
-    if current_user.id != @message.user.id
+    if current_user.id == @message.user.id
+      @message.destroy
       redirect_to root_path
@@ -162,119 +196,121 @@
+  private
+  def message_params
+    params.require(:message_tag).permit(:title, :whom, :message, :open_plan, :name, :video, images: []).merge(user_id: current_user.id)
+  end
+end
+```
+```Ruby
+message_tag.rb
+class MessageTag
+  include ActiveModel::Model
+  include ActiveModel::Attributes
+  include ActiveRecord::AttributeAssignment
+  attr_accessor :title, :whom, :open_plan, :message, :images, :video, :name, :user_id
+  with_options presence: true, length: { maximum: 50 } do
+    validates :title
+    validates :whom
+  end
+  with_options presence: true do
+    validates :message, length: { maximum: 200 }
+    validates :open_plan
+    validates :user_id
+    validates :name, length: { maximum: 20 }
+  end
+  def save
+    @message = Message.create(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
+    tag = Tag.find_or_create_by(name: name)
+    @message_tag = MessageTagForm.create(message_id: message.id, tag_id: tag.id)
+  end
   def update
+    @message = Message.create(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
-    @message = Message.find(params[:id])
+    tag = Tag.find_or_create_by(name: name)
-    if @message = MessageTag.new.update
+    @message_tag = MessageTagForm.create(message_id: message.id, tag_id: tag.id)
-      return redirect_to root_path
-    else
-      render :edit
-    end
+  end
-  end
+end
-  def destroy
-    @message = Message.find(params[:id])
-    if current_user.id == @message.user.id
-      @message.destroy
-      redirect_to root_path
-    end
-  end
-  private
-  def message_params
-    params.require(:message_tag).permit(:title, :whom, :message, :open_plan, :name, :video, images: []).merge(user_id: current_user.id)
-  end
-end
 ```
 ```Ruby
-message_tag.rb
+message.rb
-class MessageTag
+class Message < ApplicationRecord
+  belongs_to :user
+  has_many_attached :images
-  include ActiveModel::Model
+  has_one_attached :video
+  has_many :message_tag_forms, dependent: :destroy
+  has_many :tags, through: :message_tag_forms
+  def self.search(search, id)
-  include ActiveModel::Attributes
+    if search != ""
-  include ActiveRecord::AttributeAssignment
-  attr_accessor :title, :whom, :open_plan, :message, :images, :video, :name, :user_id
+      Item.joins(:tags).where(tags: {name: "#{search}"} , user_id: id)
-  with_options presence: true, length: { maximum: 50 } do
+    else
-    validates :title
+      Item.where(user_id: id)
-    validates :whom
-  end
+    end
-  with_options presence: true do
-    validates :message, length: { maximum: 200 }
-    validates :open_plan
-    validates :user_id
-    validates :name, length: { maximum: 20 }
-  end
-  def save
-    @message = Message.create(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
-    tag = Tag.find_or_create_by(name: name)
-    @message_tag = MessageTagForm.create(message_id: message.id, tag_id: tag.id)
-  end
-  def update
-    @message = Message.create(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
-    tag = Tag.find_or_create_by(name: name)
-    @message_tag = MessageTagForm.create(message_id: message.id, tag_id: tag.id)
   end
@@ -284,35 +320,17 @@
 ```Ruby
-message.rb
+tag.rb
-class Message < ApplicationRecord
+class Tag < ApplicationRecord
-  belongs_to :user
-  has_many_attached :images
-  has_one_attached :video
-  has_many :message_tag_forms, dependent: :destroy
-  has_many :tags, through: :message_tag_forms
+  has_many :messages, through: :message_tag_forms
-  def self.search(search, id)
-    if search != ""
+  has_many :message_tag_forms
-      Item.joins(:tags).where(tags: {name: "#{search}"} , user_id: id)
-    else
-      Item.where(user_id: id)
+  validates :name, uniqueness: true
-    end
-  end
 end
@@ -320,17 +338,11 @@
 ```Ruby
-tag.rb
-class Tag < ApplicationRecord
+class MessageTagForm < ApplicationRecord
-  has_many :messages, through: :message_tag_forms
-  has_many :message_tag_forms
+  belongs_to :message
-  validates :name, uniqueness: true
+  belongs_to :tag
 end
@@ -338,11 +350,75 @@
 ```Ruby
+class CreateTags < ActiveRecord::Migration[6.0]
+  def change
+    create_table :tags do |t|
+      t.string :name, null:false, uniqueness: true
+      t.timestamps
+    end
+  end
+end
+class CreateMessages < ActiveRecord::Migration[6.0]
+  def change
+    create_table :messages do |t|
+      t.string :title, null: false
+      t.string :whom, null: false
+      t.text :message, null: false
+      t.date :open_plan, null: false
+      t.string :encrypted_password
+      t.references :user, null: false
+      t.timestamps
+    end
+  end
+end
-class MessageTagForm < ApplicationRecord
+class CreateMessageTagForms < ActiveRecord::Migration[6.0]
+  def change
-  belongs_to :message
+    create_table :message_tag_forms do |t|
+      t.references :message, foreign_key: true
+      t.references :tag, foreign_key: true
-  belongs_to :tag
+      t.timestamps
+    end
+  end
 end
@@ -350,89 +426,13 @@
 ```Ruby
-class CreateTags < ActiveRecord::Migration[6.0]
-  def change
-    create_table :tags do |t|
-      t.string :name, null:false, uniqueness: true
-      t.timestamps
-    end
-  end
-end
-class CreateMessages < ActiveRecord::Migration[6.0]
-  def change
-    create_table :messages do |t|
-      t.string :title, null: false
-      t.string :whom, null: false
-      t.text :message, null: false
-      t.date :open_plan, null: false
-      t.string :encrypted_password
-      t.references :user, null: false
-      t.timestamps
-    end
-  end
-end
-class CreateMessageTagForms < ActiveRecord::Migration[6.0]
-  def change
-    create_table :message_tag_forms do |t|
-      t.references :message, foreign_key: true
-      t.references :tag, foreign_key: true
-      t.timestamps
-    end
-  end
-end
-```
-```Ruby
 edit.html.erb
 <div class="new-message">
   <h1>残したい想い</h1>
-  <%= form_with model: @message, url: message_path(@message.id), method: :patch, multipart: true, local: true do |f| %>
+  <%= form_with model: @message_tag, multipart: true, local: true do |f| %>
   <%= render 'shared/error_messages', model: f.object %>

モデル・db追加

2021/01/31 09:04

投稿

shunxile

スコア26

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -284,6 +284,148 @@
 ```Ruby
+message.rb
+class Message < ApplicationRecord
+  belongs_to :user
+  has_many_attached :images
+  has_one_attached :video
+  has_many :message_tag_forms, dependent: :destroy
+  has_many :tags, through: :message_tag_forms
+  def self.search(search, id)
+    if search != ""
+      Item.joins(:tags).where(tags: {name: "#{search}"} , user_id: id)
+    else
+      Item.where(user_id: id)
+    end
+  end
+end
+```
+```Ruby
+tag.rb
+class Tag < ApplicationRecord
+  has_many :messages, through: :message_tag_forms
+  has_many :message_tag_forms
+  validates :name, uniqueness: true
+end
+```
+```Ruby
+class MessageTagForm < ApplicationRecord
+  belongs_to :message
+  belongs_to :tag
+end
+```
+```Ruby
+class CreateTags < ActiveRecord::Migration[6.0]
+  def change
+    create_table :tags do |t|
+      t.string :name, null:false, uniqueness: true
+      t.timestamps
+    end
+  end
+end
+class CreateMessages < ActiveRecord::Migration[6.0]
+  def change
+    create_table :messages do |t|
+      t.string :title, null: false
+      t.string :whom, null: false
+      t.text :message, null: false
+      t.date :open_plan, null: false
+      t.string :encrypted_password
+      t.references :user, null: false
+      t.timestamps
+    end
+  end
+end
+class CreateMessageTagForms < ActiveRecord::Migration[6.0]
+  def change
+    create_table :message_tag_forms do |t|
+      t.references :message, foreign_key: true
+      t.references :tag, foreign_key: true
+      t.timestamps
+    end
+  end
+end
+```
+```Ruby
 edit.html.erb
 <div class="new-message">

誤字

2021/01/30 09:39

投稿

shunxile

スコア26

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -432,7 +432,7 @@
 Request
-PParameters:
+Parameters:
@@ -456,10 +456,16 @@
 参考記事
 https://teratail.com/questions/291532
 https://qiita.com/Tiima/items/b14c73df98d0465cbb52
 https://qiita.com/divclass123/items/34c4a5a7200238a0589c