1. Ruby / Говнокод #9365

    −103

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    def main_photo
  res = nil
  pics = self.pictures.not_price.all
  if pics && pics.any?
    res = pics.first
  end
  res
end

    Это вторые рельсы :)

    Ладно он тут на 7 строчек расписал то, что решается одной, так еще вытаскивать всю коллекцию ради того, чтобы оттуда взять первый элемент - это что-то..

    Запостил: MikDiet, 09 Февраля 2012

    Комментарии (6) RSS

    • ava Lure Of Chaos 09.02.2012 13:15 # 0

      зачем вы на рубирельсы взяли пехепешнега?
      Ответить

      • ava 3.14159265 09.02.2012 13:53 # +1

        Родине нужны герои, а гражданки рожают 1Спыхапешников.
        Ответить

    • ava bugmenot 09.02.2012 13:29 # +6

      кто ж знал, что в этом вашем whitespace ссылок нет...
      pictures.not_price - бесценные пикчи?
      Ответить

    • ava MikDiet 09.02.2012 14:04 # 0

      Да мне два проектика по наследству досталось, если нравится, могу выкладывать такие, хм-хм, образчики :)
      Ответить

      • ava Vitaly44 14.02.2012 16:01 # 0

        а как вы исправили?

        res = pics.first if pics.try(:any?)
        Ответить

        • ava MikDiet 14.02.2012 20:32 # +2

          def main_photo
          pictures.not_price.first
          end

          first и так вернет nil для пустой коллекции
          Ответить

    Добавить комментарий