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

    +167.4

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if (isset($_POST["yes"]))
    {
	 $query="DELETE FROM zakaz WHERE (id_tovar='$id' and id_users='$id_u)";
	 add($query);
	 commit();
         Echo "<b> Заказ утвержден";
   }

    Знакомая писала курсач по СУБД + PHP, мало что понимала в том что пишет) увидел в одном из фалов)

    Запостил: 1_and_0, 26 Января 2010

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

    • ava nolka4 26.01.2010 06:24 # −14.2

      показать все, что скрытонармалек, хоть С++ здесь и не пахнет :D
      Ответить

      • ava 1_and_0 26.01.2010 06:26 # 0

        а причем здесь С++?)) раздел PHP ведь, и код PHP + SQL =)
        Ответить

        • ava guest 26.01.2010 08:34 # 0

          не обращай внимания
          это местный тролль
          Ответить

    • ava guest 26.01.2010 09:04 # +2

      Ткни пальцев в говнокод.

      Не видя предварительного кода не понять. Может это удаляется совсем не заказ, а что-то другое. Логические ошибки не есть говнокод.
      Ответить

    • ava ivmaroz 26.01.2010 12:15 # +1

      А что Вы хотели от студентки - какое-то гениальное решение?
      Ответить

    • ava ivmaroz 26.01.2010 12:19 # +5

      Нашел. Кавычка не закрыта в SQL. В этом говнокод?
      Ответить

      • ava Ctac0k 26.01.2010 13:55 # 0

        Ты раскрыл коварный план :(
        Ответить

      • ava ivmaroz 27.01.2010 14:41 # +1

        Ну если увидели говнокод в очепятке, то хотя бы сами пишите грамотно :)

        > увидел в одном из фалов
        Ответить

        • ava Pencioner 31.01.2010 03:26 # 0

          тэкс, входит Зигмунд Фрейд.
          в одном из фалосов должно быть, да? или какая опечатка? :)
          Ответить

    • ava noisiaman 27.01.2010 13:19 # 0

      Echo
      Ответить

    • ava mrbig66 27.01.2010 17:12 # +1

      Логика просто реализована жопой на перёд.
      Получается, что все заказы в таблице находятся в каком-то промежуточном состоянии, а при утверждении заказа, вместо того, чтобы поставить статус "архив" он удаляется.

      Хотя скорее всего просто перед этим создается/обновляется какой-либо лог-файл, а удаление для того, чтобы ДБ не загружать.

      Вообще в любой системе учета товаров/заявок и т.д. удаление -- крайний метод, в остальных случаях все должно валиться либо в архив, либо в аналог корзины. При грамотном написании запросов и их реализации в скриптах можно спокойно обрабатывать миллионы записей. И наоборот, при криворуком написании система может тупить при тысяче записей. Например вложенные циклы, в каждом из которых обрабатывается отдельный запрос к БД.
      Ответить

      • ava 1_and_0 28.01.2010 18:05 # 0

        Именно!!))
        Вот именно то, что логика здесь реализована так, как реализованная, это я и хотел показать!) А все начали искать недостающие кавычки и тому подобные вещи!)
        Спасибо!)
        Ответить

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