Funkcja - x katalogów wyżej Opcje

[el_plecinio]

Witam,
Stworzyłem sobie niedawno funkcje poniżej by móc podlinkowac obrazki z katalogu równoległego tzn dzieki niej mogłem mając strukture folderów:
dir/img
dir/strona1
pobrać obrazki z img do katalogu strona1 bez używania linków bezwzglednych
Prosze o ocenę mojej 1 funkcji.

parametry:
$ilosc - odpowiada za liczbę katalogów wstecz (domyslnie)
$p - (0|1|2) brak|http|https

[PHP] pobierz, plaintext 
function cat_back($ilosc=0, $p=1)
                {
                if ($p==1) {$pro = 'http://';}
                else if ($p==2) {$pro = 'https://';}
                else if ($p>2 OR $p<1){$pro = '';}
                $dir = $_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
                $part = explode('/', $dir);
                $i=count($part)-1;
                $dirt = '('.$part[$i-$ilosc].'(.*))';
                $katalog = preg_replace($dirt,'',$dir);
 
                if ($ilosc <$i & $ilosc>=0){return $pro.$katalog;}
                else {return 'Blad zmiennej "<i>$ilosc</i>" - wartosc parametru musi byc mniejsza od '.$i;}
                } 
 
//test
echo $_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'].'<br>';
echo cat_back().'<br>';
echo cat_back(3).'<br>';
echo cat_back(2).'<br><br>';
echo cat_back(1,0).'<br>';
echo cat_back(1,1).'<br>';
echo cat_back(1,2).'<br>';
echo cat_back(-1).'<br>';
[PHP] pobierz, plaintext 

Ten post edytował el_plecinio 29.07.2012, 17:59:45

[ethann]

Małe zmiany z mojej strony.

[PHP] pobierz, plaintext 
function cat_back($ilosc=0, $p=1)
{
	$ilosc = intval($ilosc);
 
	if 	($p==1)			$pro = 'http://';
	else if ($p==2) 		$pro = 'https://';
	else 				$pro = '';
 
 
	$dir = $_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
	$part = explode('/', $dir);
	$i = count($part)-1;
 
 
	if ($ilosc < $i && $ilosc >= 0) {
		$dirt = '('.$part[$i-$ilosc].'(.*))';
 
		$katalog = preg_replace($dirt, '', $dir);
		return $pro.$katalog;
	} 
	else {
		echo 'Blad zmiennej "<i>$ilosc</i>" - wartosc parametru musi byc nieujemna i mniejsza od '.$i;
		return false;
	}
 
} 
[PHP] pobierz, plaintext 

Na samym początku konwersja zmiennej $ilosc na int, aby uniknąć błędów z nieporządanym numerem indeksu.

[PHP] pobierz, plaintext 
$ilosc = intval($ilosc);
[PHP] pobierz, plaintext 

Skoro rozpatrujesz przypadek kiedy $p = 1, oraz $p = 2, a następnie dla każdego innego $p wartość $pro ma być '', wtedy zrób po prostu else.

[PHP] pobierz, plaintext 
if ($p==1) {$pro = 'http://';}
else if ($p==2) {$pro = 'https://';}
else if ($p>2 OR $p<1){$pro = '';}
 
->
 
if 	($p==1)			$pro = 'http://';
else if ($p==2) 		$pro = 'https://';
else 				$pro = '';
[PHP] pobierz, plaintext 

Aby uniknąć ostrzeżeń interpretera (nieistniejące indeksy tablicy $part), te dwie linijki powinny znaleźć się już wewnątrz warunku, który potwierdza poprawność danych.

[PHP] pobierz, plaintext 
 $dirt = '('.$part[$i-$ilosc].'(.*))';
$katalog = preg_replace($dirt,'',$dir);
[PHP] pobierz, plaintext 

W przypadku błędu proponuję zwrócić false, który będzie sygnałem braku poprawności zakończenia funkcji i ewentualne wyświetlenie komunikatu.

[PHP] pobierz, plaintext 
else {
	echo 'Blad zmiennej "<i>$ilosc</i>" - wartosc parametru musi byc nieujemna i mniejsza od '.$i;
	return false;
}
[PHP] pobierz, plaintext 

Zamiast pojedynczego ampersanda w warunkach, korzystaj z podwójnego.