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

    +2

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    Searching 43077 files for "PROPERTY_96" 
    
    D:\OpenServer\domains\aproject.local\public_html\bitrix\templates\aproject\components\aproject\catalog.section\list\template.php: 
    19 // говнокод, говнокодом не испортить :) 
    20 $itSitno = 0; 
    21: if ($arElement['PROPERTY_96'] > 0) 
    22 { 
    23 $itSitno = 1;

    замечательный проект.. Искал тут свойство в интернет магазине (типа макдака), которое отвечает за тип добавки к заказу (соус, мясо, и т.п.).
    Называется это свойство вот так: PROPERTY_96 встречается в нескольких местах, в самом проекте 43077 файлов.
    вообщем у меня бомбануло через 30 минут когда я его таки смог найти.

    да это битрикс :(

    Запостил: hell, 27 Ноября 2015

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

    • 43077 файлов? Это нормально?
      Ответить
      • ну если проект большой то да. на каждый класс по файлу. Представления страниц в отдельных файлах. Виджеты различные . Я сейчас работаю с проектом где около 200 различных виджетов, что то порядка 1к файлов.
        Ответить
        • У меня пока 79 файлов. Штук 15 из них это scss. Роутов около 10 всего во всем магазине. Шаблонов пока 8. 2 общих (layout, contentAndSidebar) и по 2 на три роута.
          Ответить
          • >Роутов около 10
            Контроллеры ?
            Ответить
            • У меня там не совсем контроллеры. Почитал описания паттернов, получается что-то вроде MVVM
              Ответить
              • Слушай, ну какие MVVM в PHP?
                Какие вообще паттерны в PHP кроме GodObject?
                Ответить
                • А я хз. Сделаю, залью на гитхаб и покажу
                  Ответить
                  • ну сделай, сделай)

                    то что пхпист умеет гитхаб -- это уже само по себе сомнительно что-то
                    Ответить
                  • так ты сейчас залей. будешь пушить код.
                    Ответить
                    • Пхпшники сначала пишут код, а потом, когда он уже работает, коммитят его в гит. Промежуточные коммиты для лохов.
                      Ответить
                      • это что бы никто не подумал, что они не умеют писать код
                        Ответить
                      • удобно же. Реверты не нужны, мержить ветки не нужно.
                        Ответить
                      • > Промежуточные коммиты для лохов.
                        Но ведь они и правда не нужны! Кому нужен код, который настолько неполон, что даже затычки ещё не написаны.
                        Ну, разве что ради бэкапов, чтобы потом, запилив первую нормальную версию, удалить их.
                        Ответить
                        • Js ники тоже ьак делают?
                          Ответить
                        • будете смеяться, а я чаще всего делаю amend commit, и потом только уже push :]
                          Ответить
                        • git rebase -i
                          git merge --squash
                          Ответить
                          • >rebase -i
                            Вот, сразу видна рука мастера. Аналогично не вижу никакой проблемы с количеством комитов.
                            Проблема в знании инструмента.
                            Ответить
                      • Не могу же я неработающую фиговину туда залить. Теперь работает. Разберусь с гитом (ни разу не работал) и сделаю. Ссылку сюда или отдельным говнокодом?
                        Ответить
                        • лучше отдельным.
                          >Разберусь с гитом (ни разу не работал) и сделаю
                          https://git-scm.com/book/ru/v1
                          пол часа хватит что бы разобраться.
                          Ответить
                          • Ой да не пизди что пхпшник гит осил л. Небось мердж от ребейса до сих пор не отличишь
                            Ответить
                        • https://xkcd.com/1597/
                          Ответить
                          • Именно так ) Сделал по туториалу
                            Ответить
                          • не хуа не понял о чем они ?
                            Ответить
                            • тут чтобы понять нужно гит знать, и английский еще
                              так что не парься, это не про тебя)
                              Ответить
      • npm install
        Ответить
    • Ради развлечения читаю документацию битрикса. Особенно радуют отзывы разработчиков в эпиграфах глав.
      Ответить
    • супер.
      из собственного опыта: недавно пришлось работать над проектом, где была сторонняя разработка (на джаве, но без исходников и хоть какой документации), которую надо было расширять написанием собственных однометодных классов. все знания передавались исключительно из уст в уста.
      итог: через месяц я не выдержал и положил большой и толстый.

      вижу параллель с моим случаем. вопрос: вы готовы насиловать себя каждый день, или хотите применять свой опыт и знания там, где они будут эффективны?
      Ответить
      • если хорошо заплатят, почему-бы и не поработать
        Ответить
      • хм, ну дома я вижу/стараюсь писать чистый и красивый код, там "проект мечты" который я в составе команды из пары человек пишем уже больше года, так что с моральным здоровьем у меня все в порядке =)

        а на работе приходится терпеть такое говнецо.
        Ответить
    • В русском языке нет слова «вообщем».
      Ответить
      • мы тут таких как ты не любим...
        Ответить
        • по чему вы громотеев не любете?
          Ответить
          • патамушдо.
            Ответить
          • грамотей -- не человек
            где человек разумный всё понимает, пользуясь здравым смыслом и гибкостью мышления, грамотей падает с parsing error, как тупой компьютер
            Ответить
            • Грамотей - человек разумный, который всё понимает, но после того выдаёт warning, чтобы в будущем его оппонент избежал ошибки. Это повышает всеобщий уровень грамотности, а также способствует более простой читаемости в будущем.
              Ответить
            • Если оригинал восстанавливается однозначно, parsing error не возникает.

              А в данном примере вилка: мы не знаем, имелось в виду «вообще» или «в общем». Когда автор сам не уверен, что хотел написать, это заставляет задуматься над тем, насколько внимательно он относится к алгоритмам.
              Ответить
        • Я как-то о Вашей любви и не мечтал. Вижу говнокод — пусть и по-русски — считаю своим долгом пнуть.
          Ответить
        • Неужели ты с Хабрахабра?
          Ответить
      • Скажу больше, Викисловарь даже не знает языка, в котором оно есть.

        Может быть, это творительный падеж субстантивированного слова «вообще»? Но я не знаю, какой смысл в субстантивации наречий и вводных слов. Разве что кто-то получил прозвище Вообще, но тогда из уважения его надо писа́ть с большой буквы.
        Ответить

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