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

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. Artix - 09 Января, 2018 - 22:30:04 - перейти к сообщению

Вопрос к опытным кодерам как лучше и почему как бы вы использовали?

PHP:
скопировать код в буфер обмена

<?= (!empty($config['description']) ? '<meta name="description" content="'.$config['description'].'">' : '') ?>

PHP:
скопировать код в буфер обмена

<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 пишет:

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

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

4. Строитель - 10 Января, 2018 - 01:07:18 - перейти к сообщению

Мелкий Радость

5. Мелкий - 10 Января, 2018 - 11:06:12 - перейти к сообщению

Artix пишет:

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

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