Я бы написал так:

<?php
require 'top.php'; // это то, что должно быть в шляпе(div'ы, ссылки и прочее)
require 'left.php'; //это левая панель(навигация тож самое div, ссылки)
if (isset($_GET['page'])) {
$page = $_GET['page'] ? $_GET['page'] : 'news'; //не заметил что дефаулт у тя news.php отредактировал
 if(file_exists('tpl/'.$page.'.php')) {
   include ('tpl/'.$page.'.php');
  } else {
   echo "Страница не найдена";
  }
}
include 'bottom.php'; //Тут футер
?>

Ответиг: Нет не должно быть тк в итоге у тя получается 1 большой файл, со всеми переменными которые могут быть использованны в любом месте скрипта, рекомендую пользоватся функциями если у тебя похожие действия.

Renden, в топку твой вариант - он дырявый насквозь. Нельзя так делать.

Как минимум, так:

<?php
require 'top.php'; // это то, что должно быть в шляпе(div'ы, ссылки и прочее)
require 'left.php'; //это левая панель(навигация тож самое div, ссылки)

$modules = array('addnews', 'reg');
if (in_array($_GET['page'], $modules)) {
    include 'tpl/'. $_GET['page'] .'.php';
else
    include 'news.php';

include 'bottom.php'; //Тут футер

neadekvat
Да согласен такой вариант лутьше, неподумал чет. Вплане дырявый, да в папке tpl можно выполнить любой скрипт который там лежит, но там ведь только шаблоны страниц...
Я тоже учусь так что прокоментируй пожалуйста)

т.е. как я понял сначала создаем массив со всеми имеющимися страничками, потом проверяем есть ли запрашиваемая страница в массиве и если есть, то подключаем ее, если нет, то можно ошибку 404...
Вроди так понял. Так строится навигация по страничкам?

kmaks
Ну в данном случае так, я же храню заместо массива информацию о страницах в базе mysql)

а как передать переменную страничке?
типа shownews.php?id=1

kmaks

$id = (int)$_GET['id'];

Это то я знаю...

<?php
require 'top.php'; // это то, что должно быть в шляпе(div'ы, ссылки и прочее)
require 'left.php'; //это левая панель(навигация тож самое div, ссылки)

$modules = array('shownews', 'reg');
if (in_array($_GET['page'], $modules)) {
    include 'tpl/'. $_GET['page'] .'.php';
else
    include 'news.php';

include 'bottom.php'; //Тут футер

Ведь смотри, формируется страничка index.php, а не shownews.php и если передать значение переменной, то получится в index.php...

там ведь shownews.php подключится просто(без передачи переменных)
:blink:

Хорошо, попробую. Спасибо всем!

Работает! Все спасибо...так даже код менее громоздкий

kmaks
Я бы доработал вариант Renden по взятию id до такого:

$id = isset($_GET['id'])?(int)$_GET['id']:0;

if ( ($id > 0) && ($id <=pagesCount) ) {
// тут ещё можно сделать жёсткую проверку по имеющимся ссылкам! 
// выполняем наши странички
}
else {
// ошибка 404
}

denizkin, зачем это? И откуда берется pagesCount? Почему в константе всего одна буква в верхнем регистре, когда все цивилизованные программисты пишут имена констант в верхнем регистре?

neadekvat
pagesCount - это общее кол-во твоих страниц, тобишь макс id, после которого нет страниц! Чтоб если ввести неверную страницу был переход на 404, а не переход на несуществующую страницу!
И вообще ты незамечаешь, что ты фигню пишешь?
Человек спросил как id взять ему ответили, я привёл пример как лучше это дело экранировать, более жёсткий шаблон! Я привёл в данном конкретном случае!
Цивилизованные программисты жёстко затачивают всё, чтобы не было сбоев и пишут всё под конкретную задачу!

Ты там мне так и не ответил по поводу таблицы, чем она плоха в замену той таблицы связей которую привёл Snus!

neadekvat
Ты такой... я люблю твороженные булочки, просто ужас!

1) Чтоб не возникло такой ситуации я внутри сделал подпись, что можно сделать доп. условие которое и будет выбирать на остальные страницы!

2) Где ты увидел константу? pagesCount получается из запроса к бд! Я тебе вставил кусок кода, ты в нём константу увидел!

3) Там где разговор про таблицы! Не прикидывайся, я люблю булочки!

4) Да ты вообще смешён!))

Если хочешь я до твоего кода тоже докапаюсь, который ты сверху привёл
если введёшь index.php?page=reg&page=addnews сработает news! Фигли ты такой умный и не учёл этого?

Залогинься и подтверди, что это действительно ты, denizkin. В противном случаи я не буду тратить время на анонима.

neadekvat
А то ты не понял с кем ты говоришь! Опять прикидываешься! Я с другого компа просто зашёл не заметил, что не был залогинен! Короче одним словом, я люблю твороженные булочки!

neadekvat
1) У кого ник тот он и есть!

2) Я pagesCount по твоему откуда беру? Внутри у себя в коде я проверяю так же на существование в базе, просто если у меня стоит флажок, что удалённых страниц нет или страниц которые не отображаются, я не делаю лишних запросов в базу при проверке ид!

3) Это опечатка, я люблю твороженные булочки!

4) я люблю твороженные булочки

5) Ну вот такой неадекват как ты и будет 2 параметра одинаковых передавать запросе и кричать что почему ты не сделал так!))

Короче будь проще, и выражай мысли нормально!

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

<?php

define ("BLOCK", "YES");
include 'connect.php';
include 'tpl/header.php';
$modules = array('products', 'register', 'buy', 'more', 'cart');

  if(in_array($_GET['page'], $modules))
    {
    include 'tpl/'. $_GET['page'] .'.php';
    }

  else
    {
   include 'tpl/products.php';
    }

include 'tpl/left.php';	
include 'tpl/footer.php';
?>

neadekvat
Какая это константа тебе выше написано опечатка $ забыл поставить! Но понятное дело куда тебе мега умники догнать то!

Если ты ссышь, то тебе пора к врачу!
По фене?! Ты её хоть знаешь ?

OoOo. Вы чё так завелись, да:

1. Частенько кто нить смеётся над чьим то кодом: хотя можно было бы деликатно указать на минусы

2. И всегда ТС будет так реагировать на всё это, так уж устроен человек.

Так что есть "недодуманный скрипт" и есть правокатор.

Кто-то технарь и ставит собеседника на равне с собой.
Кто-то сам учиться, потому что нравиться и учиться на своих ошибках.

Может просто учиться помогать, а не критиковать?

Игорь_Vasinsky
Грамотный подход!