mam przyjazne linki w .htaccess

RewriteRule ^([a-zA-Z-_0-9]+)/?$ index.php?id1=$1 [L]

ładowanie podstron za pomocą kodu

[PHP] pobierz, plaintext 
...
if (file_exists($id1.'.php')) {
require_once($id1.'.php');
} else {
	echo '<p></br>Strona w trakcie aktualizacji. Zapraszamy na stronę główną</p> ';
	}
...
[PHP] pobierz, plaintext 

jeżeli odwiedzający próbuje odwiedzić stronę za pomocą linku
http://ommo.pl/produkty/oswietlenie-led/zasilacze
to wszystko jest ok i zostaje przekierowany do strony głównej
ale jeżeli wpisze np. coś takiego:
http://ommo.pl/index.php/wejscie-na-strone-ktorej-nie-ma/
to zostaje przełączony na tą stronę, na stronę której nie ma.

W sumie to nie byłoby problemu gdyby nie fakt, że mam automatyczne liczniki odwiedzin podstron i takie odwiedziny są rejestrowane. Powoduje to ogromny bałagan w wynikach odwiedzin - stron, których nie ma.
Pomóżcie

Zacznijmy od tego że twój pomysł jest zły bo pozwala na ataki ;]
Zmień system includowania na szukanie klas-kontrolerów, obecnie masz dziurę.

Przenieś licznik, tak aby zliczał dopiero po załadowaniu poprawnej podstrony

[PHP] pobierz, plaintext 
if(file_exists())
{
$licznik++;
require();
}
else
{
echo 'Brak strony';
}
[PHP] pobierz, plaintext 

dokładnie w tym miejscu mam licznik ale nie umieszczałem go ponieważ nie to jest problemem




to wydaje się być sensowne ale nie wiem jak się do tego zabrać
czy to napewno pomoże rozwiązać mój problem
może podasz jakiś przykład

[PHP] pobierz, plaintext 
class Liczniki_Liczniki extends Controller 
{
function index()
{
echo 'Test';
}
}
[PHP] pobierz, plaintext 

mam nadzieję, że drążę w dobrą stronę ale póki co nic mi z tego nie wychodzi
dostaję błąd Fatal error: Class 'Controller' not found in ...


Podpowiedzcie jak się zabrać za tą klasę kontrolera.

http://php.pl/phppl/Wortal/Artykuly/PHP/Po...-poczatkujacych

Poczytałem na temat projektowania obiektowego i zapewne będę małymi kroczkami wprowadzał je na swoją stronę, ale ...
... tymczasem chciałbym skupić się na problemie jaki opisałem w tytule.
Czy szukanie klas kontrolerów rozwiąże mój problem?


Pomóżcie z tymi klasami.

Tu masz opisane jak zabezpieczyć się przed wejściem użytkowników na strony, które nie istnieją
http://forum.php.pl/index.php?s=&showt...t&p=1016184

co do tego Controllera

Controller.php

[PHP] pobierz, plaintext 
class Controller {
}
[PHP] pobierz, plaintext 

Klasa.php

[PHP] pobierz, plaintext 
require 'Controller.php';
class Klasa extends Controller {
}
[PHP] pobierz, plaintext 

Próbowałem zmodyfikować mój sposób ładowania według podanego przez Ciebie przykładu ale niestety wejście na podstrony, których nie ma jest nadal możliwe.
Mój kod odpowiedzialny za ładowanie stron:

[PHP] pobierz, plaintext 
function pobierz_parametr($id){
if(isset($_GET["$id"]))
{
$id=$_GET["$id"];
}
else
	{
	$id=false;
	}
	return $id;
}
 
$id1=pobierz_parametr('id1');
$id2=pobierz_parametr('id2');
 
...
if($id1!=false){
	if (file_exists($id1.'.php')) {
	print licznik2();
	require_once($id1.'.php');
	} else {
		echo '<p></br>Strona w trakcie aktualizacji. Zapraszamy na stronę główną</p> ';
		}
	}
	else{
	if (file_exists('srodek.php')) {
	print licznik2();
	require_once('srodek.php');
	}
...
 
 
[PHP] pobierz, plaintext 

CuteOne proszę, pomóż zmodyfikować aby zablokowało dostęp do stron nieistniejących.

Nigdy nie rób includowania w ten sposób

[PHP] pobierz, plaintext 
include $id.'.php';
[PHP] pobierz, plaintext 

bo wystarczy w adres wpisać http:// strona.pl/?id=config a mogą wypłynąć dane serwera/bazy danych itp. itd.

Co do samego działania skryptu, to podałem w linku w jaki sposób sobie z tym radzić. Naprawdę nie wiem w czym masz problem

mam problem z połączeniem Twojego sposobu ładowania podstron z moim sposobem na przyjazne linki

Tylko po co chcesz modyfikować tamten kod? Wystarczy dodać licznik() pomiędzy pierwszy if() + ewentualnie sprawdzenie czy plik istnieje

faktycznie ale problemem pozostaje, czyli możliwość odwiedzania stron, których nie ma i licznik nalicza odwiedzeni strony nieistniejącej
jak uniemożliwić odwiedzanie i np wyświetlić stronę błędu

Nie wiem jak przerobiłeś ten skrypt ale bez zmian nie ma możliwości wejścia na stronę, której nie ma:

1. Użytkownik wchodzi na stronę (z automatu odpala się bootstrap - index.php)
2. W index.php sprawdzasz przesłane dane z adresu przeglądarki ($_GET)
3. Jeżeli index.php?id=stronaxxx nie istnieje lub nie podano ?id= ustawiasz ładowanie default.php
4. Jeżeli strona istnieje (id znajduje się w tablicy $modules doliczasz odwiedziny)

Piszesz o przyjaznych linkach, może masz skopany .htaccess, który nie przekierowuje wszystkich requestów na index.php?

CuteOne jak zrobić to tak jak to powinno być zrobione czyli bezpiecznie ?

Ograniczyć dostęp tylko do tego, co faktycznie istnieje i jest dozwolone. Konkretnie to o co teraz pytasz nazywa się dispatcher. Zrobisz dla Twojego rozwiązania router + sensowny dispatcher po klasach kontrolerów i masz w miarę spokój

czytałem na ten temat ale ciężko mi to ogarnąć
może jakiś prosty, działający przykład

Widzę, że masz chrapkę na naukę , więc może zainteresuj się frameworkami? Na start polecam CodeIgniter - prosty i przyjemny framework dla małych projektów.