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

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if(!mysql_select_db($db_name,$data)){
     print "<BR>";
     echo mysql_error();
     die();
    }

    Нашел в одном из движков онлайн игры

    Запостил: Vasiliy, 30 Мая 2010

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

    • ava guest 30.05.2010 11:37 # −2

      да затчка как нибудь небось
      Ответить

    • ava This is obvious 30.05.2010 12:35 # −1

      и где говнокод ? лол
      Ответить

      • ava HyperGeek 30.05.2010 12:39 # +1

        Может забыли exit() после die() сделать?
        Ответить

      • ava funny_rabbit 30.05.2010 12:40 # +4

        Кэп, что с вами?) Очевидно, автор намекает, что все выводимое можно было запихнуть в die(); не используя принтов и эхов, мол короче было бы. Но да, экземпляр ГК уныл и невесел.
        Ответить

        • ava turdman 31.05.2010 10:44 # +1

          Намного короче
          mysql_select_db($db_name,$data) || die ('<br>' . mysql_error());
          Да еще '<br>' (лучше уж выводить text/plain, чем такие огрызки)
          Ответить

    • ava guest 30.05.2010 13:51 # 0

      Надо было 
      print "<BRrrrrr>";
      Ответить

    • ava guest 30.05.2010 16:22 # −8

      показать все, что скрытоопять захардкоженный таг, да еще и в устаревшем HTML 4.0.

      Все таки PHPшники все имбицилы
      Ответить

      • ava funny_rabbit 30.05.2010 16:50 # +11

        Вообще переходить на прямые оскорбления - это не правильно, но выбора не остается. Ты либо толль, но очень толстый тролль, либо мудак. Мудак ты т.к. :
        1) что бы высказать свое авторитетное мнение мог бы и залогиниться, что бы все PHPшники знали героя в лицо
        2) вменяемые люди понимают, что язык программирования сам по себе ничего не пишет. Это делают люди. Некоторые хорошо, некоторые плохо. То, что у языка порог вхождения низкий к минусам отнести можно с боооОльшой натяжкой. Но ты этого, как и положено мудаку, не понимаешь, а тупо срешь.
        Ответить

      • ava mrbig66 31.05.2010 08:24 # 0

        >опять захардкоженный таг, да еще и в устаревшем HTML 4.0.

        Интересно, какое отношение имеет тэг "<BR>" к РНР?
        Ответить

        • ava funny_rabbit 31.05.2010 10:34 # 0

          Наверное, подразумевалось разделение логики и представления, мол <BR> должен быть в шаблоне или где там.
          Ответить

          • ava guest 31.05.2010 12:08 # 0

            Нахера нужно такое разделение и таскать с собой шаблоны и прочую чепуху, если, фактически, нужно вывести текст, разово. При этом, возможно, никто никогда его и не увидит.
            Ответить

            • ava funny_rabbit 31.05.2010 12:33 # −1

              Сие мне неведомо, но товарищу с захардкоженным тагом не понрвилось, подазреваю, именно это
              Ответить

              • ava guest 02.06.2010 03:12 # −2

                Ну вот видите -- нормальным программистам очевидно, что ошибки обрабатываются эксепшенами или возвращением еррор кодов, что бы вызывающая программа могла записать их в лог, могла проверить -- вызвал ли код ошибку (в тесте) итд.

                Но PHPшникам это не ведомо. Они пишут свое тупое 'echo "<BR>"' где угодно: от бизнес-логики до кода доступа к данным.

                Потому любой PHPный проект проще выкинуть, чем довести до ума.
                Ответить

                • ava guest 02.06.2010 03:14 # 0

                  > могла проверить -- вызвал ли код ошибку (в тесте)

                  говнокодеры тестов не пишут. что насчёт пхпистов?
                  Ответить

                • ava funny_rabbit 03.06.2010 14:00 # 0

                  Госпади, опять что ли повторяться, что язык он ничего не кодит, кодят на нем?!..."ошибки обрабатываются эксепшенами или возвращением еррор кодов, что бы вызывающая программа могла записать их в лог, могла проверить -- вызвал ли код ошибку (в тесте) итд" - да, это все правильно и справедливо и МВЦ - это чудесный шаблон проектирования и на голом пхп писать что-то сложнее гостивухи из 2 файлов как-то не айс. Только нафига бросаться каллом во всех, если конкретный человек пишет говно by design. Среди "разработчиков" на любом языке хватает тех, кто слово "патерн" только слышал
                  Ответить

            • ava turdman 31.05.2010 12:37 # 0

              Ненахера. Нормальная функция вывода ошибок, ставящая http status и выводящая нормальную страницу (даже hard coded), это уже достаточное разделение, чтобы скрипт (масштаба гостевая-из-двух-фаилов) выглядел по человечески.
              Ответить

    • ava Lure Of Chaos 31.05.2010 19:25 # 0

      а зачем вообще в ответ пихать описание ошибки? все должно логгиться в лог, а в ответ красивое сообщение, понятное конечному пользователю, вроде:

      нашу систему ГовноПро(тм) поленились установить! не возвращайтесь сюда, пока тупой админ не догадается запустить сетуп!
      Ответить

      • ava guest 31.05.2010 19:29 # 0

        ага, надо понятное конечному пользователю сообщение, понятное конченому админу сообщение, и инфа чё делать - паниковать и звонить в саппорт или просто перезагрузить страницу через 5 минут.

        бесят сука "Unknown error" в некоторых поделиях
        Ответить

        • ava Lure Of Chaos 03.06.2010 15:10 # 0

          ну что поделать, если прогеров какой-нить "access denied: invalid user or password" вводит в ступор и для них это "Unknown error"
          Ответить

      • ava guest 02.06.2010 03:17 # −1

        Потому что это PHP.
        тут нет понятия "лог", "админ" и прочих атрибутов серьезного софта.
        Тут есть школьник, который тотал коммандером по FTP зашел на сайт, там нажал F4, и в открывшемся EditPlus прямо на живом сайте что-то поправил.

        Если сайт не открывается -- ему звонят.
        Зачем ему логи, если он сам этот сайт писал?

        Ошибки надо отдавать прямо в аутпут, на глаза удивленному пользователю.
        Поищите в гугле "Warning: mysql_connect() ", и увидите тысячи сайтов с ошибками типа

        Warning: mysql_connect(): access denied for vpupik in
        /home/v/vpupkinsite/www.myfuckingsite.ru/gostevaja_kniga.php: 1952 line
        Ответить

        • ava guest 02.06.2010 03:44 # −1

          > тут нет понятия "лог", "админ" и прочих атрибутов серьезного софта.

          серьезный софт пишется на аспнете или яве?
          Ответить

          • ava Lure Of Chaos 03.06.2010 15:32 # 0

            видимо, на руби и питоне...

            а вообще, что значит "серьезный софт"? это когда маленький проект, но хорошо сделанный, или космический корабль с тоннами Ге?
            Ответить

            • ava guest 03.06.2010 18:36 # 0

              > а вообще, что значит "серьезный софт"?


              спроси у того геста, это он же сказал "тут нет понятия "лог", "админ" и прочих атрибутов серьезного софта."
              Ответить

        • ava guest 02.06.2010 03:46 # 0

          > Warning: mysql_connect(): access denied for vpupik in
          /home/v/vpupkinsite/www.myfuckingsite.ru/gostevaja_kniga.php: 1952 line

          а вот это гуд, можно изучить строение сайта
          Ответить

          • ava Lure Of Chaos 03.06.2010 15:20 # 0

            ой, это еще ладно,
            вот я хостился как-то на холме (*.h1.ru), захожу я как-то в ЧАВО, а мне в текстареа весь лог ошибки мускула вместе с запросом и пассвордами... с трудом вылез изпацтола от гоготу
            Ответить

        • ava Lure Of Chaos 03.06.2010 15:12 # 0

          ой, это еще ладно
          Ответить

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