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

    +161

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    <?
function save($author,$title,$pubyear,$price) {
        $sql = "INSERT INTO catalog(author,title,pubyear,price) 
                        VALUES('$author','$title',$pubyear,$price)";
        mysql_query($sql) or die (mysql_error());
}
 function selectAll() {
                $sql = "SELECT * FROM catalog";
                $result = mysql_query($sql) or die(mysql_error());
                return $result;
}

    Запостил: qbasic, 17 Марта 2011

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

    • ava telnet 17.03.2011 19:31 # 0

      http://www.youtube.com/watch?v=YYvOGPMLVDo
      Ответить

    • ava RaZeR 19.03.2011 12:39 # −3

      | $pubyear
      Pedobear?
      Ответить

    • ava Oonitaz 19.03.2011 18:10 # 0

      И хуле тут не так? Код как код, чо.
      Ответить

      • ava rO_ot 19.03.2011 18:42 # −1

        * or die гавно
        * названия функций гавно
        * SELECT без LIMIT-а говно
        * чистый SQL говно, нужно хотя бы минимальную прослойку писать, чтобы не переписывать запросы (INSERT в частности) после добавление одного поля в таблицу.
        Ответить

      • ava GhOsTMZ 19.03.2011 18:42 # −1

        1) Запросы выполняются функциями mysql_*, а не элементарнейшей обертокой или в идеале полноценным классом для работы с базой
        2) or die (mysql_error())
        Не дай боже искать косяки в запросах... поидее такие вещи должны журналироваться, а пользователь не должен видеть подобных ошибок
        3) $result = mysql_query($sql) or die(mysql_error());
        return $result;
        Помимо того, что к это применяется первые 2 пункта, так еще результат мы возвращаем как ссылку ресурс, а не чтото более похожее на сам результат запроса...

        Вообщем, работа с базой ниразу не продумана...

        Плюс к этому:
        "INSERT INTO catalog(author,title,pubyear,price) VALUES('$author','$title',$pubyear,$pric e)"
        По мне так не совсем красиво... Да и неизвестно что происходится с входящими переменными до этой функции, мб они вообще никак не обрабатываются, так что безопаность тоже под вопросом, особенно учитывая как грамотно написаны эти несколько строк...
        Ответить

        • ava bugmenot 19.03.2011 20:49 # +1

          вы так говорите, как будто выбор пхп, как инструмента - признак продуманного решения
          Ответить

        • ava Oonitaz 19.03.2011 21:24 # −5

          показать все, что скрыто1. Любые обёртки ведут за собой большее количество абстракций == больше тормозов. С дополнительными классами та же хуйня. Хотя mysql_ тут внатуре неуместен, лучше mysqli_.
          2. or die — не говнокод, а классичесий ёбаный стыд, наследие видеомудака попова, пора уже привыкнуть. А запросы отлаживать надо до того как их вводить внутрь скрипта.
          3. Ну хуй его знает зачем ему это надо. Слишком мало кода есть на обозрение. Возможно, где-то есть что-то, которое бы объяснило такой подход.

          Короче, это нихуя не говнокод, а типичная разработка, которую делают 90% кодеров на пхп, которые, блядь, думают что они ниибаца крутые программеры, а на самом деле хуёвые кодеры и не более.
          Ответить

          • ava GhOsTMZ 21.03.2011 11:20 # 0

            1. Тормозов больше, но и функционал тоже, который может позволять отлавливать ошибки, вести какуюто статистику по тем же запросам и так далее
            2. Отлаживать нужно, спору нет, но все мы люди и все мы допускаем ошибки, так что перестраховка лишней не будет.
            3. Возможно, но подход не совсем стандартен, да и его можно было бы объяснить, если бы как раз эта функция была бы некой оберткой
            Ответить

          • ava rO_ot 22.03.2011 23:14 # 0

            без обёрток и получается подобное гавно
            Ответить

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