<?
function tags ($tags_expert) {
$tags_exp = explode(" ", $tags_expert); /* В $tags_expert информация "1 2 3 4" */

 foreach($tags_exp as $key => $type) {
 $tags_sql = mysql_query("SELECT * FROM `tags` WHERE `tags`.`id` = '".$type."'") or die ("Error in query : $query. -" . mysql_error( ));
 $tags_result = mysql_fetch_assoc($tags_sql);

   echo '<a href="index.php?tags='.$tags_result['id'].'">'.$tags_result['tags'].'</a>';

  }
 }
?>

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

Упрощаем Где-то так получается:

function tags ($tags_expert) 
{
/* В $tags_expert информация "1 2 3 4" */

 $tags_expert=str_replace( ' ', ',' , $tags_expert);

 $tags_db = "SELECT * FROM `tags` WHERE `tags`.`id` in ($tags_expert)";
 $tags_sql = mysql_query($tags_db) or die ("Error in query : $tags_db. -" . mysql_error( ));
 
 while( $tags_result = mysql_fetch_assoc($tags_sql) )

 echo '<a href="index.php?tags='.$tags_result['id'].'">'.$tags_result['tags'].'</a>';

}

При чем тут вообще "быстрее"? Алгоритм другой. Через explode будем создавать массив. А str_replace просто заменит пробелы на запятые, чтобы получить список, который потом используем в условии. Если передавать уже готовый список с запятыми (в виде символьной строки), то и эта процедура не нужна будет.

function tags ($tags_expert) 
{
/* В $tags_expert информация "1 2 3 4" */

 $tags_expert=str_replace( ' ', ',' , $tags_expert);

 $tags_db = "SELECT * FROM `tags` WHERE `tags`.`id` in ($tags_expert)";
 $tags_sql = mysql_query($tags_db) or die ("Error in query : $tags_db. -" . mysql_error( ));
 
 while( $tags_result = mysql_fetch_assoc($tags_sql) )

 echo '<a href="index.php?tags='.$tags_result['id'].'">'.$tags_result['tags'].'</a>';

}

Спасибо помог! +1

Что мне в Вас нравится это аргументированная критика

а как не ужос ???

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

А когда доводим эту идею до абсурда... То получаем один из самых больших маразмов, которые я когда-либо видел и слышал Потому что до абсурда можно довести всё, что угодно.

Основное, что должно быть (тот самый "азбукиведи") - это максимальная чёткость и логичность скрипта. И если для формирования элемента меню или строки таблицы мне нужна функция, то я её сделаю и весь вывод в браузер (для этого одного элемента) буду делать внутри функции. Потому что это чётко и логично. И ни один апологет MVC не убедит меня в том, что я не прав. Потому что не надо до абсурда всё доводить.
И я не буду для реализации такой хрени изобретать свой шаблонизатор. Хотя бы потому, что View будет всё равно кодом формироваться...

Ту не в разделении дело вовсе. Я тоже иногда пишу разметку в исполняемых файлах. И тоже ни кто не убедит меня прикручивать шаблонизатор из-за пары тегов <li>

Но тут совсем другое. Если ты выводишь что то в поток, должно быть видно, что ты это делаешь. А если это функция, то она скорее всего находится в другом файле. И вот ты делаешь вывод:

<div><?php  tags ('1,2,3')  ?></div>

что тут делает функция... Считает теги, устанавливает, модифицирует или выводит в поток? Нихрена не понятно. Другое дело через ретурн

<div><?php  echo tags ('1,2,3')  ?></div>

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

arvitaly

Вот именно. Начинаем гадать на кофейной гуще - голые не голые.
Если есть запрос, должно быть видно обработку. Это как презерватив в тревожную ночь. Защищаться обязательно. А гдето когдато может быть... это не разговор.

Тоже самое... Может не может.
Если пользуешься or die то именно в момент отладки. И тут же уберись за собой. Иначе забудешь и повиснет навсегда. А лучше обертку юзать, тогда гарантия.

И не нужно оправдывать топикстартера и оправдываться самим - мол он мал и не смышлен. Ужос, он и есть ужос. Мне просто некогда было объяснять тогда.

Это зависит еще и от того, КАК ты функцию назвал , а ткже от того, как ты её используешь.
Вот есть у меня такая функция, написана где-то на 2-3-й неделе изучения мной ПХП. Работает до сих пор и я не вижу смысла её менять:

function create_main_menu( $active, $ref, $text )
{
  if( $active )
    echo ' <td class="menu_active" >';
  else
    echo " <td class=\"menu_inactive\" onmouseover=\"this.className ='menu_inactive_over';\" onmouseout =\"this.className='menu_inactive';\">
  <a href=\"$ref\">";
  echo $text;
  if( !$active ) echo '</a>';
  echo '</td>';
}

Вызывается она так (один из пунктов):

if( $user_abilities->en_users_edit ) // выводим возможность редактирования пользователей 
  create_main_menu( isset($usersediting), 'usersediting.php', 'Редактирование пользователей');	

Далее, если мне надо создать новую строку меню, то я просто добавлю новый вызов функции с нужными параметрами. И я "набираю" меня динамически, в зависимости от имеющихся у данного юзера прав.
Если я захочу изменить ПРЕДСТАВЛЕНИЕ, то мне достаточно только поменять начинку этой функции. В скрипте от этого НИЧЕГО не изменится. Т.е. тут как раз получается хорошее соответствие модели MVC.
И учитывая, что данная фунция нужна только в одном скрипте, она в нем же и определена. Как раз с точки зрения удобства. Максимум, что можно сделать, это с кавычками разобраться

Вопрос твину: и ты считаешь, что надо всё это усложнить, исходя из чьих-то туманных предствлений?

Да не надо иму они. Если стоит CSS класс, этого вполне достаточно.

sergeiss
Ну как хочешь. Я просто за прозрачность кода ратую всегда. Из функций (методов), которые содержат вывод в теле на мой згляд приемлимы только те, токорые называются display(), потому что это уже стало общепринятой практикой и не вызывает неоднозначных толкований.

У меня в пагинаторе есть функция createMenu()
Так вот она выполняет именно то, что и обязывает название - генерирует меню.
Но не выводит в поток. По этому толкование названия твоей функции ни как не дает однозначного ответа на её предназначение.

Вот открываешь чей-нибудь скрипт, а там адская смесь какого-нибудь такого безобразия:

<?php  
tags("1,2,3");
create_main_menu( isset($usersediting), 'usersediting.php', 'Редактирование пользователей');
getReklama();
setFooter();

и полез в доку или еще хуже (чаще всего) в функции и классы смотреть, как же именно там происходит это create...

Так то оно так, да только есть ведь еще такая вещь как процедура, а в php они одно и то же

Что значит должна? Кому?

а нельзя ли вместо \" использовать одинарные кавычки?

Не везде. Вот смотри, надо вывести такой HTML код

onmouseover="this.className ='menu_inactive_over';"

Тут, как ни крути, и одинарные, и двойные кавычки. Или одно экранировать, или другое. Я решил делать всё в двойных кавычках, потому что дальше в строке выводится переменная $ref.
И уже если я что-то внутри сделал в двойных кавычках, то и остальное тоже надо так сделать. Чтобы было всё единообразно. Чтобы стиль был определенный.

onmouseover="this.className ='menu_inactive_over';"

ява скрипт сразу не заметил