？Rails のモデルについての疑問？

Question

###確認したいこと
Railsのモデルについてですが、いろいろ触っているうちに不安になる事柄があったので下記の２点について
認識があっているのか確認していただきたいです。
###マイグレーションについて
基本的にマイグレーションを行うタイミングとはどういうタイミングでしょうか？
また、change記法とup-down記法では違いがあるのでしょうか？
###モデルに書き込むことについて
一度作り上げたモデルファイルに後から、何か書き加えることはNGですか？
僕の認識では、「モデルはデータベースのテーブルに対する設定」といった感じで、後から書き込むのはOKだと思っています
逆にDBのスキーマに書き込むのはNGだと思っています。(db/schema.rb)
例えば次のようなuser.rbというモデルファイルがあります。

class User < ApplicationRecord

# validates :user_name, presence: true, length: { maximum: 50 }
# VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i
# validates :email, presence: true, length: { maximum: 255 },
#                   format: { with: VALID_EMAIL_REGEX },
#                   uniqueness: { case_sensitive: false }
has_secure_password
validates :password, presence: true, length: { minimum: 6 }, allow_nil: true
  #
  # def User.digest(string)
  # cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
  #                                               BCrypt::Engine.cost
  # BCrypt::Password.create(string, cost: cost)
  # end

end

このファイルに対し、下記のような設定を加えてユーザの認証なんかを実装することはNGですか？

 # Returns the hash digest of the given string.
  def User.digest(string)
    cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
                                                  BCrypt::Engine.cost
    BCrypt::Password.create(string, cost: cost)
  end

  # Returns a random token.
  def self.new_token
    SecureRandom.urlsafe_base64
  end

  # Remembers a user in the database for use in persistent sessions.
  def remember
    self.remember_token = User.new_token
    update_attribute(:remember_digest, User.digest(remember_token))
  end

  # Returns true if the given token matches the digest.
  def authenticated?(attribute, token)
    digest = send("#{attribute}_digest")
    return false if digest.nil?
    BCrypt::Password.new(digest).is_password?(token)
  end

  # Forgets a user.
  def forget
    update_attribute(:remember_digest, nil)
  end

  # Activates an account.
  def activate
    update_attribute(:activated,    true)
    update_attribute(:activated_at, Time.zone.now)
  end

  # Sends activation email.
  def send_activation_email
    UserMailer.account_activation(self).deliver_now
  end

  # Sets the password reset attributes.
  def create_reset_digest
    self.reset_token = User.new_token
    update_attribute(:reset_digest,  User.digest(reset_token))
    update_attribute(:reset_sent_at, Time.zone.now)
  end

  # Sends password reset email.
  def send_password_reset_email
    UserMailer.password_reset(self).deliver_now
  end

  # Returns true if a password reset has expired.
  def password_reset_expired?
    reset_sent_at < 2.hours.ago
  end

  # Returns a user's status feed.
  def feed
    following_ids = "SELECT followed_id FROM relationships
                     WHERE  follower_id = :user_id"
    Micropost.where("user_id IN (#{following_ids})
                     OR user_id = :user_id", user_id: id)
  end

  # Follows a user.
  def follow(other_user)
    following << other_user
  end

  # Unfollows a user.
  def unfollow(other_user)
    following.delete(other_user)
  end

  # Returns true if the current user is following the other user.
  def following?(other_user)
    following.include?(other_user)
  end

  private

    # Converts email to all lower-case.
    def downcase_email
      self.email = email.downcase
    end

    # Creates and assigns the activation token and digest.
    def create_activation_digest
      self.activation_token  = User.new_token
      self.activation_digest = User.digest(activation_token)
    end
end

Answer 1

１）change と　up,down
単に前へ進むだけならどちらも同じです。
rollback や　VERSION=**　で古い状態に戻すときに差がでます。
大体は change で作っても戻れますが、修正の内容によって戻れない場合があります。そういう修正の場合に進むための UP　と戻るためのdownを書き分けます

２）一度作り上げたモデルファイルに後から、何か書き加えることはOKです。

３）DBのスキーマに書き込むのはNGです
その行為そのものは通常は何の影響もありませんが、schemaを元にDBを再構築するコマンドがあるので、それを行った時にmigrationと食い違ってしまうのでまずいです

Comments

[ts21]

ご返答ありがとうございます。
DBのスキーマに関しての構造が理解できました。