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

    +23.5

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    // Проверяет, есть ли у страницы вложенные страницы
function exists_sub_pages($page_id)
{
$select_sub_pages=mysql_query("select * from pages where parent='".$page_id."'");
if (mysql_num_rows($select_sub_pages)>0) {$result=1;} else {$result=0;}
return $result;
}

    Не понимаю, зачем так всё усложнять :-(

    Запостил: guest, 09 Января 2009

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

    • ava guest 09.01.2009 20:40 # 0

      USED:
      Ну вполне нужная функция, если например в цикле нужно проверить кучу вложенстей..
      Ответить

    • ava guest 10.01.2009 08:27 # 0

      страйко:
      [quote=USED]
      Ну вполне нужная функция, если например в цикле нужно проверить кучу вложенстей..
      [/quote]
      получить + n запросов на эту кучу и убить сервер БД. Да, вполне нужная :)
      Ответить

    • ava guest 10.01.2009 10:25 # 0

      USED:
      страйко, предложите лучший вариант как проверить N вложеных страниц, получая их $page_id по ходу выполнения скрипта?
      Ответить

    • ava guest 10.01.2009 12:26 # 0

      Функция то нужная, но написано очень коряво..

      Во-первых, нафига выбирается вся запись из базы, если нужно только узнать кол-во подходящих записей??
      Вместо SELECT *, делаем SELECT COUNT(*)

      И зачем вводить еще одну переменную?

      Проще ли сразу написать
      return mysql_num_rows($select_sub_pages)>0
      Ответить

    • ava guest 22.01.2009 01:08 # 0

      Stam:
      function count_sub_pages($page_id) {
  return mysql_result(mysql_query("select count(*) from pages where parent='".$page_id."'"), 0);
}

function exists_sub_pages($page_id) {
  return count_sub_pages($page_id) > 0;
}

      ?
      Ответить

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