Форумы портала PHP.SU » » Вопросы новичков » Вопрос к опытным кодерам

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

1. Artix - 09 Января, 2018 - 22:30:04 - перейти к сообщению
Вопрос к опытным кодерам как лучше и почему как бы вы использовали?
PHP:
скопировать код в буфер обмена
  1. <?= (!empty($config['description']) ? '<meta name="description" content="'.$config['description'].'">' : '') ?>


PHP:
скопировать код в буфер обмена
  1. <meta name="keywords" content="'.$config['keywords'].'">
2. Мелкий - 09 Января, 2018 - 23:03:04 - перейти к сообщению
Artix пишет:
если теги не заполнены мы бд не дергаем лишний раз

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

Имя $config для хранения специфичных для каждой страницы keywords и description явно неудачная мысль. И забыли сделать htmlspecialchars при выводе в html
Ну а пустой keywords бесполезен, зачем его выводить?
3. Artix - 09 Января, 2018 - 23:07:57 - перейти к сообщению
Мелкий пишет:
Artix пишет:
если теги не заполнены мы бд не дергаем лишний раз

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

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

htmlspecialchars() чтобы не было этих &lt; крякозябров а норм символы выводил с бд < ?
4. Строитель - 10 Января, 2018 - 01:07:18 - перейти к сообщению
Мелкий Радость
5. Мелкий - 10 Января, 2018 - 11:06:12 - перейти к сообщению
Artix пишет:
htmlspecialchars() чтобы не было этих &lt; крякозябров а норм символы выводил с бд < ?

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

 

Powered by ExBB FM 1.0 RC1