Цикл заканчиваеться раньше времени.

[FirstMedved]

В общем такая ситуация: делаю вывод товаров из БД, в случае вывода таблицы 3 на 4 на каждой странице исчезает по 2 товара, при увеличении на 1 строку исчезает еще один товар. Понимаю, что ошибка в цикле, а где точно найти не получается.

<?

//Кол-во товаров

$kol_tovar = 12;

//Определение страницы

if(empty($_GET["page"])){
        $page = 1;
    } else {
if(!is_numeric($_GET["page"])) die("Неправильный формат номера страницы!");
        $page = $_GET["page"];
    }
$start = $page*$kol_tovar-$kol_tovar;
 /* Создаем соединение */
mysql_connect($hostname, $username, $password) or die ("Не могу создать соединение");

/* Выбираем базу данных. Если произойдет ошибка - вывести ее */
mysql_select_db($dbName) or die (mysql_error());

/* Составляем запрос для извлечения данных из полей "articul, photo, name, opisanie, price таблицы "$table", в нужной странице */
$query = "SELECT id, articul, photo, name, opisanie, price FROM $table LIMIT $start, $kol_tovar";

/* Выполняем запрос. Если произойдет ошибка - вывести ее. */
$res = mysql_query($query) or die(mysql_error());
// Подсчёт страниц
$count = mysql_query("SELECT COUNT(*) FROM $table") or die(mysql_error());
$count = mysql_fetch_array($count);
$count = $count[0];
$pages = ceil($count/$kol_tovar);

echo ('<table width="100%" border="0" cellspacing="30" cellpadding="0">');

//цикл ввывода строк
for ($strok=0; $strok<3; $strok++) {
   echo ('<tr>');
   $stolb= 0 ;
   //Цикл вывода сталбцов
  while ($row = mysql_fetch_array($res) and $stolb<4) {
   echo ('<td>');
    /*Здесь таблица товара*/
  echo ('</td>'); 
  $stolb++;
  }
  echo ('</tr>');
}
echo ('</table>');
echo ('<table border="0" cellspacing="30" cellpadding="0" style="margin-right:10px" style="position:absolute; right:40% top:40%;"><tr>');
if($page>1){
 for($i=1;$i<$page;$i++){
  echo ('<td>');
  echo ('<A HREF="?p=catalog&page='.($i).'"><font face="impact" size="2" color="#2A2A29">'.$i.'  </A></font>');
  echo ('</td>');
  }
}
echo ('<td><b>'.$page.'  </b></td>');
if($pages>$page){
 for($i=$page+1;$i<=$pages;$i++){
  echo ('<td>');
  echo ('<A HREF="?p=catalog&page='.($i).'"><font face="impact" size="2" color="#2A2A29">'.$i.'  </A></font>');
  echo ('</td>');
  }
}
echo ('</tr></table>');
mysql_close();

P.S.

Я новичок т.ч. не судите строго.

Заранее благодарю :-)

[Int]

if(empty($_GET["page"])){
        $page = 1;
    } else {
if(!is_numeric($_GET["page"])) die("Неправильный формат номера страницы!");
        $page = $_GET["page"];
    }

Если $page меньше нуля, зря только дёргаете базу. Если $page больше числа страниц, опять зря дёргаете базу, да ещё на выходе получите пустую страницу.

$count = mysql_fetch_array($count);
$count = $count[0];

Можно сделать короче через mysql_result()

Попробуем упростить цикл

$str=0;
$stl=0;
echo ('<tr>');
while ($row = mysql_fetch_array($res))
{
	if (!(++$stl % 4))
	{
		echo '</tr><tr>';
		$stl=0;
	}
	echo ('<td>'./**/$row[1]/**/.'</td>'); 
	if (++$str == 3) break;
}
echo ('</tr>');

Но вроде не должно ничего исчезать, смотри на всякий случай, что пришло из базы. И ещё погляди, создаются ли ячейки таблицы, которые у тебя пустые.

[Быколай]

стоит $page привести к нужному виду, как-то так примерно:

$page = 1;
if(isset($_GET['page'])
 $page = (int) $_GET['page'];

затем гуглить "php SELECT FOUND_ROWS() пагинация"

[Int]

$page = 1;
if(isset($_GET['page'] && (int)$_GET['page'] > 0)
 $page = (int) $_GET['page']

[Veseloff]

Капец вообще. Я бы за такой код выгнал с работы пинком под зад, чесслово. И не из-за того, что сам код плохой — это лечится, а из-за

id, articul, photo, name, opisanie, price

Что помешало назвать поля «id», «mark», «photo», «name», «description», «price»? Это первое.

Второе, как правильно заметил Быколай, подстчёт записей надо делать через SQL_CALC_FOUND_ROWS.

Третье. Если количество товаров определено и надо его вывести в таблицу4?3, то надо делать что-то типа такого

<table>
<tbody>
<tr>
<?php
$q=mysql_query("SELECT * FROM `tablename` LIMIT 0, 12");
for ($s=0; $r=mysql_fetch_assoc($q); $s++)
{
if ($s%4==0 && $s!=0) echo '</tr><tr>';
echo '<td>'.$r['name'].'</td>';
}
?>
</tr>
</tbody>
</table>

Ну и в конце «допинать» пустыми <td>, если товаров меньше 12. И всё. А вы какой-то жути нагородили.

[rus]

COUNT() будет быстрее чем SQL_CALC_FOUND_ROWS().

ф-ция постранички:

function show_page_list ($page, $num, $count_on_page)
{
	echo "<div class=\"navi_box\"><ul class=\"pagi_navi\">";
	if (!isset($page) && intval($page) == 0) $page = 1;
	$count_pages = intval($num / $count_on_page);
	$ostatok = $num % $count_on_page;
	if($ostatok > 0) $count_pages++;
	if ($page>1)
	echo "<li><a href = \"http://". $_SERVER['HTTP_HOST'] ."/".($page - 1)."/\" title = \"Предыдущая страница\">←</a></li>";
	$diapazon = 10;
	$page_from = $page - $diapazon; 
	if($page_from < 1) $page_from = 1;
	$page_to = $page + $diapazon; 
	if($page_to > $count_pages) $page_to = $count_pages;

	for ($i = $page_from; $i <= $page_to; $i++)
	{
		if ($i == $page) echo "<li>".$i."</li>";
		else
		echo "<li><a href=\"http://".$_SERVER['HTTP_HOST']."/".$i."/\" title = \"".$i." страница\">".$i."</a></li>";
	}
	if ($page<$count_pages)
	echo "<li><a href = \"http://". $_SERVER['HTTP_HOST'] ."/".($page + 1)."/\" title = \"Следующая страница\">→</a></li>";
	echo "</ul></div>";
}

вывод:

// узнаем количество категорий
$razdel = mysql_query("SELECT * FROM `pages`");
$num = mysql_num_rows ($razdel);
// используем ф-ию для постраничной навигации
@$start = page_list($_GET['page'], $num, 2);
$res = mysql_query("SELECT * FROM `pages` WHERE ORDER by `id` ASC LIMIT $start, 2");
while ($row = mysql_fetch_array($res, MYSQL_ASSOC))
{
echo "выводим контент";
}
// выводим постраничную навигацию
@show_page_list($_GET['page'], $num, 2);

если не ошибся, то так вот как-то...

[FirstMedved]

Даже не ожидал количества ответов =) Ну как Вы поняли из кода, Я совсем чайнеГ. По этому несколько вопросов

Всё таки как лучше так:

$str=0;
$stl=0;
echo ('<tr>');
while ($row = mysql_fetch_array($res))
{
        if (!(++$stl % 4))
        {
                echo '</tr><tr>';
                $stl=0;
        }
        echo ('<td>'./**/$row[1]/**/.'</td>'); 
        if (++$str == 3) break;
}
echo ('</tr>');

Или так(Опробовать сейчас не могу, но если Я правильно понимаю, то в этом варианте будет не таблица 3 на 4, а 1 на12, Я ошибся?):

<table>
<tbody>
<tr>
<?php
$q=mysql_query("SELECT * FROM `tablename` LIMIT 0, 12");
for ($s=0; $r=mysql_fetch_assoc($q); $s++)
{
if ($s%4==0 && $s!=0) echo '</tr><tr>';
echo '<td>'.$r['name'].'</td>';
}
?>
</tr>
</tbody>
</table>

Хотя если Я правильно понимаю варианты отличаются только циклами?

А можно функцию постранички с комментариями для полных чайников?

Я бы за такой код выгнал с работы пинком под зад, чесслово. И не из-за того, что сам код плохой — это лечится, а из-за

Это тоже лечиться

Что помешало назвать поля «id», «mark», «photo», «name», «description», «price»?

Знания английского, в следствии мне проще воспринимать транслитом... На крайний случай это можно легко исправить =)

Edited April 14, 2011 by FirstMedved

[Быколай]

я бы советовал избекать embedded-синтаксиса, в серьёзных проектах это будет не айс.

в первом премере что не даёт вам сделать

print '<table>';

?

кстати, скобки в для echo() не обязательны.

Знания английского, в следствии мне проще воспринимать транслитом... На крайний случай это можно легко исправить =)

лучше сразу делать чем потом исправлять.

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

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

[FirstMedved]

я бы советовал избекать embedded-синтаксиса

А это что такое?

с художественным английским у меня хуже чем с "программерским".

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

Исправлюсь, Я ведь только учусь =)

[Быколай]

А это что такое?

http://php.net/manual/ru/language.basic-syntax.phpmode.php

[Veseloff]

COUNT() будет быстрее чем SQL_CALC_FOUND_ROWS().

Гм, и правда. Причём когда-то быстрее, а когда-то нет. Надо будет занться выяснением.

[FirstMedved]

Всем благодарность за помощь =)

Если кому интересно ошибка была в условиях цикла надо так while ($row = $stolb<4 and mysql_fetch_array($res)) а было while ($row = mysql_fetch_array($res) and $stolb<4)...

PS

Конечно исправлю на вариант предложенный вами, т.к. на сколько Я понимаю так нагрузка меньше... Или Я ошибаюсь?

[Veseloff]

Для выполнения данной задачи нужен счётчик, а значит for. Почему while-то?

И не оффтопа ради, но грамматики для советую автору ознакомиться со следующим ресурсом

[Int]

while ($row = $stolb<4 and mysql_fetch_array($res))

о-О_о-О_о_О-о А расскажи лигику работы этого всего, а то я чё-то не сразу догнал. $row булевого типа? А куда же тогда данные попадают ил fetch_array?

[FirstMedved]

Честно говоря Я сам логику не уловил... Но попробовал... Работает %)

Для выполнения данной задачи нужен счётчик, а значит for. Почему while-то?

А почему нет?