Разделение php и html

PaRtiZzaN

18 фев 2019 г., 20:53

¯\_(ツ)_/¯

Не знаю правильно ли назвал тему, но вот вам такой вопрос.

Правильно ли я делаю изменяя код в таком стиле.
Было

echo '<!DOCTYPE html>' .
    "\n" . '<html lang="' . $config->lng . '">' .
    "\n" . '<head>' .
    "\n" . '<meta charset="utf-8">' .
    "\n" . '<meta http-equiv="X-UA-Compatible" content="IE=edge">' .
    "\n" . '<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=2.0, user-scalable=yes">' .
    "\n" . '<meta name="HandheldFriendly" content="true">' .
    "\n" . '<meta name="MobileOptimized" content="width">' .
    "\n" . '<meta content="yes" name="apple-mobile-web-app-capable">' .
    "\n" . '<meta name="Generator" content="JohnCMS, http://johncms.com .
    "\n" . '<meta name="keywords" content="' . $keywords . '">' .
    "\n" . '<meta name="description" content="' . $descriptions . '">' .
    "\n" . '<link rel="stylesheet" href="' . $config->homeurl . '/theme/' . $tools->getSkin() . '/style.css">' .
    "\n" . '<link rel="shortcut icon" href="' . $config->homeurl . '/favicon.ico">' .
    "\n" . '<link rel="alternate" type="application/rss+xml" title="RSS | ' . _t('Site News', 'system') . '" href="' . $config->homeurl . '/rss/rss.php">' .
    "\n" . '<title>' . $textl . '</title>' .
    "\n" . '</head><body>';

Стало

<!DOCTYPE html>
<html lang="<? echo $config->lng; ?>">
<head>
<meta charset="utf-8">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=2.0, user-scalable=yes">
<meta name="HandheldFriendly" content="true">
<meta name="MobileOptimized" content="width">
<meta content="yes" name="apple-mobile-web-app-capable">
<meta name="Generator" content="JohnCMS, http://johncms.com
<meta name="keywords" content="<? echo $keywords; ?>">
<meta name="description" content="<? echo $descriptions; ?>">
<link rel="stylesheet" href="<? echo $config->homeurl; ?>/theme/<? echo $tools->getSkin(); ?>/style.css">
<link rel="shortcut icon" href="<? echo $config->homeurl; ?>/favicon.ico">
<link rel="alternate" type="application/rss+xml" title="RSS | <? echo _t('Site News', 'system'); ?>" href="<? echo $config->homeurl; ?>/rss/rss.php">
<title><? echo $textl; ?></title>
</head><body>

Не будет потом никаких из за этого проблем? Просто мне кажется что так как то красивее и удобнее.

Nells

18 фев 2019 г., 21:38

Ну думаю что норм)

ДоХтор

18 фев 2019 г., 22:04

PaRtiZzaN, правильно. Только есть один существенный недостаток: вот такая запись

<? echo $config->lng; ?>

может вдруг на том или ином хостинге не сработать. Причина - короткие php-теги <?. Если не ошибаюсь, в новых версиях PHP подобная запись уже не поддерживается. Пиши или так:

<?php echo $config->lng; ?>

Или так

<?= $config->lng ?>

PaRtiZzaN

18 фев 2019 г., 22:11

¯\_(ツ)_/¯

ДоХтор, Понял, спасибо.

kantry

19 фев 2019 г., 7:58

Hey guys! Finally I'm gonna change status!?

ДоХтор, Поддерживается, из php.ini php7.3
; should be disabled, as enabling it may result in issues when generating XML
; documents, however this remains supported for backward compatibility reasons.

Constantin

19 фев 2019 г., 9:03

ENERGETIK

Стало лучше