Составил index.php для одностраничника, что не так?

Question

[Антон]

Здравствуйте. Я только изучаю php, написал простой index.php для одностраничника, состоящего из блоков. Что сделано неправильно?

Header("Content-Type: text/html;charset=UTF-8");

    define("_ROOT", __DIR__ . '/');
    define("_BASEURL", "http://" . $_SERVER["HTTP_HOST"]);
    define("_BLOCKS", __DIR__ . '/blocks/');

	include_once _BLOCKS . 'header.php';
	include_once _BLOCKS . 'nav.php';
	include_once _BLOCKS . 'slider.php';
	include_once _BLOCKS . 'about.php';
  include_once _BLOCKS . 'portfolio.php';
	include_once _BLOCKS . 'contacts.php';
	include_once _BLOCKS . 'footer.php';

Comments

[Ernest Faizullin]

а что за ошибка?

Answer 1

[Anar4you]

define("_ROOT", __DIR__ . '/');
    define("_BASEURL", "http://" . $_SERVER["HTTP_HOST"]);
    define("_BLOCKS", __DIR__ . '/blocks/');

зачем вообще все это?

Comments

[Ernest Faizullin]

на будущее видимо. Для расширения фреймворка))

[Антон]

Anar4you : использую в блоках для подсоединения скриптов например.

[nirvimel]

erniesto77: Мой фреймворк послужит основой для моей будущей CMS.

Answer 2

[M K]

Заголовок раньше времени отправил, если у тебя в инклудах ещё php код выполняется

Comments

[Антон]

А куда его переместить?

[M K]

DigitalEmotions: перед первым выводом(он может быть в подключаемом файле).
В данном примере, я думаю, этим заголовком можно пренебречь, обычно он ставится автоматически

[Валерий Рябошапко]

Если включена буферизация вывода, то это не играет роли.

[Ernest Faizullin]

DigitalEmotions: header надо оставить только в index.php, в остальных файлах убрать

[Антон]

erniesto77: он и есть только в индексном файле.

[Ernest Faizullin]

DigitalEmotions: а что за ошибка то выпадает?)

[Антон]

erniesto77: нет никаких ошибок.

[Ernest Faizullin]

DigitalEmotions: в чем тогда проблема?

[Антон]

erniesto77: не знаю, правильная ли архитектура.

[Ernest Faizullin]

DigitalEmotions: вполне себе имеет право на существование, единственное нужно не повторять свой код, сделать его как можно короче и удобнее, если можно сократить код то надо сокращать, вот например можно все урлы страниц в нужном порядке засунуть в массив и инклудить форычем. Если у тебя появится новая страница, то ее адрес нужно будет просто добавить в этот массив и она сама подключится

[Ernest Faizullin]

то есть если появится новый блок

[Ernest Faizullin]

include_once _BLOCKS . повторяется аж семь раз

Answer 3

[Михаил Лялин]

А нужен ли вообще PHP для одностраничника?

Comments

[Дмитрий Евграфович]

письма отправлять и мониторинг юзеров делать "новый, уже приходил"

[Дмитрий Евграфович]

Но в данном случае вы правы, использовать php только чтобы собирать блоки в одну страницу - это хрень.

Answer 4

[Дмитрий Евграфович]

открывающий тег <?php в начале забыли

Answer 5

[Алексей]

Смотри в сторону автозагрузки habrahabr.ru/post/138920