Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770
Форумы портала PHP.SU :: Версия для печати :: Поправить чуть-чуть мой код
Форумы портала PHP.SU » » Вопросы новичков » Поправить чуть-чуть мой код

Страниц (1): [1]
 

1. event - 14 Февраля, 2012 - 01:08:23 - перейти к сообщению
Привет, есть такой код:
PHP:
скопировать код в буфер обмена
  1.         $bb = array(); $html=array();
  2.         $row = sql_query("SELECT id, name FROM persons") or die(mysql_error());
  3.         while ($res = mysql_fetch_assoc($row)) {
  4.         if (!empty($res['name'])) { $name = explode(',',$res['name']);
  5.         foreach ($name as $word) {
  6.         $bb[] = "#$word#si";
  7.         $html[] = "<a href=\"persons.php?id={$res['id']}\">$word</a>";
  8. }
  9.  }
  10.   }  


Он ищет в описание релиза имя актера, если такое находит, то это слово преобразовывает в ссылку. Ну, вот сразу пару вопросов:
1. Можно как нибудь уменьшить код? Так как в будущем будет давать не слабую нагрузку, а не хотелось бы. Огорчение
2. Как Вам реализация кода, сойдет?
(Добавление)
Да ну нафик такой код, по 3 запроса на страничке, где используется функция, у меня этот вставлен в функцию обратывания BB кодов...
(Добавление)
Жесть, таким кодом можно угробить нахер сервак.
+1 слово, +1 запрос, лучше уже при записи в базу редактировать код...
(Добавление)
Короче тему можно закрывать...............
2. DlTA - 14 Февраля, 2012 - 01:15:12 - перейти к сообщению
скажем так, для того чтоб этот код положил сайт достаточно 1 пользователя и очень большой таблички (чем больше памяти выделено сервером тем больше надо табличка)

а проблема в том что код пытается выгребсти из базы все, нет ни условий ни ограничений
3. event - 14 Февраля, 2012 - 01:16:53 - перейти к сообщению
И какой вариант будет оптимален?
4. DlTA - 14 Февраля, 2012 - 01:21:32 - перейти к сообщению
event пишет:
И какой вариант будет оптимален?

за таким нюансом не комне

а вообще насколько я понял, код выдает не правильный результат, возвращает всех
а не тех кого надо
5. Ezi - 14 Февраля, 2012 - 08:06:59 - перейти к сообщению
event, люди правильно говорят, тебе нужно лимит и условия в запросе делать, например:

sql_query("SELECT id, name FROM persons WHERE что-то_помогающее_найти_нужное = `значение` LIMIT максимум_строк_которые_вернет_запрос")

Поправте если ошибка, просто с SQL не очень дружу))
(Добавление)
И вообще, пора бы с ООП подружиться и взять нормальный паттерн для работы с БД Улыбка

 

Powered by ExBB FM 1.0 RC1