raz masz $_GET['page']
, a potem $this->page
, a dla mnie wygląda, że Z ZAŁOŻEŃ ma robić to samo.
Zamiast pisać:
switch($this->page) {
case 'friends':
$var = 'friends';
break;
case 'friends_friends':
$var = 'friends_friends';
break;
case 'suggest':
$var = 'suggest';
break;
case 'recommend':
$var = 'recommend';
break;
}
return $var;
nie lepiej po prostu:
switch($this->page) {
case 'suggest':
case 'recommend':
case 'friends':
case 'friends_friends':
// tu ew. include($this->page.'.class.php'); jeżeli nazwy plików mają te same nazwy
return $this->page;
break; // break nawet niepotrzebny, ale dla czytelności niech zostanie
default:
// coś w innym przypadku, jeżeli faktycznie $_GET['page'] ma być tym samym co $this->page, to po prostu wywal sprawdzenie u góry i dorzuć to tutaj.
}
Niestety nie wyjaśniłeś - na co to Tobie? Nie zbudujesz "wzorca projektowego" na jakimś gotowym nieobiektowym kodzie w 2 dni i to nie mając (chyba) pojęcia o obiektach - jeżeli działa to już lepiej będzie go zostawić jak jest.