Как указать значения по умолчанию в модели в рельсах?

Question

[Mr Freeman]

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

Есть запрос

@conversion = Conversion.joins(:track)
    .select('SUM(conversions.revenue) as sum_revenue, COUNT(*) as count_conversions')
    .group('tracks.campaign_id')
    .where(:tracks => { :campaign_id => campaign_id })
    .order('tracks.campaign_id ASC')
    .first

Я вот хочу чтобы у меня при вызове @conversion.count_conversions не возникало undefined method `count_conversions' for nil:NilClass - а это возникает тогда, когда при агрегации у нас пуст @conversion

Я как понимаю, что могу это обойти с помощью

!@conversion.blank? ? @conversion.count_conversions.to_i : 0,

но это как-то слишком громоздко для руби и боюсь что совсем не "DRY"

От гугла вот что нашел, но все равно ошибка выходит

#Conversion model

  after_initialize :default_values

  def default_values
    self.count_conversions ||= 0
  end

# или вот такое вставляю, тоже нету смысла

  def count_conversions
    read_attribute(:count_conversions) or 0
  end

Объясните новичку, пожалуйста, в чем я запутался, чего не могу понять и как можно красиво (а я почти уверен что это можно сделать) вынести значения по умолчанию в модель. Спасибо!

Answer 1

[thepry]

От гугла вот что нашел, но все равно ошибка выходит

Да, потому, что если твой запрос не нашел ничего, то first вернет nil. А у объектов класса nil нужно тебе метода нет.
Выход может быть инициализация нового объекта:

@conversion = Conversion.joins(:track)
    .select('SUM(conversions.revenue) as sum_revenue, COUNT(*) as count_conversions')
    .group('tracks.campaign_id')
    .where(:tracks => { :campaign_id => campaign_id })
    .order('tracks.campaign_id ASC')
    .first

@conversion ||= Conversion.new

Хотя, на мой взгляд, если тебе нужно несколько раз получать значение этого метода, то да, используй под это дело дополнительную переменную.

@count_conversions = @conversion.count_conversions if @conversion

UPDATE: Или так, если важно, чтобы всегда в @count_conversions было число

@count_conversions = @conversion.try(:count_conversions).to_i .

Comments

[Mr Freeman]

Да, наверное, вы правы, лучше использовать отдельную переменную. За способ инициализации нового объекта спасибо. А if @conversion помоему не подходит, там нужно present? помоему

[thepry]

vladamir: нет, present? там не обязателен. Если @conversion == nil, то операция не будет выполнена. Но, возможно, будет лучше так @count_conversions = @conversion.try(:count_conversions).to_i . Такой код всегда вернет число, даже, если @conversion == nil.

[Mr Freeman]

thepry: хм, точно, у меня почему-то ранее не срабатывало. Да, я остановлюсь на выражении @conversion.try(:count_conversions).to_i

[Mr Freeman]

thepry: А кстати, у меня рубимайн выдает предупреждение - controller action should call one model method over then- когда много .try() - точнее много вызовов метода модели, то есть это плохая практика? Для производительности? Тогда лучше @conversion ||= Conversion.new ?

[thepry]

vladamir: try должен работать быстрее, чем инициализация нового объекта. Ruby mine советует не использовать несколько моделей и несколько методов модели, но реальность часто диктуает свои требования.

Answer 2

[vsuhachev]

@conversion.try(:count_conversions) || 0

Comments

[Mr Freeman]

Спасибо за предложение! Намного так лучше, правда все же хотелось это на уровне модели. То есть аля динамический аттрибут на уровне модели, но потом он используется после запросов, подобных COUNT(*) as count_conversions

[vsuhachev]

undefined method `count_conversions' for nil:NilClass Ы?? Вам же ясно написали что объекта у вас нету. То есть вызвать метод у несуществующего объекта все равно будет выглядеть как var.try(:func)

[Mr Freeman]

А может тогда можно как-то чтобы AR возвращал не nil:NilClass , а пустой объект модели? И тогда этот метод можно указать в модели. Или это уже совсем ужасный костыль?

[vsuhachev]

AR так не работает. Это же логично nil - нет данных. Собственно для таких случаев и придуман try.

[Mr Freeman]

Понятно! Ну я этого не знал, спасибо! Буду теперь его использовать.

[thepry]

vladamir: first_or_initialize вернет первый, или инициализирует.

Answer 3

[Eugene Burmakin]

conversions - это атрибут модели? Если да, то такое вещи прописываются в миграции, вида

t.string :conversions, default: 'some default value'

Comments

[Mr Freeman]

Это динамический атрибут, который на уровне модели, а в базе его нету. Поэтому такой вариант не подходит. Аттрибут появляется в случае агрегации COUNT(*) as count_conversions

[TSobolev]

Тут есть нюанс: если вы определите значение по умолчанию в миграции, что при вызове Model.new это не сработает.

Answer 4

[Jeiwan]

!@conversion.blank? ? @conversion.count_conversions.to_i : 0,

— в данном случае вполне себе решение, ничего плохого в нем нету. Repeat'а тут никакого нету.

Comments

[Mr Freeman]

А если мне нужно будет в этом методе еще раз обратиться к этому методу? Условие за собой придется таскать, либо новую переменную создавать