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

    • ava Lure Of Chaos 28.12.2010 19:12 # +3

      копипаст из селекта?
      Ответить

      • ava wmmorgun 28.12.2010 19:17 # +1

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

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

        • ava bugmenot 28.12.2010 19:24 # 0

          это PHP
          а значит MySQL
          следовательно автору молотком вбили про синтетический ключ id

          DELETE ... ORDER BY i`lol`d
          Ответить

          • ava Lure Of Chaos 28.12.2010 19:33 # 0

            > это PHP

            следовательно[color=red],[/red] автор сего вообще не дал труда себе разобраться в чем либо. Так, скопировал код с форума, подправил. Залил на продакшн, убедился, что не кидает fatal error, и доволен
            Ответить

        • ava Lure Of Chaos 28.12.2010 19:24 # 0

          для некоторых - первая колонка должна называться id
          Ответить

          • ava bugmenot 28.12.2010 19:33 # 0

            а, дада, еще следы молотка на "обязательно первой колонкой"
            Ответить

          • ava wmmorgun 28.12.2010 19:40 # 0

            В любом случае оторвано от контекста это не настолько говнисто, как могло бы быть.
            Может включен 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}';
?>


            Реквестирую кусочек побольше
            Ответить

            • ava Lure Of Chaos 28.12.2010 19:51 # +1

              вот за такой аякс нужно точно урезать зарплату. Бо удаляй не хочу кто хошь и что хошь
              Ответить

              • ava bugmenot 28.12.2010 22:35 # 0

                sucksass: true
                поубивал бы за "ajax"
                а за JSON где не надо в сортир
                Ответить

              • ava wmmorgun 29.12.2010 00:18 # +1

                Я просто показал, что оторванный от контекста код может не показывать степень своей говнистости)
                За такой код надо как минимум увольнять )
                Ответить

            • ava Lure Of Chaos 30.12.2010 14:23 # +1

              > or die('Умри хакер');
              скрипткидди щаслив
              Ответить

    • ava ki6opr 30.12.2010 05:41 # +1

      забавно, лимит бы все объяснил конечно...
      Ответить

    • ava guest 01.01.2011 17:04 # 0

      Где здесь С++?
      Ответить

    • ava guest 02.01.2011 00:36 # −3

      Я админ этого сайта. Я вас всех ща забаню


      заведу и отибу.
      Ответить

    • ava guest 04.01.2011 16:13 # −4

      Где здесь C++, CPlusPlus?!
      Ответить

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