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]   

> Без описания
Artix
Отправлено: 09 Января, 2018 - 22:30:04
Post Id


Новичок


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


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




Вопрос к опытным кодерам как лучше и почему как бы вы использовали?
PHP:
скопировать код в буфер обмена
  1. <?= (!empty($config['description']) ? '<meta name="description" content="'.$config['description'].'">' : '') ?>


PHP:
скопировать код в буфер обмена
  1. <meta name="keywords" content="'.$config['keywords'].'">

(Отредактировано автором: 09 Января, 2018 - 22:54:23)

 
 Top
Мелкий Супермодератор
Отправлено: 09 Января, 2018 - 23:03:04
Post Id



Активный участник


Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009  
Откуда: Россия, Санкт-Петербург


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




Artix пишет:
если теги не заполнены мы бд не дергаем лишний раз

$config - это объект класса с ArrayAccess, который умеет дёргать базу? Или за счёт чего экономия?
Впрочем, в таком случае empty запрашивает offsetExists + offsetGet, поэтому можно сделать даже хуже, смотря как реализовано.

Имя $config для хранения специфичных для каждой страницы keywords и description явно неудачная мысль. И забыли сделать htmlspecialchars при выводе в html
Ну а пустой keywords бесполезен, зачем его выводить?


-----
PostgreSQL DBA
 
 Top
Artix
Отправлено: 09 Января, 2018 - 23:07:57
Post Id


Новичок


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


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




Мелкий пишет:
Artix пишет:
если теги не заполнены мы бд не дергаем лишний раз

$config - это объект класса с ArrayAccess, который умеет дёргать базу? Или за счёт чего экономия?
Впрочем, в таком случае empty запрашивает offsetExists + offsetGet, поэтому можно сделать даже хуже, смотря как реализовано.

Имя $config для хранения специфичных для каждой страницы keywords и description явно неудачная мысль. И забыли сделать htmlspecialchars при выводе в html
Ну а пустой keywords бесполезен, зачем его выводить?

htmlspecialchars() чтобы не было этих &lt; крякозябров а норм символы выводил с бд < ?

(Отредактировано автором: 09 Января, 2018 - 23:08:32)

 
 Top
Строитель Модератор
Отправлено: 10 Января, 2018 - 01:07:18
Post Id



Участник


Покинул форум
Сообщений всего: 1580
Дата рег-ции: Февр. 2014  
Откуда: Украина


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




Мелкий Радость
 
 Top
Мелкий Супермодератор
Отправлено: 10 Января, 2018 - 11:06:12
Post Id



Активный участник


Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009  
Откуда: Россия, Санкт-Петербург


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




Artix пишет:
htmlspecialchars() чтобы не было этих &lt; крякозябров а норм символы выводил с бд < ?

htmlspecialchars нужен потому что вы выводите не HTML в HTML и надо соблюдать правила вывода того формата, в который вы выводите. Чтобы случайной кавычкой не сломать в принципе всю страницу (и это в лучшем случае, в худшем - подставить ваших пользователей под XSS)


-----
PostgreSQL DBA
 
 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