Привет парни. Подскажите, такой метод реализации содержит в себе какие-то ошибки? Стыдно, но первый раз за столько лет использую цикл while с функцией внутри. Грамматика вообще правильная?
Суть: Генерируем URL (в примере сделал элементарную генерацию), проверяем в БД, если копии нет, то ОК, если найдена копия, то выполняем дальше, пока не получим уникальный URL. Я старался сделать самую быструю версию, уж проще по моему нельзя.

function check_url_copy($url)
{
  $query = "SELECT `id` FROM `pages` WHERE `url` = '".$url."' LIMIT 1";
  if(mysql_num_rows(mysql_query($query)) > 0)
  {
    return false;
  }
  else
  {
    return true;
  }
}


$url = rand(1,9); //URL для примера упрощены


while(!check_url_copy($url))
{
	$url = rand(1,9);
}

echo $url;

_____________
..Работает - не трогай!

Нормально.
Проблемы две.
1: открыта SQL-inj
2: как только в базе будет 9 url, цикл будет крутиться бесконечно.

лично я для циклов без конкретного числа итераций ставлю максимально-допустимое количество витков, после которого выкидываю исключение, типа: "не удалось сгенерировать url"

_____________
[продано копирайтерам]

Лучше ищите в база данных сразу с использованием in

$query = "SELECT `id` FROM `pages` WHERE `url` in (".implode(',', $url).") LIMIT 1";

$query = "SELECT `id` FROM `pages` WHERE `url` in (".implode(',', $url).") LIMIT 1";

$query = "SELECT `id` FROM `pages` WHERE `url` in (".implode(',', $url).") LIMIT 1";

in (".implode(',', $url).") 
+
LIMIT 1

ну и в догонку:

Тогда и это

  if(mysql_num_rows(mysql_query($query)) > 0)
  {
    return false;
  }
  else
  {
    return true;
  }

Замени на это

		  return !mysql_num_rows(mysql_query($query)) > 0;

return !mysql_num_rows(mysql_query($query))