Napisałem prosty skrypt do obsługi wielu języków na stronie. Niby działa, ale chciałbym żebyście napisali mi czy ten skrypt jest dobrze napisany, czy są jakieś lepsze rozwiązanie i czy ten skrypt jest bezpieczny.

[PHP] pobierz, plaintext 
if (isset($_SESSION['lang'])) {
	$lang = $_SESSION['lang'];
}
else {
	$lang = 'pl';
}
 
if (!empty($_GET['lang'])) {
	$lang = $_GET['lang'];
	switch($lang) {
		case 'pl':
			$lang = 'pl';
		break;
		case 'en':
			$lang = 'en';
		break;
		default:
			$lang = 'pl';
		break;
	}
	$_SESSION['lang'] = $lang;
}
require_once('languages/'.$lang.'.php');
[PHP] pobierz, plaintext 

W pliku languages/$lang.php mam coś takiego:

[PHP] pobierz, plaintext 
$Lang['title'] = 'Tytuł';
$Lang['test'] = 'Test...';
[PHP] pobierz, plaintext 

no niby jest ok, ale ten switch jest nie potrzebny, przy większej ilość języków kod by się za bardzo powiększał, na szybko można zastąpić czymś takim:

[PHP] pobierz, plaintext 
$langs = array('pl', 'en', 'de');
 
(isset($_SESSION['lang']) && in_array($langs)) ? $lang = $_SESSION['lang'] : $lang = 'pl'; 
(isset($_GET['lang']) && in_array($langs)) ? $lang = $_GET['lang'] : ''; 
 
$_SESSION['lang'] = $lang;
 
require_once('languages/'.$lang.'.php');
[PHP] pobierz, plaintext 

w sumie racja, zapomniałem że można tablice do tego wykorzystać dzięki za odp. A i w sumie zamiast sesji lepiej ciastko dać, dopiero teraz o tym pomyślałem.