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

    −109

    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, 12 Февраля 2015

    Комментарии (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/Это_норма
                        Ответить
                        • А у вас лурк испорченный
                          Ответить
                          • На обычном не нашёл, только статью про первый канал нагуглил.
                            Ответить
                            • функционал, заданный на векторном пространстве
                              Ответить
              • Давайте тогда понаделаем кучу оберток над всем.. Абстракция ведь..
                Ответить

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