Revisar código (selección de idioma o, en su defecto, autodetección)

[ORPHEO]

Hola amigos:

He creado este código basándome en otros similares. Se trata de procesar el idioma elegido por el visitante o, en caso de no hacerlo, tratar de autodetectarlo y enviarle el archivo más apropiado.

Lo he probado y parece que todo funciona correctamente, pero me gustaría que alguien me lo revisara por si he metido la pata en algo y no me he dado cuenta...

Ahí va el código:

Código PHP:

  <?php 
 
// Procesar idioma 
if (isset($HTTP_GET_VARS['lang'])) { 
    $lang = $HTTP_GET_VARS['lang']; 
} 
 
if (isset($lang) 
    && !strstr($lang, '/') 
    && file_exists('lang/'.$lang.'.php')){ 
    $sitelang = strtr($lang, '$/\\:*?"\'<>|`', '____________'); 
 
} elseif ($HTTP_ACCEPT_LANGUAGE != ''){ # Verificamos que el visitante haya designado algún idioma 
    $lang = explode(",", $HTTP_ACCEPT_LANGUAGE); # Convertimos HTTP_ACCEPT_LANGUAGE en array 
    /* Recorremos el array hasta que encontramos un idioma del visitante que coincida con los idiomas 
    en que está disponible nuestra web */ 
    for ($i=0; $i<count($lang); $i++){ 
        if (!isset($sitelang)){ 
            if (substr($lang[$i], 0, 2) == "de"){$sitelang = "german";} 
            if (substr($lang[$i], 0, 2) == "en"){$sitelang = "english";} 
            if (substr($lang[$i], 0, 2) == "es"){$sitelang = "spanish";} 
            if (substr($lang[$i], 0, 2) == "fr"){$sitelang = "french";} 
            if (substr($lang[$i], 0, 2) == "it"){$sitelang = "italian";} 
            if (substr($lang[$i], 0, 2) == "pt"){$sitelang = "portuguese";} 
        } 
    } 
} else { 
    unset($lang); 
} 
 
 
# Si no hemos encontrado ningún idioma que nos convenga, mostramos la web en el idioma por defecto 
if (!isset($sitelang)){$sitelang = "english";} 
require "lang/{$sitelang}.php"; 
 
?>   

Os doy las gracias de antemano

Salu2.

[ORPHEO]

Por cierto, la mayor parte del código lo copié de éste que encontré en las FAQs (gracias kech ) en http://www.forosdelweb.com/showthrea...216#post443216 .

Alguien puede revisarme el resto?

Salu2.

[Manoloweb]

Pues todo parece bien,

lo unico que te sugiero es cambiar $HTTP_GET_VARS[] por $_GET[]
y $HTTP_ACCEPT_LANGUAGE por $_SERVER["HTTP_ACCEPT_LANGUAGE" ]

Para evitar que en un futuro cercano te vaya a arrojar errores o avisos de variables depreciadas.

[ORPHEO]

Gracias por revisarlo, colega

Cambiaré lo que me recomiendas. Y gracias de nuevo.

Salu2.

[josemi]

Hola,

To cambiaria este codigo:

Código PHP:

              if (substr($lang[$i], 0, 2) == "de"){$sitelang = "german";} 
 
            if (substr($lang[$i], 0, 2) == "en"){$sitelang = "english";} 
 
            if (substr($lang[$i], 0, 2) == "es"){$sitelang = "spanish";} 
 
            if (substr($lang[$i], 0, 2) == "fr"){$sitelang = "french";} 
 
            if (substr($lang[$i], 0, 2) == "it"){$sitelang = "italian";} 
 
            if (substr($lang[$i], 0, 2) == "pt"){$sitelang = "portuguese";  
   

Por uno basado en array:

Código PHP:

  $idiomas=array('es'=>'spanish','en'=>'english'); 
// y donde quieres asignarlo 
if (isset($idiomas[substr($lang[$i], 0, 2)])) $sitelang=$idiomas[substr($lang[$i], 0, 2)]; 
// y te ahorras todos los ifs 
// ademas, para añadir un nuevo idioma, 
// solo tienes que añadir un elemento 
// a $idiomas  
   

Saludos.

[ORPHEO]

Gracias Josemi, buena idea...

Salu2.