Два столбца не должны равняться друг другу в Rails - PullRequest
Новая
       5

Два столбца не должны равняться друг другу в Rails

6 голосов
/ 19 февраля 2010

Я создаю социальную сеть в Rails, и у меня есть такая модель: 

create_table "friendships", :force => true do |t|
    t.integer  "user1_id"
    t.integer  "user2_id"
    t.boolean  "hasaccepted"
    t.datetime "created_at"
    t.datetime "updated_at"
end

Проблема в том, что вы не можете добавить себя в друзья, поэтому я попробовал это в моей модели: 

def validate
    if :user1_id == :user2_id
        record.errors.add "You cannot add yourself as a friend."
        return false
    end
end

И у меня есть это в моем контроллере: 

def addfriend
    if params[:id]
        @friendship = Friendship.new()
        @friendship.user1_id = session[:user]
        @friendship.user2_id = params[:id]
        respond_to do |format|
            if @friendship.save
                format.html { redirect_to "/" } # Yes, SO users, I will fix this redirect later and it is not important for now.
                format.xml  { render :xml => @friendship, :status => :created }
            else
                format.html { redirect_to "/" }
                format.xml  { render :xml => @friendship.errors, :status => :unprocessable_entity }
            end
        end
    end
end

(где session[:user] - идентификатор пользователя, вошедшего в систему)

Однако, когда я захожу на http://localhost:3000/profile/addfriend/2.xml, когда я вошел как пользователь 2, Rails возвращает мне новый Friendship вместо сообщения об ошибке, и когда я смотрю на свою базу данных, Friendship тоже есть (и не должно). Может кто-нибудь объяснить, пожалуйста, как это исправить? Спасибо

Ответы [ 2 ]

13 голосов
/ 19 февраля 2010

Попробуй так: 

class Friendship < ActiveRecord::Base
  validate :cannot_add_self

  private

  def cannot_add_self
    errors.add(:user2_id, 'You cannot add yourself as a friend.') if user1_id == user2_id
  end
end
4 голосов
/ 19 февраля 2010 
if :user1_id == :user2_id

Это всегда будет ложью - вы сравниваете символы. Это все равно что писать if "user1_id" == "user2_id".

Вы должны написать это как if user1_id == user2_id для сравнения значений столбцов.

...