PHP OOP i strona wielojęzyczna, pytanie początkującego Opcje

[szczypior]

index.php

[PHP] pobierz, plaintext 
include 'classes/lang.class.php';
 
 
$lang = new lang_set($_GET['lang']);
 
include_once 'lang/'.$lang;
[PHP] pobierz, plaintext 

plik lang.class.php

[PHP] pobierz, plaintext 
<?php
class lang_set{
 
private $lang;
 
	public function __construct($lang_in){
 
		// gdy mamy zmienną, to znaczy że użytkownik chce zmienić obecny język
		If(!empty($lang_in))
		{
 
			switch ($lang_in) {
			  case 'en':
			  $this->lang = 'en';
			  break;
 
			  case 'pl':
			  $this->lang = 'pl';
			  break;
 
			  default:
			  $this->lang = 'pl';
			}
			$_SESSION['lang'] = $lang;
			setcookie('lang', $lang, time() + (3600 * 24 * 30));
		}
		else if(isSet($_SESSION['lang']))
		{
			//gdy mamy zmienną sesyjną, to ją odczytujemy
			$this->lang = $_SESSION['lang'];
		}
		else if(isSet($_COOKIE['lang']))
		{
			// gdy użytkownik ma ciasteczko to odczytujemy zapisany w nim język i zapisujemy dodatkowo do zmiennej sesyjnej
			$this->lang = $_COOKIE['lang'];
			$_SESSION['lang'] = $_COOKIE['lang'];
		}
		else
		{
			//domyślny język
			$this->lang = 'pl';
		}
	}
 
	public function __toString()
    {
       return $this->lang.'.lang.php';
    }
}
?>
[PHP] pobierz, plaintext 

Ten post edytował szczypior 12.08.2016, 13:00:46

[szczypior]

Czego brakuje temu kodowi, by faktycznie był obiektowy?

Wszystko jest w konstruktorze, bo ta klasa wydawała mi się na tyle mało rozbudowana, że nie potrzebuje dodatkowych funkcji. Jak mniemam, myliłem się (IMG:style_emoticons/default/smile.gif)

Będę bardzo wdzięczny za sugestie (IMG:style_emoticons/default/smile.gif)

Abstrahując od tego, że faktycznie zmianę języka można rozwiązać w lepszy sposób (tak jak sugerujesz przez htaccess), to czy możesz wskazać kierunek, jak należałoby poprawić ten kod, by stał się obiektowym.
Czy właśnie należałoby go podzielić na funkcje?

Z góry dzięki za odpowiedź (IMG:style_emoticons/default/smile.gif)

Ten post edytował szczypior 12.08.2016, 13:13:20