Linux за българи: Форуми

GeSHi (PHP):
<?php
session_start();
 
if (!isset($_SESSION['lang']))
	{
		$lang = substr($_SERVER["HTTP_ACCEPT_LANGUAGE"], 0, 2);
		switch ($lang) 
		{
			case "ru": 
				include ($_SERVER['DOCUMENT_ROOT']."/include/ru.php");
				$_SESSION['lang']="ru"; 
			break;
			case "bg": 
				include ($_SERVER['DOCUMENT_ROOT']."/include/bg.php"); 	
				$_SESSION['lang']="bg";
			break;
			default: 
				include ($_SERVER['DOCUMENT_ROOT']."/include/en.php");				
				$_SESSION['lang']="en"; 
			break;
		}
	}
else 
	{
	include ($_SERVER['DOCUMENT_ROOT']."/include/".$_SESSION['lang'].".php");
	}
?>

И

GeSHi (PHP):
<?php
session_start();
 
if (!isset($_SESSION['lang']))
	{
		if (substr($_SERVER["HTTP_ACCEPT_LANGUAGE"],0,2) == "ru")
		{
			include ($_SERVER['DOCUMENT_ROOT']."/include/ru.php");
			$_SESSION['lang']="ru";
		}
		else
		{
			if (substr($_SERVER["HTTP_ACCEPT_LANGUAGE"],0,2) == "bg")
			{
				include ($_SERVER['DOCUMENT_ROOT']."/include/bg.php");
				$_SESSION['lang']="bg";
			}
			else
			{ 
				include ($_SERVER['DOCUMENT_ROOT']."/include/en.php");			
				$_SESSION['lang']="en";
			}
		}
	}
else 
	{
	include ($_SERVER['DOCUMENT_ROOT']."/include/".$_SESSION['lang'].".php");
	}
?>
 

При мене горния с кейсовете не суичва файла и се сгомнясва интерфейса. Аз ли греша някъде?

Аз не виждам проблем в работата на горния код - ОК си работи.
Само не ми харесва как го правиш - имаш прекалено много повторение на код  ;)

GeSHi (PHP):
$languageFileMapping = Array(
    'en' => 'en',
    'bg' => 'bg',
    'ru' => 'russian',
);
 
$defaultLanguage = 'en';
 
if (empty($_SESSION['language']))
{
    $language = substr($_SERVER["HTTP_ACCEPT_LANGUAGE"], 0, 2);
    $_SESSION['language'] = array_key_exists($language, $languageFileMapping) ? $language : $defaultLanguage;
}
 
include($_SERVER['DOCUMENT_ROOT']."/print/".$languageFileMapping[$_SESSION['language']].".php");
 

Освен това проверяваш само за първия препдпочитан език, а не за всички...

GeSHi (PHP):
$languageFileMapping = Array(
    'en' => 'en',
    'bg' => 'bg',
    'ru' => 'russian',
);
 
$defaultLanguage = 'en';
 
if (empty($_SESSION['language']))
{
    $parsedLanguages = explode(',', $_SERVER["HTTP_ACCEPT_LANGUAGE"]);
    foreach ($parsedLanguages as $parsedLanguage)
    {
        $language = substr($parsedLanguage, 0, 2);
        if (array_key_exists($language, $languageFileMapping))
        {
            $_SESSION['language'] = $language;
            break;          
        }
    }
}
 
if (empty($_SESSION['language']))
    $_SESSION['language'] = $defaultLanguage;
 
include($_SERVER['DOCUMENT_ROOT']."/print/".$languageFileMapping[$_SESSION['language']].".php");

И последно - преди време ти бях писал едно лично съобщение за сайта, който правиш - прочете ли го? ;)

GeSHi (PHP):
<?php
session_start();
 
$languageFileMapping = Array('ru' => 'ru', 'bg' => 'bg', 'en' => 'en');
$defaultLanguage = 'en';
 
if (empty($_SESSION['language']))
{
   $parsedLanguages = explode(',', $_SERVER["HTTP_ACCEPT_LANGUAGE"]);
   foreach ($parsedLanguages as $parsedLanguage)
   {
       $language = substr($parsedLanguage, 0, 2);
       if (array_key_exists($language, $languageFileMapping))
       {
           $_SESSION['language'] = $language;
           break;          
       }
   }
}
 
if (empty($_SESSION['language']))
	{
		$_SESSION['language'] = $defaultLanguage;
	}
include($_SERVER['DOCUMENT_ROOT']."/print/".$languageFileMapping[$_SESSION['language']].".php");
 

Това е кода който сложих в сайта си сега и работи!!! Благодарности! Даже ако в хедърите браузъра прати de, fr-bl, ru, bg, en скрипта превключва на първия срещтнат "познат" език.

Малко чистота на кода

Не използвай това:

Примерен код

$_GET[lang]

Пълни ти се лог-а с предупреждения и забавяш работата.
Трябва да е:

Примерен код

$_GET['lang']

Същото се отнася за

Примерен код

if ($_GET[lang])

=>

Примерен код

if (empty($_GET['lang']))

И парчето:

Примерен код

if (empty($_SESSION['language']))    $_SESSION['language'] = $defaultLanguage;

трябва да се намира след всички опити за установяване на езика - това е default value за езика.

Мое лично правило е, че ако имам switch в кода (или нужда от него) нещо не е наред

Да не говорим, че използването на $_GET[lang] може да доведе до доста трудно откриваема грешка:

Примерен код

define('lang', 'something')

Т.е. използването на нестриктните форми води до един вид използване на глобални променливи, които винаги могат да бъдат "замърсени".

използването на неинициализирани променливи специално в РНР не мисля, че е кой знае какъв проблем - за мен това е толкова "несериозно", колкото и това, че не е нужно да се декларира типа на порменливата, но да не навлизаме в тази полемика ..........

а това със switch-a изобщо не разбрах в какъв контекст се появи

иначе балгодарско за линковете - има смислени неща наистина

В първото му съобщение, първото парче код ...

Не я цапаме - заради този код е започната темата и именно използването на switch в този конкретен случай трябва да се избягва

Най-лесно ми е да ти задам контра въпрос:
Ако използваме парчето код със switch-a, как ще изнесеш поддържаните езици в конфигурационен файл? Или в DB?


Кодът е чист пример за имплементирането на "данни" чрез използване на код. В един хипотетичен случай, някой може да реши, че не се нуждае от DB - ще си направи един огромен switch (primary key) със вложени switch-ове (foreign keys) и да връща някакви данни ...

По отношение на ООП - http://en.wikipedia.org/wiki/State_pattern

Цитат

The state pattern can be used to replace switch() statements and if {} statements which can be difficult to maintain and are less type-safe. For example, the following is similar to the above but adding a new tool type to this version would be much more difficult.

Относто swich-a...
Един човек който разбера повече от мен е написал тука за swich-a:
"в много случаи имплементира обект от data layer в application layer - както е в този случай "
И ето защо се получават "колизии" или не работещ код както иска "програмиста". Това се получава при недостатъчна компетентност на задклавиатурното устройство.

Затова и аз зададох тоя въпрос тука, е се радвам на дискусията.

мдааааа
примера в wikipedia е доста добър наистина - наследяването и класовете там са доста по-добро решение от ползването на условна конструкция
друг е въпроса, че ако сайта, който се прави не ползва ООП възможностите на РНР аз лично не бих се сетил за такъв подход за превключването между езиците - не казвам, че не е правилно, а просто, че аз не бих се сетил

Доста странно (да не използвам други думички) предложение за отстраняване на бъга.
Използването на единични кавички в PHP означава: стрингът да НЕ се обработва за променливи и специални символи (прим. \n ,\r и т.н.).
Т.е., изходът от този фрагмент:

GeSHi (PHP):
$var = 'value';
echo '$var';

е точно $var,  а не value.

С други думи, използването на това решение (array_key_exists('$category', $categories)) ще води ВИНАГИ до търсенето на ключ, който е '$category', а не '{стойността на $category}' ...

По-скоро направи малко дебъг, сложи един var_dump($language) преди if-a с array_key_exists и виж за какво става въпрос когато излиза тази грешка.
Най-вероятно е да се окаже, че $language е празен стринг.

GeSHi (PHP):
if (empty($_SESSION['lang']))
{
   $parsedLanguages = explode(',', $_SERVER["HTTP_ACCEPT_LANGUAGE"]);
   foreach ($parsedLanguages as $parsedLanguage)
   {
       $language = substr($parsedLanguage, 0, 2);
	   var_dump($language);
	   if (empty($language))
	   {
			echo ("1");
			break;	
	   }
	   else
	   {	   
	       if (array_key_exists($language, $languageFileMapping))
	       {
				echo ("2");
				$_SESSION['lang'] = $language;
				break;          
	       }
		}
   }
}
 

Така е решен проблема според мен? Правилно ли съм го "разгънал" кода?  var_dump($language); е добре когато браузъра подава език. Но пробвах с Гугъл Хрома да премахна всички езици и тогава и вар_дъмп казава че няма стойност променливата и array_key_exists реве, реве, реве че няма стойност променливата. Аз се заех да осигуря ВИНАГИ стойност на променливата която ще се подава на array_key_exists и разклоних логиката. Така правилно ли е? При мене кода вече работи без тази забележима грешка. Даже сложих ехо1, ехо2 за да виждам в кой клон на логиката съм отишъл в зеависимост от входните данни.

Най-лесно е само да проверяваш за стойността му:

GeSHi (PHP):
if (!empty($language) && array_key_exists($language, $languageFileMapping))

В случай, че нищо не определи езика, най-накрая се изпълнява принудителното задаване на default езика.

ПП: Тъй като скоро ме направиха модератор на тази секция е редно да помодераторствам :) :) :P
Редактирах ти съобщението така, че да използва code=php и съответното синтактично оцветяване :)

Разделяне на темата:
http://www.linux-bg.org/forum/index.php?topic=33005.0