Вывод товаров из категорий

[MilkeyWay]

Здравствуйте!
Хочется узнать ваше мнение,насколько это рабочий код,не возникнет ли проблем с безопасностью,быстродействием и т.п?
Впопыхах своял,черпая инфу из инета,вот что получилось(вывод осуществляется из заранее известной категории,пока выводятся только линк,название товара,цены,фото):

Скрытый текст (вы должны войти под своим логином или зарегистрироваться и иметь 50 сообщение(ий)):

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

Задача вообще стоит-см.название темы,скажите,что у меня своялось - не бред сумасшедшего?Все что нужно работает,но я как обычно сомневаюсь,ибо только учусь

зы сори,разделом промахнулся,наверное надо в общие вопросы

[Aleksey]

1. Для кода PHP есть соответствующий тег, используй его.
2. Разработчики скрипта ViartShop вместо va_ пишут $table_prefix.
3. discount_percent для чего в выборке? Нигде не используется.
4. В конце MySQL-запроса обычно пишут точку с запятой.
5. Выражение FROM va_items,va_items_categories WHERE va_items_categories.$ii = va_items.$ii можно немного изменить на FROM va_items i, va_items_categories ic WHERE ic.$ii = i.$ii, что особенно актуально для больших запросов.
6. Нет закрывающей фигурной скобки, которая закрывает цикл while.
7. Открывающих div - 7, закрывающих - 6. Один где-то пропустил.
8. Нет закрывающего тэга img.

[MilkeyWay]

1. Для кода PHP есть соответствующий тег, используй его.
2. Разработчики скрипта ViartShop вместо va_ пишут $table_prefix.
3. discount_percent для чего в выборке? Нигде не используется.
4. В конце MySQL-запроса обычно пишут точку с запятой.
5. Выражение FROM va_items,va_items_categories WHERE va_items_categories.$ii = va_items.$ii можно немного изменить на FROM va_items i, va_items_categories ic WHERE ic.$ii = i.$ii, что особенно актуально для больших запросов.
6. Нет закрывающей фигурной скобки, которая закрывает цикл while.
7. Открывающих div - 7, закрывающих - 6. Один где-то пропустил.
8. Нет закрывающего тэга img.

1.можно по подробнее,не совсем понимаю о каком теге идет речь?
2.$table_prefix-переменная со значением va,подумал можно обойтись и без переменной,да и те переменные выше тоже наверное ни к чему
3.думал скидка хранится в бд,оказалось скриптом выводится,забыл удалить
4.хм,вроде везде стоят,или я опять чего-то не понимаю о.О
5.ага,видел такое объединение,не стал разбираться,теперь обязательно разберусь
6.стоит в конце закрывающего тега a,она же там должна стоять?
7.везде 7 насчитал
8.ага,не разобрался еще,почему-то если поставить "/>" ,то в шаблоне отображаются два слеша и img не срабатывает(только ща пришла мысль-похоже нужен пробел перед слешом,т.к слеш приклеивается к адресу картинки и браузер доставляет сам закрывающий тэг)

зы Aleksey,спасибо за дельные советы,благодарю,что не прошел мимо!ко всем пунктам прислушался,буду штудировать
зыы не критично будет,если я в этом же файле сделаю еще один/пару таких же запросов,но к другим к категориям?

[Aleksey]

1. Вот он:

2. Можно обойтись и без нее.
4. После AND $c = 76, но это не критично.
6 и 7. Да, все правильно, че то нижние строчки в первый раз не скопировались.
8. Ага, надо пробел перед />.

PS: Да делай сколько угодно запросов.