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

[viking]

W samej klasie masz co najmniej kilka funkcjonalności. Pierwsze to przekazanie i odczyt wartości języka. Tu dodatkowo jeśli byś miał 20 języków to taki switch mocno by sie rozrosł. Nie można z tej klasy wyciągnąć bieżącej wartości języka. Drugi to adapter storage który akurat jest sesją domyślną ale gdybyś chciał zmienić na jakiś komponent zewnętrzny wtedy cały kod jest do przepisania a nie tylko dany fragment. Dodatkowo na sztywno czas sesji którego nie można zmienić przez konfigurację. O fatalnym nazewnictwie nospor już pisał.