Тема: Тия два кода еднакви ли са? (Прочетена 6910 пъти)

a_panov · « -: Oct 20, 2008, 00:01 »

Код

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");
	}
?>

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

VladSun · « **Отговор #1 -:** Oct 20, 2008, 01:13 »

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

Код

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");

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

a_panov · « **Отговор #2 -:** Oct 20, 2008, 09:33 »

Цитат (VladSun @ Окт. 20 2008,02:13)

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

'>

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

'>

При мене първия код не ми включва файловете а втория си работи. Днес ще направя още една проверка която се сетих като се събудих днес сутринта. От 6 до 8 часа циклих на тоя проблем и когато тръгнах да ставам от компютъра писах по форумите.

ДА, да, да и затова съм се разбързал да то преработвам. Това е "парче лайно" от тоя сайт за който говориш.

a_panov · « **Отговор #3 -:** Oct 20, 2008, 12:25 »

Код

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 скрипта превключва на първия срещтнат "познат" език.

a_panov · « **Отговор #4 -:** Oct 20, 2008, 14:15 »

Примерен код

<?php
session_start();

$languageFileMapping = Array('ru' => 'ru', 'bg' => 'bg', 'en' => 'en');
$defaultLanguage = 'en';

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

if (empty($_SESSION['lang']))
   {
      $_SESSION['lang'] = $defaultLanguage;
   }

if ($_GET[lang])
{
   if (array_key_exists($_GET[lang], $languageFileMapping))
   {
      $_SESSION['lang'] = $_GET[lang];
   }
}

include($_SERVER['DOCUMENT_ROOT']."/include/".$languageFileMapping[$_SESSION['lang']].".php");
...
...
Кода продължава тука с други неща...
...
...
?>

Това е крайния код на триезичен сайт!!!

VladSun · « **Отговор #5 -:** Oct 20, 2008, 15:09 »

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

'>

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

Примерен код

$_GET[lang]

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

Примерен код

$_GET['lang']

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

Примерен код

if ($_GET[lang])

=>

Примерен код

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

И парчето:

Примерен код

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

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

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

'>

senser · « **Отговор #6 -:** Oct 20, 2008, 16:39 »

Цитат (VladSun @ Окт. 20 2008,15:09)

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

Примерен код

if ($_GET[lang])

=>

Примерен код

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

От известно време ме гложди съмнението доколко е обосновано използването на empty() в условни конструкции вместо вътрешното преобразуване на типа .........
Не че е кой знае какъв дерт де - по скоро е за спорта, но все нямам време да направя някой тест скрипт

П.П. Какво напрЕко имаш против switch, или еквивалентни вложени if-ове?

VladSun · « **Отговор #7 -:** Oct 20, 2008, 17:03 »

Примерен код

if ($_GET['language'])
{

}

=>
error.log

Цитат

PHP Notice: Undefined index: language in ....

Примерен код

if ($_GET[language])
{

}

=>

Цитат

PHP Notice: Use of undefined constant language - assumed 'language' in .....
PHP Notice: Undefined index: language in ...

Относно switch-a:
- в много случаи имплементира обект от data layer в application layer - както е в този случай;
- в повечето от останалите случаи използването се налага заради лош ООП подход.

Два интересни линка:
http://www-128.ibm.com/developerworks/library/os-php-designptrns/
http://www.ibm.com/develop....atterns

За втория - по-специално The state pattern

Искрено и чистосърдечно апелирам, всеки, който иска да се занимава с ООП, да прочете
Design Patterns: Elements of Reusable Object-Oriented Software

http://en.wikipedia.org/wiki/Design_Patterns

VladSun · « **Отговор #8 -:** Oct 20, 2008, 17:19 »

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

Примерен код

define('lang', 'something')

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

senser · « **Отговор #9 -:** Oct 20, 2008, 22:52 »

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

'>

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

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

'>

VladSun · « **Отговор #10 -:** Oct 20, 2008, 23:00 »

Цитат

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

Обичам в логовете ми за грешки да влизат само съобщения, само когато наистина има нещо. А, и аз повече наблегнах на използването на "неявни" стрингове.
В нестриктните езици дебъгването може да стане голям проблем, ако не се спазват някои определени правила

'>

Цитат

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

Цитат

П.П. Какво напрЕко имаш против switch, или еквивалентни вложени if-ове?

Ми ... ти писа за него

'>

senser · « **Отговор #11 -:** Oct 21, 2008, 12:11 »

Цитат (VladSun @ Окт. 20 2008,15:09)

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

'>

ето заради това попитах за switch-a

'>

не видях никъде в кода на a_panov да го е изполвал и затова не разбрах защо го включи

VladSun · « **Отговор #12 -:** Oct 21, 2008, 13:36 »

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

senser · « **Отговор #13 -:** Oct 21, 2008, 14:42 »

Цитат (VladSun @ Окт. 21 2008,13:36)

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

мда - прав си
пропуснал съм го
но въпреки това продължавам да не разбирам каква ти е основната идея срещу switch ........ ама това си е мой проблем де

'>
за да не цапаме темата на човека - спирам

VladSun · « **Отговор #14 -:** Oct 21, 2008, 15:49 »

Не я цапаме - заради този код е започната темата и именно използването на 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.

Автор Тема: Тия два кода еднакви ли са? (Прочетена 6910 пъти)