Дайте совет по CMS строению

"WHERE id"

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

Не понял эту запись

Почему не так

if(!$_GET['id']) {
 $cat =$_GET['catid'];
 if($cat){ 
 

У вас же $_GET['catid'] не может быть ноль. Или isset или !isset. Зачем лишние проверки делаете?

Devider добавил 16.03.2009 в 14:05

Повторы новостей, если новостей больше чем категорий, и наоборот )

Плохо представляю как это грамотно сделать в один запрос. Наверное из меня некудышный mysql-оптимизатор)) В принципе можно сделать одним запросом, если присваивать ключ массиву catid, дублироваться ничего не будет, но это неудобно на будущее. Лучше лишний раз с кэшем поработать )

Там где нужно считать все категории и товары, считываем все категории, по id

$Q=mysql_query("SELECT * FROM TABLE");

while ($R=mysql_fetch_assoc($Q)) { $A[$R['catid']]=$R; }

Потом новости

$Q=mysql_query("SELECT * FROM TABLE2");

while ($R=mysql_fetch_assoc($Q)) { $A[$R['catid']]['SUB'][]=$R['id']; $B[$R['id']]=$R; }

Шаблонизатор вы видимо не используете...

foreach ($A as $i=>$CAT_NEWS)

{

echo 'Категория '.$CAT_NEWS['name'];

if (isset($CAT_NEWS['SUB'])) foreach ($CAT_NEWS['SUB'] as $j=>$id_news) { echo $B[$id_news]['name'].'<br>'; }

}

Если немножко подумать, можно конечно сделать и грамотнее, а можно и так ;)

Добавлю еще что в целом сама цель неясна. Зачем выбирать ещё и категории, когда выбираете из конкретной категории новости?

И если представите структуру таблиц - будет пояснее, конечно=)

Категория там выбирается она, просто дублируется к новостям ) Желание сэкономить на запросах видимо завело ТС не туда...

ну если так, тогда

if($cat_id){

$dop_query = "WHERE n.cat_id=$cat_id";

}

SELECT * FROM news n INNER JOIN cats c ON c.id=n.cat_id $dop_query GROUP BY n.id

чёнить такого плана

выборка по ID.

seosniks добавил 16.03.2009 в 16:22

Выбирать из конретной категории,это кода мне надо все новости из категории интренет. Я жму на ссылку которая имеет например ID номер 3

и все носоти этой категории видны.

А так мне надо чтоб на морде сайцта выводились категории без сортировки, тоест новость кинул в раздел деньги и она видна.

таблица

category    

 

catid	title_cat	cpu_cat

 1	политика	politika

 2	новости	news

 3	интернет	internet

 0	Сегодня	segodnya 

таблица news

id title author dates text url_istok views status cpu md5 catid

где catid имеет номер новости из таблицы категорий

например catid 1= категории политика

Категории я хотел сделать чтоб новост идобавлять в разны екатегории.

Прочел книгу но видать структура кода не очень правильная. :(

Я не волшебник я только учусь :D

seosniks добавил 16.03.2009 в 16:24

Да именно дублируеться каждая категория.

Просто как составить верный запрос не вкурю.

Если у меня урл без дополнительных параметров,тогда надо все носоти

если урл имеет парамерты после ? тогда выводим то что идет после ?.

Мне кажется так не стоит делать:

category.catid='".$cat."'

ибо:

$cat = "10'(закрыли кавычку) UNION SELECT (что нужно) WHERE '1'='1(для закрывающей)"

нужно хотя бы так:

category.catid='" . (int)$cat . "'

а лучше по нормальному:

$cat = mysql_real_escape_string($cat)

PS: давно на php не кодил могу ошибиться, но не принципиально

это не код вообще в принципе, это кусок говна

Ну я тогда написал же как сделать лучше всего будет в вашем случае=)

/ru/forum/comment/4517474

исправляя под вашу структуру

$dop_query = '';

if(is_numeric($cat_id)){

    $dop_query = "WHERE n.cat_id=$cat_id";

}

SELECT * FROM news n INNER JOIN category c ON c.catid=n.catid $dop_query GROUP BY n.id

В этом случае выберутся все данные, сгруппированные по id из news - хотя в целом при такой структуре дублей быть не может)

то есть усложняем -

$dop_query = '';

if(is_numeric($_GET['cat_id'])){

    $dop_query = "WHERE n.cat_id=".$_GET['cat_id'];

}

$query = "SELECT c.title_cat, c.catid, n.title FROM news n INNER JOIN category c ON c.catid=n.catid ".$dop_query." GROUP BY n.id";

$res = mysql_query($query);

if($res && mysql_num_rows($res)!=0){

    while($row = mysql_fetch_assoc($res)){

        echo "Категория: <a href='/?catid=".$row['catid']."'>".$row['title_cat']."</a><br>Заголовок новости - ".$row['title']."<hr>";

    }

}

Это понятно что это не код. Но ваш пост к чему?

Я Просил помощи, совета, Ваш ответ не очень похож на него.

Хотя судя по вашему блогу, у Вас все пишут гавном ;):D