Смена языка на сайте

[Red Alert]

Смена языка на сайте

Подскажите плз как улучшить код смены языка
в смысле безопасности и любой совет подойдёт
я только учусь :

function lang(){
if (($_GET['lg'] == "") OR ($_GET['lg'] == "en")){
 $lg = $_GET['lg']."lang_en.php";
 require("lang/lang_en.php");
}elseif (($_GET['lg'] == "ru")){
  require("lang/lang_ru.php");
}elseif (($_GET['lg'] == "es")){
  require("lang/lang_es.php");
}else {
 require("lang/lang_en.php");
 }
}

 

[SelenIT]

Непонятен смысл строчки

$lg = $_GET['lg']."lang_en.php";

и первый if я бы заменил на

if (!isset($_GET['lg']))

А в смысле безопасности, имхо, неплохо.

 

[Red Alert]

работает tnx )))

 

[Loshadka]

Red Alert
немного по-другому...
в этом варианте проще изменять путь к файлам и удалять/добавлять новые языки....

// Все существующие языки.
$lang = array('en', 'ru', 'es');

// Если язык не задан, или выбран несуществующий, то используем по-умолчанию.
if( !isset($_GET['lg']) || !strlen($_GET['lg']) || !in_array($lang, $_GET['lg']) )
    $_GET['lg'] = 'en';

require('lang/lang_'.$_GET['lg'].'.php');

 

[Red Alert]

Спасибо Loshadka
но не работаетб только берёт английский язык по умолчанию.
и пришет: Warning: in_array(): Wrong datatype for second argument in

 

[Гравицапа]

Red Alert
Loshadka
http://php.rinet.ru/manual/ru/function.in-array.php

 

[Toxic_Cat]

Red Alert
100% Лучше делать с массивом, как Вам подсказал Loshadka

 

[Loshadka]

Гравицапа, Red Alert

да, сорри... утро

// Все существующие языки. 
$lang = array('en', 'ru', 'es'); 

// Если язык не задан, или выбран несуществующий, то используем по-умолчанию. 
if( !isset($_GET['lg']) || !strlen($_GET['lg']) || !in_array($_GET['lg'], $lang) ) 
    $_GET['lg'] = 'en'; 

require('lang/lang_'.$_GET['lg'].'.php');

 

[Red Alert]

Автор оригинала: Loshadka

 !in_array($_GET['lg'], $lang) )

Я пробовал так, там в мануалах вычитал, но мне выдавало полную белиберду на введение языка типа 'fr' или 'dgsgfggsg' , а скопировал твоё исправление всё ОК.
Видно руки слишком кривые

Ладно, спасибо, всё работает