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

    +160.6

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if(isset($_POST['add_st']) && $_POST['add_st'] != false && $_POST['add_st'] != '')
{ 
	if($res=mysql_query("select max(ordering) from category3 where category2=".$cat)) 
		{
			$ord=mysql_fetch_row($res);
			$order=$ord[0]+1;
		}

    $_POST['add_st'] - из формы с <input type='submit' name='add_st' value='Сохранить' />

    Я уж не говорю, что $order можно было сделать одним запросом

    Запостил: pasha, 16 Марта 2010

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

    • ava guest 16.03.2010 14:13 # 0

      А просто isset($_POST['add_st']) уже не модно? И вообще это все что?
      Ответить

      • ava pasha 16.03.2010 15:29 # 0

        Это обавление товара ))
        Ответить

        • ava mrbig66 16.03.2010 17:17 # 0

          А вся мутотень с max(ordering) -- от незнания autoincrement?
          Ответить

          • ava guest 16.03.2010 20:49 # +1

            А вот, кстати, и не факт... с чего вы решили, что order будет ключом ??? А вдруг это поле сортировки... т.е. автор вначале нашёл значение текущего максимального значения сортировки, и потом возможно будет добавлять новую запись с этим значением в поле ord, т.е. не автоинктимент!
            Ответить

            • ava guest 16.03.2010 20:51 # 0

              И, кстати, писать запросы в виде
              ... WHERE category2=".$cat) небезопастно.
              лучше так ...WHERE category2='$cat' ");
              и $cat привести к целому типу предварительно...
              Ответить

              • ava guest 17.03.2010 20:15 # +1.8

                А если $cat - безопасная переменная из глубин скрипта?
                Ответить

                • ava Death 18.03.2010 03:19 # −0.8

                  А если нет??? Люблю используя такие запросы убивать всю базу :)
                  'DELETE FROM table WHERE cat='.$cat
                  где $cat = '100 or true' типа такого...
                  Ответить

                  • ava murz 18.03.2010 08:24 # 0

                    Но этого мы уже не узнаем :)
                    Обычно в начале функции целочисленная переменная приводится к целому типу, чтобы на каждом шагу не писать intval
                    Ответить

    • ava a282750 24.08.2021 22:16 # 0

      vanished
      Ответить

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