Почему браузер видит код PHP как комментарий?

Question

[Евгений]

Возникла такая странная ситуация. Решил написать, для своих нужд, мини движок для сайта. Не хочу учиться сразу с фреймворка или юзать чьи то решения. Просто для учебы, и ничего сверхсекретного. По сути пару страниц шаблона, главная страница и класс для отображения страниц. Но столкнулся с такой бедой, на главной странице код PHP работает отлично, а вот на второстепенных страницах не работает от слова совсем. Тупо пустая страница с одной строкой HTML и одним выводом phpinfo() выводит только HTML.
В режиме разработки браузер отображает мой код PHP вот так:

<!--?php
phpinfo();
?--> ==$0

Я уже думал, что я где-то начудил с тегами HTML, может где не закрыл, ибо использую отдельно файлы футера и хедера, но проверил уже все. Что может быть не так?
В index.php все работает, подключаются файлы, создаётся объект, работает переключение страниц. Только вот на этих страницах код уже не работает.

Comments

[Dmitry Bay]

1) Настройки сервера кривые, может только index.php и отрабатывает.
2) Есть вероятность все дело в short_open_tag

[Евгений]

Dmitry Bay, как проверить первый пункт?
По второму, так я ещё не использовал сокращений, всегда открываю и закрываю полным тэгом.

[FanatPHP]

Dmitry Bay, при чем здесь short tag, если у него в примере полный.

[Dmitry Bay]

FanatPHP, потому что 26 минут назад, когда я отвечал - примера не было.

[Дмитрий]

Что за сервер вообще?
Расширение файлов .php?

[Евгений]

Дмитрий, сервер я использую XAMPP, Apache 2.4.41, PHP 7.4.3. Расширение .php

[P747]

Сам файл имеете расширение ".php"? Покажите более подробно ваш код

[Евгений]

Я все больше склоняюсь, что у меня где-то проблема в классе, именно в коде:

<?php
class Engine {
 
    private $_page_file = null;
    private $_error = null;
 
    public function __construct() {
        if (isset($_GET["page"])) { //Если открыта какая-нибудь страница
            //Записываем в переменную имя открытого файла (из GET запроса)
            $this->_page_file = $_GET["page"]; 
            //Небольшая защита
            $this->_page_file = str_replace(".", null, $_GET["page"]);
            $this->_page_file = str_replace("/", null, $_GET["page"]);
            $this->_page_file = str_replace("", null, $_GET["page"]);
 
             //Проверяем, если шаблон не существует
            if (!file_exists("templates/main/" . $this->_page_file . ".php")) {
                $this->_setError("Шаблон не найден"); //Ошибку на экран
                $this->_page_file = "main"; //Открываем главную страницу
            }
        }
         //Если в GET запросе нет переменной page, то открываем главную
        else $this->_page_file = "main";
    }
 
    //Записывает ошибку
    private function _setError($error) {
        $this->_error = $error;
    }
 
    //Возвращает текст ошибки
    public function getError() {
        return $this->_error;
    }
 
    //Возвращает текст открытой страницы
    public function getContentPage() {
        return file_get_contents("templates/main/" . $this->_page_file . ".php");
    }
 
    //Возвращает заголовок открытой страницы
    public function getTitle() {
        switch ($this->_page_file) {
            case "main":
                return "Главная страница";
                break;
            case "about":
                return "Про портал";
                break;
            case "news":
                return "Новости";
                break;
            case "polls":
                return "Опрос";
                break;
            case "login":
                return "Вход";
                break;
            default:
                break;
        }
    }
 
}
?>

И как я это вызываю:

include_once "class/Engine.php"; //Подключаем класс-движка
$engine = new Engine(); //Создаем объект класса Engine

include_once "templates/main/header.php"; //Подключаем шапку сайта

if ($engine->getError()) { //Если возникли ошибки, выводим сообщение на экран
    echo "<div style='border:1px solid red;padding:10px;margin: 10px auto; 
        width: 500px;'>" . $engine->getError() . "</div>";
}
echo $engine->getContentPage(); //Выводим страницы сайта

include_once "templates/main/footer.php";//Подключаем подвал сайта

[P747]

В вашем коде, php код берется просто как код без всякого выполнения, тоесть отдается как текст браузеру.

Поменяйте:

echo $engine->getContentPage()

На:

echo eval(' ?' . '>' . $engine->getContentPage() . '<' . '?php ');

[Lander]

P747, Какой интересный аналог include. :)

[Евгений]

P747, вот оно!!! Спасибо. Я даже понял зачем это))) Получается я в одном тэге php открывал еще один... Спасибо.

[Дмитрий]

Angel_D13, в шаблонах тоже надо открывать теги, ну и использовать include вместо file_get_contents

[FanatPHP]

Чувак, ты правда не понял что тебе Lander, написал? Что весь красивый класс "Engine" заменяется одной командой include?

[Евгений]

FanatPHP, получается не понял. Через eval() это костыль?

[FanatPHP]

Во-первых костыль, во-вторых небезопасно, а главное - бессмысленно.
Сейчас весь движок - это такой карго культ - выглядит красиво и солидно, но несет очень мало смысла.

Но главный косяк в том, что заголовок - это не единственная часть страницы, которую нужно знать до вывода шапки сайта.
Некоторым страницам в шапке нужна информация, которую можно узнать только в коде страницы - например - заголовок новости.
Некоторым страницам надо еще вывести допольнительный НТТР заголовок
Некоторые страницы используют допольнительные CSS или JS
Некотоыре сраницы выводят не хтмл а джейсон, и шапка сайта им как корове седло
Некоторые страницы вообще ничего не выводят, им шапка тоже не нужна

У тебя мысль правильная, но сработала она не до конца.
Движок нужен не для того чтобы выводить шапку перед контентом. А для того, чтобы отделить логику от вывода
Сначала у тебя должен отработать код страницы, а потом - если нужно - выводим шапку, контент и подвал.
То есть каждая страница должна состоять из двух частей - из логики и из представления.
А движок должен сначала инклюдить логику, а потом - представление. Передавая в него переменные, с данными, коорые были получены в первой части.

[Lander]

FanatPHP, Можно тебя попросить вот этот твой комментарий в ответ вынести? :)

[FanatPHP]

Lander, если вспомнить что по исходной формулировке вопроса, как оказалось, вообще невозможно догадаться о проблеме, то такой ответ будет выглядеть странно :)

[Lander]

FanatPHP, То есть твой изначальный ответ

Потому что дети, воспитанные ютубочкой, не умеют гуглить

не выглядит странно, а этот будет. Хорошо. Я понял. :)))

[Евгений]

Всем спасибо. На свой вопрос точно получил ответ. Даже понял где нужен include и, соответственно, убрал костыль. Дальше уже тема для других разговоров, буду учить.

[Евгений]

Дмитрий, вынеси пожалуйста в ответы свой комментарий

в шаблонах тоже надо открывать теги, ну и использовать include вместо file_get_contents

Он и есть ответ, просто я долго его осмысливал)))

Answer 1

[FanatPHP]

Потому что дети, воспитанные ютубочкой, не умеют гуглить

Копируешь адрес страницы, на которой РНР работает, и адрес на которой не работает.
Вставляешь в чистый файл в редакторе один под другим.
Ищешь 10 различий

Comments

[Евгений]

Я бы не был так опрометчив в высказываниях, не имея что сказать. Хотя могу согласиться с тем, что я, возможно, не понимаю, что спрашивать. Это первый мой вопрос здесь, я в основном только читал и решал все сам. Так есть что сказать, что искать?

[FanatPHP]

Искать надо заголовок своего вопроса.

[FanatPHP]

И да, конечно, это я не знаю что сказать.
Вторая беда детей воспитанной ютубой - это конечно абсолютная уверенность в своей гениальности, и соответственно - тупости всех окруажщих.

[Евгений]

FanatPHP, мне кажется это вы ещё ребенок. К чему здесь про гениальность и прочее? Меня интересует ответ, а не споры про возраст. И да, ещё раз повторю, я Гуглил и долго Гуглил.

[Ярослав Иванов]

Angel_D13,

Что-то не верится
https://www.google.com/search?q=%D0%9F%D0%BE%D1%87...

[Евгений]

Ярослав Иванов, да я на это смотрю уже не один день. Нет у меня подобных проблем, а то что перепробовал не помогает. Нет у меня проблем ни с сервером, ни с PHP. Не выводится у меня код в браузере. Пробовал я создавать тестовые страницы с переходом на другие, все ОК. А вот здесь...

[edward_freedom]

FanatPHP, ты так свое недовольство высказываешь, как будто этой твой личный форум

[Илья]

FanatPHP, что-то похоже в тупости то здесь ты всех обвиняешь. Как же наверное трудно твоему великому гению живется, да?

[FanatPHP]

Илья, нормально живется, спасибо за заботу :)
Тут вопрос скорее не в тупости а в упертости. Люди в массе своей не хотят учиться ничему новому. ВЫучили, как обезьяны, пару операторо - вот цикл, вот htmlspecialchars() - и пытаются их применять на любй случай жизни, не важно - подходит лии нет. Это грустно

[Илья]

FanatPHP, ясненько, видимо нормальный программист на ближайшие миллионы километров только ты. Печально конечно работодателям наверное. Где ж ты работаешь, такой самородок-то, кому удалось урвать такое сокровище?

[FanatPHP]

Илья, вот опять у тебя логика хромает :)
из утверждения "вокруг много недоучек" никак не следует утверждение "нормальный программист только я".

Ну и если говорить насчет работодателей - то в целом, большинство людей, котоыре понимают что делают - а, главное, думают перед тем как отвечать на вопрос - обычно не сидят на тостере. Так что выборка нерелевантная

[Илья]

FanatPHP, почему она хромает? Из твоих упреков ко всем подряд (а особенно если посмотреть какие вопросы ты задавал тут), то можно сделать вполне закономерный вывод что все крайне тупы, особенно в сравнении с тобой. И это если что был утрированный вывод (видимо у тебя проблемы с восприятием информации).

И как же так случилось - люди которые думают, не сидят на тостере, а ты что здесь забыл тогда?!
P.S. это если что тоже было утрирование и я видел слова "в массе" и "обычно", но в целом это равносильно "все" и "всегда" если дело касается тебя. Какое же у тебя огромное это, это здец просто

[FanatPHP]

Илья, потому что хромает.
про сравнение со мной - мимо, нигде никаких сравнений нет
про "все" - тоже мимо. да, я чаще пишу коммент когда не согласен чем когда согласен - такова особенность человеческой писхологии. но из этого никак не следует что я упрекаю "всех подряд". На тостере есть и хорошие специалисты, которые спосбны видеть дальше собственного носа и думать перед тем как дать совет, а не кидаются строчить ответ, увидев одно ключевое слово в заголовке вопроса.

[Илья]

FanatPHP, да уж, на таких как ты и держится шаткий мир QA сервисов. Неплохо!