code plus simple ?

Version imprimable

code plus simple ?

Hello,

Voilà comment je récupère la langue d'un visiteur.

Il y aurait-il plus simple ou intelligent que ce code ?

Notamment la deuxième partie où je met une langue par default si pas supporté.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
function get_lang()
{
	$lang = $_GET["lang"];
 
	// if lang empty set lang from browser
 
	if (!isset($lang) || $lang == '')
	{
		$lang = explode(",", $_SERVER['HTTP_ACCEPT_LANGUAGE']);
		$lang = StrToLower(substr(chop($lang[0]) , 0, 2));
	}
	return $lang;
}

Code:

1
2
3
4
5
6
7
8
9
10
 
 
/* Language detection */
$lang = get_lang();
 
// set default if lang not supported and include lang file
if ($lang != "en" ||  "jp" || "fr")
include ('lang/en/en_index.php');
else
include ('lang/'.$lang.'/'.$lang.'_index.php');

22/07/2015, 09h16
bretus

Citation:

Envoyé par thibotus01

Il y aurait-il plus simple ou intelligent que ce code ?

Utiliser le composant de traduction de Symfony2 (la documentation en français)?

(Tu n'es pas obligé de passer tout ton projet sur Symfony2, intégrer composer pour gérer tes dépendances peut aider par contre)
22/07/2015, 11h01
thibotus01

J'esperais plutôt une simple réponse à mon code, pas une autre solution complète. Mais merci.
22/07/2015, 11h11
bretus

Citation:

Envoyé par thibotus01

J'esperais plutôt une simple réponse à mon code, pas une autre solution complète. Mais merci.

Ton erreur consiste à ne pas avoir une fonction qui te donne une chaîne de caractère traduite. Si tu commences à faire autant de script PHP que tu as de langue, ton code ne sera pas maintenable : Tu devras appliquer le moindre ajout de code sur chaque script.

La solution propre, c'est donc une solution qui évite l'anti-pattern DRY (Dont Repeat Yourself) : Une fonction/classe qui permet de traduire une chaîne. Toujours pour rester dans le "propre", tu dois éviter de réinventer la roue et prendre un composant qui existe. Si le "Translator Symfony2" ne te plait pas, je pense que tu en trouveras facilement d'autres (l'extension PHP INTL par exemple)

++
22/07/2015, 22h24
Tsilefy

Il faudrait savoir ce que tu cherches:
- une version plus simple de ton code?
- une meilleure version de ton code?
- une bibliothèque qui gère la traduction? (on t'a fait la bonne suggestion de Symfony/Translation dans ce cas)
23/07/2015, 00h03
Tsilefy

Citation:

Envoyé par bretus

Ton erreur consiste à ne pas avoir une fonction qui te donne une chaîne de caractère traduite. Si tu commences à faire autant de script PHP que tu as de langue, ton code ne sera pas maintenable : Tu devras appliquer le moindre ajout de code sur chaque script.

Je présume qu'il se contente de charger un fichier de langue différent, et pas un script PHP différent pour chaque langue!!
23/07/2015, 02h27
ABCIWEB

Oui bof installer un composant symfony pour détecter la langue du navigateur c'est pas un peu too much ? On serait dans le forum symfony je comprendrais mieux mais là...

Déjà on pourrait peut-être essayer locale_accept_from_http, c'est prévu pour ça. Juste en local sous wampserver ça peut retourner éventuellement un "Class 'Locale' not found", mais sinon en production ça fonctionne toujours même sur des mutu entrée de gamme "récents" (mini php 5.3).

Après pour la langue par défaut, je met mes langues traduites dans un tableau, donc la condition se fait sur un classique in_array($lang,$lang_traduites). En poussant un peu plus loin pour trouver la langue la plus proche on pourrait utiliser locale_lookup.