Яuбy / Говнокод #17630 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
def self.get_fio(l_name, f_name, s_name)
  "#{l_name} #{f_name} #{s_name}"
end

В модели User

Запостил: rezerbit rezerbit, (Updated )

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

    • Почему бы не сделать просто конкатенацию, без метода? Зачем этот метод? Зачем лишняя абстракция?

      А лучше чтобы был не статический метод, а метод экземпляра:

      def get_fio
      "#{l_name} #{f_name} #{s_name}"
      end
      Ответить
          • > Зачем этот метод? Зачем лишняя абстракция?
            Именно поэтому этот код и появился здесь.
            Ответить
            • Абстракция должна упрощать код. Иначе это плохая абстракция =)

              Т.е. вы считаете, что User.get_fio(l_name, f_name, s_name)
              лучше читается, чем @user.get_fio ?

              По сути это даже не абстракция..
              Ответить
              • Никто и не спорит, что предложенный выше тобою вариант является правильным и более чем адекватным (и вполне кстати очевидным).
                Просто меня несколько удивило то, что оригинальный код вызвал у тебя столько вопросов, учитывая тот факт, что мы сейчас сидим на govnokod.ru =)
                Ответить
                • > Просто меня несколько удивило то, что оригинальный код вызвал у тебя столько вопросов, учитывая тот факт, что мы сейчас сидим на govnokod.ru =)
                  Просто меня несколько удивило то, что оригинальный код вызвал у тебя столько вопросов, учитывая тот факт, что ты сам его запостил =)
                  Ответить
                  • Меня удивил факт, что кто-то считает это нормальным =)

                    (Комментарий guest)
                    Ответить
                    • А вот не факт, что он так считает.
                      Нормы разные бывают. lukomore.org/lurk/Это_норма
                      Ответить
            • Давайте тогда понаделаем кучу оберток над всем.. Абстракция ведь..
              Ответить

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

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?


    8