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

    −166

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    class User < ActiveRecord::Base
    
  def full_name
  	user_info=User.find(self.id)
  	[user_info.name,user_info.surname].join(" ")
  end
end

    Получаем полное имя себя

    Запостил: 1101_debian, 13 Сентября 2012

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

    • ava mr.The 13.09.2012 12:29 # +6

      >>User.find(self.id)
      Это прекрасно.
      Ответить

    • ava guest 02.10.2012 11:41 # 0

      надо использовать общее решение.
      self.class.find(self.id)
      можно его еще в ActiveRecord::Base вынести

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

      • ava 1101_debian 02.10.2012 11:44 # 0

        Даже если и так, то Ваша проблема решается просто self.reload.
        Ответить

    • ava guest 09.10.2012 09:42 # +2

      def full_name
      return "#{self.name} #{self.surname}"
      end

      just perfect
      Ответить

      • ava guest 25.01.2013 23:50 # 0

        def full_name
        "#{self.name} #{self.surname}"
        end

        зачем ещё return?
        Ответить

        • ava LeshaXakir 26.01.2013 01:36 # 0

          Зачем еще self?
          Ответить

          • ava guest 05.02.2013 15:40 # +1

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

    • ava guest 09.04.2013 17:34 # 0

      а я пишу
      [name, surname].reject(&:blank?).join(" ")
      Ответить

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