1. PHP / Говнокод #9159

    +157

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    /**
 * Появилась из-за опечатки. См. getCurrentEnterpriseIds()
 */
public function getCurrnetEnterpriseIds($user_id) {
    return $this->getCurrentEnterpriseIds($user_id);
}

    рефакторинг? не, не слышал

    Запостил: ghost404, 18 Января 2012

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

    • ava roman-kashitsyn 18.01.2012 15:38 # 0

      небось код написан в фаре, какой уж тут рефакторинг
      Ответить

      • ava istem 18.01.2012 18:43 # 0

        в vi
        Ответить

        • ava roman-kashitsyn 19.01.2012 09:55 # +1

          :s/getCurrnetEnterpriseIds/getCurrentEnterpriseIds/g
          Ответить

        • ava Nemoden 19.01.2012 15:55 # 0

          я в vi пишу код - всё там прекрасно с рефакторингом
          Ответить

          • ava istem 19.01.2012 19:37 # −1

            Сколько бы не говорили, как бы не хвалили, но "vi", таки уёбищный редактор.
            Не могу никак забыть истории про то, как этот "vi" открывали и не могли закрыть. Приходилось исхитряться: один открывал на соседнем компе гугл - и гуглил "как закрыть vi", другой открывал ещё одну консоль, смотрел там "pid" этого "vi" и писал в коммандной строке "kill -9 {pid}"... ужоснах...
            Тем кто пишет в "vi" в двадцать первом веке - я искренне сочувствую.
            Ответить

            • ava kyzi007 19.01.2012 19:38 # 0

              IDEA IDEA IDEA
              Ответить

            • ava roman-kashitsyn 19.01.2012 19:39 # +3

              просто в юниксе принято читать мануал ПЕРЕД использованием программы.
              Ответить

              • ava istem 19.01.2012 19:43 # 0

                а как же интуитивно понятный интерфейс? или "vi" писали не люди?
                Ответить

                • ava kyzi007 19.01.2012 19:45 # 0

                  Интуитивно понятная консоль же
                  Ответить

                • ava roman-kashitsyn 20.01.2012 09:44 # 0

                  vi написали на заре развития юникса. Тем не менее, после преодоления определённого барьера, он очень эргономичен. Думаю, ни в одной IDE не получишь такого удобства работы с текстом, как в vim (да, классическому vi не хватает современных фич). Подумайте: ведь все эти красноглазые фанаты фанатеют не просто так. Такое количество поклонников должно чем-то объясняться. Причина - невероятная эргономичность после прохождения порога (+ отличная встроенная система мануалов, цветовые схемы из коробки, огромное количество отличных плагинов, встроенный скриптовый язык, возможность писать расширения на python/perl/...).
                  Ответить

                  • ava defecate-plusplus 20.01.2012 09:49 # 0

                    http://ru.wikipedia.org/wiki/HTCPCP
                    Ответить

                    • ava roman-kashitsyn 20.01.2012 09:57 # 0

                      Сомневаюсь, что наша офисная кофеварка его поддерживает, а жаль. Может есть реализация клиента для emacs... А, нет "Пока не существует ни одной известной реализации этого протокола." (q)
                      Странно, это было бы в духе emacs.
                      Ответить

                  • ava defecate-plusplus 20.01.2012 09:56 # −2

                    по теме холивара
                    когда я пришёл давным давно на нынешнюю работу, программер писал на vim под виндой кучу кода, которая имела отношение к embedded устройствам со своей хтонической ОС и компилятором, а компиляция запускалась заботливо написанными nmake/bat файлами
                    стоило мне спортировать всю сборку на gmake, все сорцы причесать и загнать в vcproj/sln и настроить компиляцию по F7/build/rebuild solution, как подсветка синтаксиса, переход по определениям/объявлениям, автодополнение и tabbed-интерфейс уютной вижуал студии (еще и заправленной вижуал ассистом) оказался на порядок удобнее вроде бы уже привычного и родного того же функционала в vim
                    такая вот история из жизни
                    зы - мне совсем не сложно из консоли ssh что то отредактировать в файлике на ходу в vi на линуксовой машине, но разрабатывать этим инструментом проекты на миллион строк - ну уж нет
                    Ответить

                    • ava roman-kashitsyn 20.01.2012 10:07 # +1

                      > разрабатывать этим инструментом проекты на миллион строк - ну уж нет
                      я и не предлагаю этого делать и сам работаю в среде разработки, которая понимает и анализирует мой код. И всё же vim очень удобен для правки текста и позволяет его преобразовывать очень быстро. Опять же можно в лёгкую открывать гигабайтные файлы (реально приходилось несколько раз) и производить по ним поиск, IDEA бы точно загнулась.
                      Ответить

                      • ava kyzi007 20.01.2012 10:39 # 0

                        Да, идее становится не очень хорошо уже на 3000 строчных файлах.
                        Ответить

                        • ava roman-kashitsyn 20.01.2012 10:42 # 0

                          У меня в принципе идея приемлемо отображает диффы для файла ~100.000 строк, правда слегка притормаживает.
                          Ответить

                          • ava kyzi007 20.01.2012 10:45 # 0

                            Хм, вероятно ас какой то для нее особо тяжелый или ноут слабоват.
                            Ответить

                  • ava istem 20.01.2012 20:35 # 0

                    Да, пожалуй соглашусь, что данный редактор, в отличии от некоторых других, похож на эдакий "конструктор - сделай сам", который может чуть ли не "предугадывать желания пользователя", при условии соответствующей "обработки напильником" (чем-то напоминает отечественный автопром).
                    В целом и общем, обсуждения vi приводят к войнам, поэтому здесь дело вкуса каждого.
                    Если у кого-то есть возможность кататься на современном автомобиле, он ни за что не поедет на "копейке", несмотря ни на какой тюнинг.
                    Ответить

            • ava eth0 19.01.2012 19:45 # +2

              Сегодня я за Капитана Галактическая Птеросня.

              "У редактора vi есть два режима работы: в одном он пищит, а в другом все портит." (q)
              "Почти каждый UNIX-нуб спрашивал своего Гуру «Как выйти из vi?»" (q)
              Ответить

            • ava Nemoden 19.01.2012 19:54 # +1

              Я пишу в vi в 21 веке, сочувствуйте мне :-)
              Ответить

              • ava istem 19.01.2012 19:56 # 0

                Скорее всего вас заставляет это делать злобный работодатель/заказчик.
                Иначе - вы отъявленный извращенец, сэр...
                Ответить

            • ava bugmenot 19.01.2012 20:33 # 0

              Ctrl+Z же!
              Ответить

    • ava TarasB 18.01.2012 15:53 # 0

      А это не рекурсия?
      Ответить

      • ava kyzi007 18.01.2012 15:56 # 0

        Currnet - Current, точняк фар или что то подобное.
        Ответить

        • ava TarasB 18.01.2012 16:02 # −2

          А, действительно. Защита от опечаток. Опечатался - всё равно получи работающую программу. Удобно. Но до Перла, который компилирует любую последовательность символов, далеко в этом плане.
          Ответить

        • ava Depriver 19.01.2012 15:58 # 0

          >Currnet
          Курнет, и дальше в фаре сидит
          Ответить

    • ava istem 18.01.2012 18:58 # +1 

      public function __call( $method, $params ) {

	$methods = get_class_methods(get_class($this));
	foreach ($methods as $m_name) {

		if ( soundex ( $m_name ) == soundex ( $method ) ) $this->$m_name($params);
	}
}
      Ответить

    • ava Lure Of Chaos 18.01.2012 22:52 # 0

      обычная практика при опечатках
      Ответить

      • ava Nemoden 19.01.2012 15:59 # +2

        Добавлять alias'ы? А исправить никак?
        P.S. не встречался с такой практикой, а если бы и встретился, точно высказал бы свое мнение по этому поводу тому, кто такое делает
        Ответить

        • ava roman-kashitsyn 19.01.2012 16:14 # +1

          В собственном хомпаге это разумеется не имеет смысла.
          А вот если ты опечатался в названии функции в какой-нибудь мега-распространённой библиотеке или другом публичном апи, с которым все линкуются, то выхода у тебя нет.
          Один раз юниксовую функцию назвали creat из экономии одной буквы, сорок лет уже все баттхёрт огребают.
          Ответить

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