1. SQL / Говнокод #5099

    −183

    1. 1
    2. 2
    3. 3
    $sql = "DELETE  FROM ".PREFIX."_orders
                WHERE id = '".$order_id."'
    	    ORDER BY order_date DESC";

    Запостил: 1_and_0, 28 Декабря 2010

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

    • копипаст из селекта?
      Ответить
      • Возможно, а возможно там дальше еще делается $sql .= " LIMIT 1";
        Хотя, в любом случае, хорошим такой код не назовёшь.

        P.S. чтобы не было вопросов: тут не указано, что id - это уникальный индекс
        Ответить
        • это PHP
          а значит MySQL
          следовательно автору молотком вбили про синтетический ключ id

          DELETE ... ORDER BY i`lol`d
          Ответить
          • > это PHP

            следовательно[color=red],[/red] автор сего вообще не дал труда себе разобраться в чем либо. Так, скопировал код с форума, подправил. Залил на продакшн, убедился, что не кидает fatal error, и доволен
            Ответить
        • для некоторых - первая колонка должна называться id
          Ответить
          • а, дада, еще следы молотка на "обязательно первой колонкой"
            Ответить
          • В любом случае оторвано от контекста это не настолько говнисто, как могло бы быть.
            Может включен register_globals, и это например колбек на удаление по аджаксу, с кодом:
            <?php 
            include 'config.inc';
            mysql_connect($dbadres, $dbluzer, $dbporol) or die("Мне урезали зарплату! Вася.");
            mysql_selectdb($dbimya) or die("Куда дели базу?");
            $sql = "DELETE  FROM ".PREFIX."_orders
                        WHERE id = '".$order_id."'
            	    ORDER BY order_date DESC";
            mysql_query($sql) or die('Умри хакер');
            echo '{"success": true}';
            ?>


            Реквестирую кусочек побольше
            Ответить
            • вот за такой аякс нужно точно урезать зарплату. Бо удаляй не хочу кто хошь и что хошь
              Ответить
              • sucksass: true
                поубивал бы за "ajax"
                а за JSON где не надо в сортир
                Ответить
              • Я просто показал, что оторванный от контекста код может не показывать степень своей говнистости)
                За такой код надо как минимум увольнять )
                Ответить
            • > or die('Умри хакер');
              скрипткидди щаслив
              Ответить
    • забавно, лимит бы все объяснил конечно...
      Ответить
    • Где здесь С++?
      Ответить
    • Я админ этого сайта. Я вас всех ща забаню


      заведу и отибу.
      Ответить
    • Где здесь C++, CPlusPlus?!
      Ответить

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