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

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Поправить чуть-чуть мой код

 PHP.SU

Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


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

> Без описания
event
Отправлено: 14 Февраля, 2012 - 01:08:23
Post Id


Частый посетитель


Покинул форум
Сообщений всего: 542
Дата рег-ции: Май 2011  


Помог: 0 раз(а)




Привет, есть такой код:
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 запрос, лучше уже при записи в базу редактировать код...
(Добавление)
Короче тему можно закрывать...............
 
 Top
DlTA
Отправлено: 14 Февраля, 2012 - 01:15:12
Post Id



Постоянный участник


Покинул форум
Сообщений всего: 2952
Дата рег-ции: Окт. 2010  


Помог: 53 раз(а)




скажем так, для того чтоб этот код положил сайт достаточно 1 пользователя и очень большой таблички (чем больше памяти выделено сервером тем больше надо табличка)

а проблема в том что код пытается выгребсти из базы все, нет ни условий ни ограничений
 
 Top
event
Отправлено: 14 Февраля, 2012 - 01:16:53
Post Id


Частый посетитель


Покинул форум
Сообщений всего: 542
Дата рег-ции: Май 2011  


Помог: 0 раз(а)




И какой вариант будет оптимален?
 
 Top
DlTA
Отправлено: 14 Февраля, 2012 - 01:21:32
Post Id



Постоянный участник


Покинул форум
Сообщений всего: 2952
Дата рег-ции: Окт. 2010  


Помог: 53 раз(а)




event пишет:
И какой вариант будет оптимален?

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

а вообще насколько я понял, код выдает не правильный результат, возвращает всех
а не тех кого надо
 
 Top
Ezi
Отправлено: 14 Февраля, 2012 - 08:06:59
Post Id



Частый гость


Покинул форум
Сообщений всего: 242
Дата рег-ции: Дек. 2010  


Помог: 2 раз(а)




event, люди правильно говорят, тебе нужно лимит и условия в запросе делать, например:

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

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


-----
Хочешь жить - умей вертеться
 
 Top
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Вопросы новичков »


Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.
 



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB