質問編集履歴

モデルの追記

2020/08/31 12:31

投稿

maeshu

スコア1

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -258,198 +258,84 @@
-### 該当すると思ったコード
+### 追記
-```
+```
+create_table :users do |t|
+      ## Database authenticatable
+      t.string :username,           null: false
+      t.text :profile
+      t.string :profile_image_id
+      t.string :email,              null: false, default: ""
+      t.string :encrypted_password, null: false, default: ""
+```
+```
-class TweetsController < ApplicationController
+class CreateTweets < ActiveRecord::Migration[6.0]
-  before_action :authenticate_user!, except: [:index]
-  def index
-    @tweets = Tweet.all
-  end
-  def show
-    @tweet = Tweet.find(params[:id])
-  end
-  def new
-    @tweet = Tweet.new
-  end
-  def create
+  def change
-    @tweet = Tweet.new(tweet_params)
+    create_table :tweets do |t|
-    @tweet.user_id = current_user.id
-    if @tweet.save
-      redirect_to tweet_path(@tweet), notice: '投稿に成功しました'
-    else
-      render :new
+      t.integer :user_id
+      t.string :title
+      t.text :body
+      t.timestamps
     end
   end
+end
+```
+```
+class CreateFavorites < ActiveRecord::Migration[6.0]
-  def edit
+  def change
-    @tweet = Tweet.find(params[:id])
-    if @tweet.user != current_user
+    create_table :favorites do |t|
+      t.integer :user_id
-      redirect_to tweets_path, alert: '不正なアクセスです'
+      t.integer :tweet_id
+      t.timestamps
     end
   end
-  def update
-    @tweet = Tweet.find(params[:id])
-    if @tweet.update(tweet_params)
-      redirect_to tweet_path(@tweet), notice: '更新に成功しました'
-    else
-      render :edit
-    end
+end
-  end
-  def destroy
-    tweet = Tweet.find(params[:id])
-    tweet.destroy
-    redirect_back(fallback_location: root_path)
-  end
-  private
-  def tweet_params
-    params.require(:tweet).permit(:title, :body)
-  end
-end
-```
+```
-### 追記
-```
+```
-create_table :users do |t|
-      ## Database authenticatable
-      t.string :username,           null: false
-      t.text :profile
-      t.string :profile_image_id
-      t.string :email,              null: false, default: ""
-      t.string :encrypted_password, null: false, default: ""
-```
-```
-class CreateTweets < ActiveRecord::Migration[6.0]
+class RemoveNameFromTweets < ActiveRecord::Migration[6.0]
   def change
-    create_table :tweets do |t|
-      t.integer :user_id
-      t.string :title
-      t.text :body
-      t.timestamps
-    end
-  end
-end
-```
-```
-class CreateFavorites < ActiveRecord::Migration[6.0]
-  def change
-    create_table :favorites do |t|
-      t.integer :user_id
-      t.integer :tweet_id
-      t.timestamps
-    end
-  end
-end
-```
-```
-class RemoveNameFromTweets < ActiveRecord::Migration[6.0]
-  def change
     remove_column :tweets, :image_id, :string
   end
@@ -516,4 +402,76 @@
 ```
+```
+class Tweet < ApplicationRecord
+  belongs_to :user
+  attachment :image
+  has_many :favorites, dependent: :destroy
+  with_options presence: true do
+    validates :title
-よろしくお願いします。
+    validates :body
+  end
+end
+```
+```
+class User < ApplicationRecord
+  # Include default devise modules. Others available are:
+  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
+  devise :database_authenticatable, :registerable,
+         :recoverable, :rememberable, :validatable
+  validates :username, presence: true, uniqueness: true
+  attachment :profile_image
+  has_many :tweets, dependent: :destroy
+  has_many :favorites, dependent: :destroy
+  def already_favorited?(tweet)
+    self.favorites.exists?(tweet_id: tweet.id)
+  end
+end
+```
+```
+class Favorite < ApplicationRecord
+  belongs_to :user
+  belongs_to :tweet
+  validates_uniqueness_of :tweet_id, scope: :user_id
+end
+```

必要ない箇所の削除

2020/08/31 12:31

投稿

maeshu

スコア1

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -260,6 +260,140 @@
 ### 該当すると思ったコード
+```
+class TweetsController < ApplicationController
+  before_action :authenticate_user!, except: [:index]
+  def index
+    @tweets = Tweet.all
+  end
+  def show
+    @tweet = Tweet.find(params[:id])
+  end
+  def new
+    @tweet = Tweet.new
+  end
+  def create
+    @tweet = Tweet.new(tweet_params)
+    @tweet.user_id = current_user.id
+    if @tweet.save
+      redirect_to tweet_path(@tweet), notice: '投稿に成功しました'
+    else
+      render :new
+    end
+  end
+  def edit
+    @tweet = Tweet.find(params[:id])
+    if @tweet.user != current_user
+      redirect_to tweets_path, alert: '不正なアクセスです'
+    end
+  end
+  def update
+    @tweet = Tweet.find(params[:id])
+    if @tweet.update(tweet_params)
+      redirect_to tweet_path(@tweet), notice: '更新に成功しました'
+    else
+      render :edit
+    end
+  end
+  def destroy
+    tweet = Tweet.find(params[:id])
+    tweet.destroy
+    redirect_back(fallback_location: root_path)
+  end
+  private
+  def tweet_params
+    params.require(:tweet).permit(:title, :body)
+  end
+end
+```
+### 追記
+```
+create_table :users do |t|
+      ## Database authenticatable
+      t.string :username,           null: false
+      t.text :profile
+      t.string :profile_image_id
+      t.string :email,              null: false, default: ""
+      t.string :encrypted_password, null: false, default: ""
+```
 ```
 class CreateTweets < ActiveRecord::Migration[6.0]
@@ -286,7 +420,29 @@
 ```
+```
+class CreateFavorites < ActiveRecord::Migration[6.0]
+  def change
+    create_table :favorites do |t|
+      t.integer :user_id
+      t.integer :tweet_id
+      t.timestamps
+    end
+  end
+end
+```
 ```
@@ -304,105 +460,53 @@
 ```
-class TweetsController < ApplicationController
-  before_action :authenticate_user!, except: [:index]
-  def index
-    @tweets = Tweet.all
-  end
-  def show
-    @tweet = Tweet.find(params[:id])
-  end
-  def new
-    @tweet = Tweet.new
-  end
-  def create
-    @tweet = Tweet.new(tweet_params)
-    @tweet.user_id = current_user.id
-    if @tweet.save
-      redirect_to tweet_path(@tweet), notice: '投稿に成功しました'
-    else
-      render :new
-    end
-  end
-  def edit
-    @tweet = Tweet.find(params[:id])
-    if @tweet.user != current_user
-      redirect_to tweets_path, alert: '不正なアクセスです'
-    end
-  end
-  def update
-    @tweet = Tweet.find(params[:id])
-    if @tweet.update(tweet_params)
-      redirect_to tweet_path(@tweet), notice: '更新に成功しました'
-    else
-      render :edit
-    end
-  end
-  def destroy
-    tweet = Tweet.find(params[:id])
-    tweet.destroy
-    redirect_back(fallback_location: root_path)
-  end
-  private
-  def tweet_params
-    params.require(:tweet).permit(:title, :body)
+ActiveRecord::Schema.define(version: 2020_08_23_132550) do
+  create_table "tweets", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
+    t.integer "user_id"
+    t.string "title"
+    t.text "body"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+  create_table "users", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
+    t.string "username", null: false
+    t.text "profile"
+    t.string "profile_image_id"
+    t.string "email", default: "", null: false
+    t.string "encrypted_password", default: "", null: false
+    t.string "reset_password_token"
+    t.datetime "reset_password_sent_at"
+    t.datetime "remember_created_at"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+    t.index ["email"], name: "index_users_on_email", unique: true
+    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
+    t.index ["username"], name: "index_users_on_username", unique: true
   end
@@ -412,148 +516,4 @@
 ```
-### 追記
-```
-create_table :users do |t|
-      ## Database authenticatable
-      t.string :username,           null: false
-      t.text :profile
-      t.string :profile_image_id
-      t.string :email,              null: false, default: ""
-      t.string :encrypted_password, null: false, default: ""
-```
-```
-class CreateTweets < ActiveRecord::Migration[6.0]
-  def change
-    create_table :tweets do |t|
-      t.integer :user_id
-      t.string :title
-      t.text :body
-      t.timestamps
-    end
-  end
-end
-```
-```
-class CreateFavorites < ActiveRecord::Migration[6.0]
-  def change
-    create_table :favorites do |t|
-      t.integer :user_id
-      t.integer :tweet_id
-      t.timestamps
-    end
-  end
-end
-```
-```
-class RemoveNameFromTweets < ActiveRecord::Migration[6.0]
-  def change
-    remove_column :tweets, :image_id, :string
-  end
-end
-```
-```
-ActiveRecord::Schema.define(version: 2020_08_23_132550) do
-  create_table "tweets", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
-    t.integer "user_id"
-    t.string "title"
-    t.text "body"
-    t.datetime "created_at", precision: 6, null: false
-    t.datetime "updated_at", precision: 6, null: false
-  end
-  create_table "users", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
-    t.string "username", null: false
-    t.text "profile"
-    t.string "profile_image_id"
-    t.string "email", default: "", null: false
-    t.string "encrypted_password", default: "", null: false
-    t.string "reset_password_token"
-    t.datetime "reset_password_sent_at"
-    t.datetime "remember_created_at"
-    t.datetime "created_at", precision: 6, null: false
-    t.datetime "updated_at", precision: 6, null: false
-    t.index ["email"], name: "index_users_on_email", unique: true
-    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
-    t.index ["username"], name: "index_users_on_username", unique: true
-  end
-end
-```
 よろしくお願いします。

マイグレーションファイルの追加

2020/08/31 11:36

投稿

maeshu

スコア1

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -18,14 +18,6 @@
 ```
-2020-08-29T16:07:23.603505+00:00 app[web.1]: [d54d13b6-d44c-45d0-977e-874c9fc672ca]   User Load (2.7ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 1 ORDER BY `users`.`id` ASC LIMIT 1
-2020-08-29T16:07:23.607711+00:00 app[web.1]: [d54d13b6-d44c-45d0-977e-874c9fc672ca]   Tweet Load (3.7ms)  SELECT `tweets`.* FROM `tweets` WHERE `tweets`.`id` = 11 LIMIT 1
-2020-08-29T16:07:23.610865+00:00 app[web.1]: [d54d13b6-d44c-45d0-977e-874c9fc672ca]    (2.4ms)  BEGIN
-2020-08-29T16:07:23.613490+00:00 app[web.1]: [d54d13b6-d44c-45d0-977e-874c9fc672ca]   Favorite Load (2.5ms)  SELECT `favorites`.* FROM `favorites` WHERE `favorites`.`tweet_id` = 11
 2020-08-29T16:07:23.618829+00:00 app[web.1]: [d54d13b6-d44c-45d0-977e-874c9fc672ca]   Tweet Destroy (4.9ms)  DELETE FROM `tweets` WHERE `tweets`.`id` = 11
 2020-08-29T16:07:23.623514+00:00 app[web.1]: [d54d13b6-d44c-45d0-977e-874c9fc672ca]    (4.5ms)  ROLLBACK
@@ -240,16 +232,6 @@
-ローカル環境エラー発生
-```
-ActiveRecord::PendingMigrationError
-Migrations are pending. To resolve this issue, run: rails db:migrate RAILS_ENV=development
-```
 rails db:migrate:status
@@ -430,6 +412,148 @@
 ```
+### 追記
+```
+create_table :users do |t|
+      ## Database authenticatable
+      t.string :username,           null: false
+      t.text :profile
+      t.string :profile_image_id
+      t.string :email,              null: false, default: ""
+      t.string :encrypted_password, null: false, default: ""
+```
+```
+class CreateTweets < ActiveRecord::Migration[6.0]
+  def change
+    create_table :tweets do |t|
+      t.integer :user_id
+      t.string :title
+      t.text :body
+      t.timestamps
+    end
+  end
+end
+```
+```
+class CreateFavorites < ActiveRecord::Migration[6.0]
+  def change
+    create_table :favorites do |t|
+      t.integer :user_id
+      t.integer :tweet_id
+      t.timestamps
+    end
+  end
+end
+```
+```
+class RemoveNameFromTweets < ActiveRecord::Migration[6.0]
+  def change
-初めてのherokuにデプロイとマイグレーション系のエラーでとても焦っています...。
+    remove_column :tweets, :image_id, :string
+  end
+end
+```
+```
+ActiveRecord::Schema.define(version: 2020_08_23_132550) do
+  create_table "tweets", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
+    t.integer "user_id"
+    t.string "title"
+    t.text "body"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+  create_table "users", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
+    t.string "username", null: false
+    t.text "profile"
+    t.string "profile_image_id"
+    t.string "email", default: "", null: false
+    t.string "encrypted_password", default: "", null: false
+    t.string "reset_password_token"
+    t.datetime "reset_password_sent_at"
+    t.datetime "remember_created_at"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+    t.index ["email"], name: "index_users_on_email", unique: true
+    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
+    t.index ["username"], name: "index_users_on_username", unique: true
+  end
+end
+```
-足りない箇所はすぐに添付いたします。よろしくお願いします。
+よろしくお願いします。