$query = "Select * from `table`";
$sql = mysql_query($query, $db) or die('<p>Выборка данных не произведена. Сообщите админу код ошибки:</p>'.mysql_error());
   while ( $myrow = mysql_fetch_assoc($sql) ){
      ...
   }

+1 тема закрыта

@dmir@l
Оставь if(mysql_num_rows($sql) > 0), запись эта пригодится. Хочешь усовершенствовать, вместо * запиши все имена таблиц.

inpost
А я бы сделал так

$query = "Select * from `table`";
$sql = mysql_query($query, $db) or die('<p>Выборка данных не произведена. Сообщите админу код ошибки:</p>'.mysql_error());
$i = 0;
   while ( $myrow = mysql_fetch_assoc($sql) ){
      $i++;
      ...
   }

if($i == 0) die('Пусто');

Snus
Я бы так не делал.

linker
Сколько людей - столько мнений. Мне вариант с mysql_num_rows не нравится.

Snus
Почему?

linker
А ты подумай логически. Ты предлагаешь считать строки из запроса, не зависимо пустой результат вернула БД или нет. А потом проверку проводить. Лишний процесс. Я предложил вариант более оптимальный. Передвигаем бегунок $i только в том случае, если БД вернула не пустой результат. А дальше проверяет $i, если 0 - значит никаких данных нет.

Snus
Да, но если результат вернул 10 строк или больше? Каждая итерация цикла будет медленнее на время $i ++, что в совокупности даст задержку T(i) * n. По-моему, это несколько расточительно.

linker
Я же говорю - сколько людей - столько мнений. Можно подумать, что mysql_num_rows не затрачивает времени.

Snus
Затрачивает конечно, но только один раз не зависимо от количества строк в возвращаемом результате.

linker
Dron19
Вы реальные цифры говорите, а не с потолка!
Вот вам пример реальный

$start = microtime(1);

$query = "SELECT * FROM `doc`.`db_risk` LIMIT 50000";

$sql = mysql_query($query) or die(mysql_error());
$cnt = mysql_num_rows($sql);
	while ( $row = mysql_fetch_assoc($sql) ){
		
	}

echo '<p>'.$cnt.'<p>';
$end = microtime(1) - $start;
echo '<p>'.$end.'</p>';

$start = microtime(1);

$sql = mysql_query($query) or die(mysql_error());
$i = 0;
	while ( $row = mysql_fetch_assoc($sql) ){
		$i++;
	}


echo '<p>'.$i.'</p>';
$end = microtime(1) - $start;
echo '<p>'.$end.'</p>';

Вот результат

50000

0.18717694282532

50000

0.18670797348022

Здесь еще есть один плюс - ты знаешь количество возвращенных строк еще до входа в цикл.

Вот тоже самое только LIMIT 100

100

0.00047802925109863

100

0.00037693977355957

linker
А зачем?

Интересно, а зачем ты цикл сделал проверяя mysql_num_rows, она тебе и так вернет результат, а во втором случае ты без него не посчитаешь? Скажи зачем ?

Dron19
М-да )

Dron19
Автору нужно было вывести данные из БД, но в случае, если таблица пуста - вывести сообщение об этом. Вот я на основании этого пример привел. А ты говоришь про подсчет строк.

у тебя же тема зашла про подсчет строк, вот я и сказал, потому что там не нужен цикл

мда)) так что посоветуете новичку??? мне с пояснениями желательно)))

так чтобы по простому, например:

запрос, если жива либо есть строки и результат тру, то в значение переменной кидаем весь результат как массив.) но так чтобы было понятно, и желательно покороче, попроще, не так как у меня)

Dron19
Вот поэтому и написал М-да. Прочитай внимательно еще раз. Вопрос был использовать

if($i == 0) die('Пусто');

или

mysql_num_rows($sql);

насколько я понимаю если мы используем mysql_num_rows($sql); то результ будет возвращать тру даже если внутри пустота?

@dmir@l
mysql_num_rows($sql) вернет значение 0

Snus я понял!

я забиваю базу пустыми данными, mysql_num_rows($sql) возвращает мне 1 строку.
1 строка с пустыми данными.

Значит условие истина, выборка пошла..))здорово

@dmir@l
Нет, mysql_num_rows($sql) считает кол-во строк из результата запроса mysql_query()

а я могу написать таким образом:

<?php 
$query = 'Select id,money oplata';
$sql = mysql_query($query,$db) or die('<p>База данных пустая!</p>');
if (mysql_num_rows($sql) > 0)
{
echo mysql_result($sql,1,2);
}
?>

@dmir@l
Конечно можешь, только работать этот кусок кода у тебя не будет.

Snus
Почему?

@dmir@l

$query = "SELECT `id`, `money` FROM `oplata`";

mysql_query($query) or die('<p>База данных пустая!</p>');

mysql_query выполняет запрос, а не проверяет на "заполненность"

посоветуйте статьи, сайт, почитать про выборку данных из БД

ок я понял! спасибо!

Snus, значит я тебя не так понял, я на своей волне был

@dmir@l
mysql.ru

Snus
Твой тест невалидный. Моя метода. Создаем два скрипта:


a.php(мой вариант)

<?php
function microtime_float()
{
    list($usec, $sec) = explode(" ", microtime());
    return ((float)$usec + (float)$sec);
}

$h = mysql_connect('localhost', '', '');
mysql_select_db('pdfio3', $h);
$r = mysql_query("SELECT * FROM `sys_users`");
$s = microtime_float();
$c = mysql_num_rows($r);
while($d = mysql_fetch_assoc($r)) {}
$e = microtime_float();
echo ($e - $s) . " sec.";

?>

и второй
b.php(твой вариант)

<?php
function microtime_float()
{
    list($usec, $sec) = explode(" ", microtime());
    return ((float)$usec + (float)$sec);
}

$h = mysql_connect('localhost', '', '');
mysql_select_db('pdfio3', $h);
$r = mysql_query("SELECT * FROM `con_costs`");
$s = microtime_float();
$i = 0;
while($d = mysql_fetch_assoc($r)) { $i ++; }
$e = microtime_float();
echo ($e - $s) . "sec";

?>

Записей в таблице почти 23 тысячи. Запускаем два скрипта по несколько раз и получаем средний

a.php - 0.0041739940643311 sec.
b.php - 0.082695007324219 sec.

как говорится почувствуй разницу.

linker
Ты забыл уточнить сколько у тебя данных в таблице sys_users и con_costs. Чтобы было справедливо делай выборку из одной и той же таблицы sys_users. Жду результатов. Потом покажу свои результаты.

По твоим "валидным" методам результаты:

test.php

<?php

function microtime_float(){
    list($usec, $sec) = explode(" ", microtime());
    return ((float)$usec + (float)$sec);
};

mysql_connect('xxx','xxx','xxx');
mysql_set_charset('utf8');

$query = "SELECT * FROM `doc`.`db_risk` LIMIT 50000";

$s = microtime_float();

$sql = mysql_query($query) or die(mysql_error());
$cnt = mysql_num_rows($sql);
	while ( $row = mysql_fetch_assoc($sql) ){

	}

echo '<p>'.$cnt.'<p>';
$e = microtime_float();
echo '<p>'.($e-$s).'</p>';

?>

50000

0.17970204353333

test2.php

<?php

function microtime_float(){
    list($usec, $sec) = explode(" ", microtime());
    return ((float)$usec + (float)$sec);
};

mysql_connect('xxx','xxx','xxx');
mysql_set_charset('utf8');

$query = "SELECT * FROM `doc`.`db_risk` LIMIT 50000";

$s = microtime_float();

$i = 0;
$sql = mysql_query($query) or die(mysql_error());
	while ( $row = mysql_fetch_assoc($sql) ){
		$i++;
	}


echo '<p>'.$i.'</p>';
$e = microtime_float();
echo '<p>'.($e-$s).'</p>';

?>

50000

0.16882801055908

Вот и думай.

linker
И твой любимый mysql_num_rows() не покажет результатов, если выполнить mysql_unbuffered_query(). А для экономии памяти иногда приходится выполнять подобные запросы. Ты все еще будешь настаивать на преимуществах mysql_num_rows() ?

Прошу прощения, там в обоих примерах `con_costs`, не то скопипастил (тестировал на двух таблицах). В ней 23 тысячи записей.

Про mysql_unbuffered_query() конечно mysql_num_rows() тут не поможет, но и тут есть свои минусы. Предлагаю провести тут опрос, кто и как часто использует mysql_unbuffered_query(), думаю результат известен заранее.

linker
Еще раз проверь свои результат. Они у тебя явно врут.

Я не понимаю, какая разница?

Снус, ты в своем начале просто создаешь в обоих случаях переменную неизвестного типа - память+время.

Цикл просчитал, и после этого, проврил результат, если нет ничего - фигня.

А значение кол-ва строк проще так проверить if (mnr == 0) die()

Все здесь упирается в написание кода.

Но все-же, твои результаты для меня - носенс.