Как работать с результатом запроса из RoR в БД?

Question

[Кирилл Казаков]

Есть модель в RoR. Делаю запрос в БД , должно вернуться из запроса boolean значение.

logger.debug "subscribe_on subs-: #{subs}"

- показывает

subscribe_on subs-: #Subscription:0x7f9954e08158

logger.debug "subscribe_on subs-: #{subs.inspect}"

показывает

subscribe_on subs-: #Subscription subscribe: false

Какой тип данных возвращает запрос и как из него получить boolean тип ?

Upd:
Модель :

validates :email, :presence=> true, 
  					:format=> VALID_EMAIL_REGEX,
  					:uniqueness=> {:if=> :subscribe_on?}
def subscribe_on?
  subs = Subscription.find(:email self.email).subscribe
  logger.debug "subscribe_on subs-: #{subs}"
end

UPD2:
В общем -то нужен почти такой код

class Subscription < ActiveRecord::Base
  before_save { self.email = email.downcase}
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
  validates :email, :presence=> true, :format=> VALID_EMAIL_REGEX, :uniqueness=> {:scope => :subscribe}

Выполняется такой запрос

SELECT 1 FROM `subscriptions` WHERE `subscriptions`.`subscribe` = 0 AND (`subscriptions`.`email` = BINARY '[e-mail]') LIMIT 1

А надо вместо subscribe` = 0
subscribe` = 1
Пока не получилось( , как ?
ruby 1.8.7

Answer 1

[Дима]

class Subscription < ActiveRecord::Base
  before_save { email = email.downcase}
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
  validates :email, :presence=> true, :format=> VALID_EMAIL_REGEX, :uniqueness=> {:if=> :subscribe_on?}
  
  scope :subscribe_on, lambda(subscribe_email){ where(email: subscribe_email, subscribe: true) }
        
  def subscribe_on?
    subs = self.class.subscribe_on(email)
    logger.debug "subscribe_on subs-: #{subs}"
    subs.first
  end
end

Comments

[Кирилл Казаков]

Ругается
syntax error, unexpected ')', expecting kEND
на точку .subscribe

[Дима]

Кирилл Казаков: А у вас какая версия Ruby? Я, кстати, поправил, не `find`, а `find_by`.

[Кирилл Казаков]

Дима: ruby 1.8.7

[Дима]

Кирилл Казаков: оу, тогда ещё раз поправил.

[Кирилл Казаков]

Дима:

NoMethodError (undefined method `find_by' for #<Class:0x7f9954808690>):

Говорит, что не знает find_by

[Дима]

Кирилл Казаков: Выложи полный код модели. С определением класса.

[Кирилл Казаков]

Дима:

class Subscription < ActiveRecord::Base

  before_save { self.email = email.downcase}
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
  validates :email, :presence=> true, 
  					:format=> VALID_EMAIL_REGEX,
            #:uniqueness=> true
  					:uniqueness=> {:if=> :subscribe_on?}
            
  def subscribe_on?
    #subs = Subscription.select('subscribe').where(:email=>self.email).first
    subs = Subscription.find_by(:email => self.email).subscribe
   
    logger.debug "subscribe_on subs-: #{subs}"
   
  end

end

[Дима]

Кирилл Казаков: удали метод `subscribe_on`. В валидации напиши вместо `:uniqueness=> {:if=> :subscribe_on?}` - `:uniqueness=> {:if=> :subscribe?}`. Если, конечно, поле `subscribe` существует, и описано в `schema.rb`

[Кирилл Казаков]

Дима: В БД subscribe существует.
В логере не происходит запроса в бд , соответственно не происходит валидации.
В таком случае валидатор ждет subscribe из поля ввода вместе с email , как я думаю.
Его там конечно нет.

[Дима]

Кирилл Казаков: В приведенном коде метод `subscribe_on?` возвращает результат выражения ` logger.debug "subscribe_on subs-: #{subs}"`. Я много думал, и понял, что суть вашей проверки - проверить, существует ли подписка с передаваемым `email` и `subscription` равным `true`. Сейчас напишу код в ответе.

[Кирилл Казаков]

Дима: schema.rb - вообще нет, нужен ?

[Дима]

Кирилл Казаков: Ещё скажите, что миграции нет. Как таблица была создана?

[Кирилл Казаков]

Дима: не, миграцию не делал.
Таблица subscriptions

CREATE TABLE IF NOT EXISTS `subscriptions` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `email` varchar(255) CHARACTER SET utf8 COLLATE utf8_unicode_ci NOT NULL,
  `name` varchar(255) CHARACTER SET utf8 COLLATE utf8_unicode_ci DEFAULT NULL,
  `subscribe` tinyint(1) NOT NULL DEFAULT '0',
  `created_at` datetime NOT NULL,
  `ip` int(10) unsigned DEFAULT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 AUTO_INCREMENT=103 ;

[Дима]

Кирилл Казаков: Ну тогда я ничем помочь не могу. Вам сюда - rusrails.ru/rails-database-migrations

[Кирилл Казаков]

Дима: Благодарю за приложенные усилия !

[Кирилл Казаков]

Дима: воспользовался такой штукой db:schema:dump для содания актуальной shema.rb

[Кирилл Казаков]

Дима: На вашу последнюю версию кода ругается

syntax error, unexpected ':', expecting '='
...l: subscribe_email, subscribe: true) }

subscribe_email - откуда берется эта переменная ( на нее так же ругается, говорит, что не знает что это такое)?
В shema.rb все поля описаны.

В такой версии кода

class Subscription < ActiveRecord::Base
  before_save { self.email = email.downcase}
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
  validates :email, :presence=> true, :format=> VALID_EMAIL_REGEX, :uniqueness=> {:if=> :subscribe_on?}
  scope :subscribe_on, lambda(self.email){ where(:email=> self.email, :subscribe=> true) }
  
        
  def subscribe_on?
    #subs = Subscription.select('subscribe').where(:email=>self.email).first
    subs = self.class.subscribe_on
    logger.debug "subscribe_on subs-: #{subs}"
    subs.first
  end
end

ruby говорит, что не знает self.email

NoMethodError (undefined method `email' for #

Answer 2

[Кирилл Казаков]

Вероятно криво\косо, но задачу решил таким способом

class Subscription < ActiveRecord::Base
  before_save { self.email = email.downcase}
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
  validates :email, :presence=> true, :format=> VALID_EMAIL_REGEX, :uniqueness=> {:if=> :subscribe_on?}
  
        
  def subscribe_on?
    subs = Subscription.where(:email=>self.email)
    unless subs.size == 0
      subs = subs.first
      subs1 = subs.subscribe 
    end
    
  end
end

Работает на ruby 1.8.7

Answer 3

[Eugene Burmakin]

Если у вас модель Subscription с булевым атрибутом subscribe, то и запрашивать нужно subs.subscribe. Это должно вернуть true или false.

Comments

[Кирилл Казаков]

Такой запрос

subs = Subscription.select('subscribe').where(:email=>self.email).first

[Eugene Burmakin]

Судя по вашему запросу, вы ищете объект уже имея его в наличии (self). Попробуйте вместо subs вывести self.subscribe. Либо, если уж так делаете, можете попробовать так: subs = Subscription.find_by(email: self.email), а затем вывести булево значение: subs.subscribe.

[Кирилл Казаков]

Evgeniy Burmakin: self.email - это результат ввода инфы в форму на сайте.

Processing by SubscribtionsController#validate as JSON
  Parameters: {"subscribtion"=>{"email"=>"[e-mail]"}}

Answer 4

[Артем Пьянков]

Стоило вынести сам запрос в тело вопроса, а то не очень понятно. Из комментов я понял, что вы строите запрос через модель, так что возвращаться будут экземпляр(ы) этой модели. При этом если вы через select указываете атрибуты, которые должен вернуть запрос, то только они и будут в модели.
Допустим у вас есть модель Foo с атрибутами bar и baz.

f = Foo.select(:bar).first
# выведет bar
f.bar
# будет ошибка, т.к. этот атрибут не был загружен
f.baz

Comments

[Кирилл Казаков]

Да, верно.
Я в модели и оперирую результатом этой выборки.

validates :email, :presence=> true,
:format=> VALID_EMAIL_REGEX,
:uniqueness=> {:if=> :subs?}

Но внутри subs у меня не boolean .

[Артем Пьянков]

Кирилл Казаков: вы делаете что-то странное. Если вы хотите условную валидацию, то нужно вынести if в параметры метода validates. Если хотите добавить условие в запрос на проверку уникальности, то нужно использовать conditions (так же как параметр validates) - api.rubyonrails.org/classes/ActiveRecord/Validatio...