Форум PHP

Перенесите if ($i && !($i % 3)) echo '<div style="clear:both"></div>'; в самое начало цикла.

<style> .products {     list-style: none;     margin: 0;     padding: 0; } .products li {     display: inline-block;     width: 30%; /* по 3 товара в строке*/     vertical-align: top;     margin-right: 20px;     margin-bottom: 50px; }</style> <ul class="products"> <? for($i=0; $i<13; $i++) echo '<li><h4>Product</h4><img /><p>text text text text text text text text text text text text text</p></li>'; ?> </ul>

И забейте на div. Стоит только изменить ширину li в процентах, и можно выводить в строке сколько влезет в родителя.

Да стили в css это понятно, у меня почему то первые 4 элементы выводятся в одну строчку, а другие уже по 3

Пропустил, добавьте.

А что думать, если надо через три вставлять, то я написал как. Могут быть только вариации, но принцип будет тот же. Если форматирование по другому, то лучше <u><li>элементы описывающие товар</li></ul>, и стили перенести в CSS.

Хорошо буду думать как решить по другому
У Вас ошибочка
<p><a href="/shop/'.$row['hpu_cat'].'/'.$row['id'].'>'.$row['title'].'"</a></p>
двойная кавычка нужна для закрытия ссылки

$result, это ресурс возвращенный запросом, а $i < $result;, это мягко выражаясь несуразица.

о ясно, а я думаю что у меня <div style:clerar:both;></div> выводится столько же раз сколько и позиций к выводу, ок теперь увидел свою ошибку, спасибо!

<? $i = 0; while ($row = mysql_fetch_assoc($result)) {      echo '<div style="float:left;padding:30px">              <p><img alt="'.$row['title'].'" title="'.$row['title'].'" src="'.$row['img'].'"/></p>              <p><a href="/shop/'.$row['hpu_cat'].'/'.$row['id'].'>'.$row['title'].'</a></p>              <p>'.$row['settings'].'</p>              <p>'.$row['price'].' руб. за 1 метр</p>          </div>';      if ($i && !($i % 3)) echo '<div style="clear:both"></div>';     $i++;    }

То что в одинарных кавычках, это строка:

<? $a = 12; echo 'string 1 $a'; //выведет как есть

То что в двойных кавычках подлежит обработке:

<? $a = 12; echo "string 2 $a"; //выведет string 2 12

Поэтому не "<div style='clear..., а '<div style="clear..., так как нет в этой строке ничего, на что бы нужно было обратить внимание интерпретатора, нет в ней данных для него.

printf выводит отформатированную строку согласно параметрам sprintf. Нечего в данном случае форматировать, и использовать эту функцию здесь, ну ради что нагрузить интерпретатор лишней работой, более ничего.

PS. Кстати, div float:left и прицепом к нему '<div style="clear:both"></div>', это не единственный и далеко не лучший способ вывода товаров на страницу, в смысле форматирования страницы.

Спасибо Вам за совет читать :) Уже начал просматривать мануал php, доступно написано.
Еще один вопрос...
Что я делаю тут не так? Мне нужна что выводилось по три элемента на странице, а далее перенос строки и опять вывод трех элементов

<?php  while ($row = mysql_fetch_assoc($result)) {     printf ("                  <div style='float:left; padding:30px;'>         <p><img alt='$row[title]' title='$row[title]' src='$row[img]'/></p>             <p><a href='/shop/%s/%s'>$row[title]</a></p>             <p>$row[settings]</p>             <p>$row[price] руб за 1 метр</p>         </div>         ", $row["hpu_cat"],$row["id"],$row["title"]);      for ($i=0; $i < $result; $i++){       if ($i % 3 == 0) echo "<div style='clear:both;'></div>";             }     } ?>

да, плюс ко всему, конкатенация еще и хорошо видна в любом редакторе.

Не знаю с видео уроков ли эта мода или нет, но уж коли так хочется эту функцию прикрутить к выводу массива, то почему тогда не vprintf или vsprintf, если цель упростить. Не понятно вообще чего тут форматировать надо, при выводе незатейливых данных из базы.

Самое лучшее, это не заставлять интерпретатор ковыряться в строке без надобности, то есть одинарные кавычки и конкатенация.