[PHP] Zbędne wyświetlenie tablicy 2 raz Opcje

[!*!]

[PHP] pobierz, plaintext 
class Lang
{
	private $prefix;
	private $lines = array();
 
	private static $instance = false;
 
	public function __construct($prefix)
	{
		$this->prefix = $prefix;
		$this-> set_lang();
	} // end __construct
 
	private function instance()
	{
		//echo $prefix;
		if(self::$instance == false)
		{
			self::$instance = new Lang(DEFAULT_LANG);
		}
	return self::$instance;
	} // end instance
 
	private function set_lang()
	{
		if(isset($_GET['lang']))
		{
			//echo $prefix;
			$this->prefix = strtolower($_GET['lang']);
			//$_SESSION['lang'] = $this->prefix;
			//echo 'wybrano:.....'.$this->prefix;
		}
		elseif(isset($_SESSION['lang']))
		{
 
			$this->prefix = strtolower($_SESSION['lang']);
			$_SESSION['lang'] = $this->prefix;
		}
		else
		{
		$this->prefix = strtolower(DEFAULT_LANG);
		}
 
		$this-> load_lang_file();
	} // end set_lang
 
	private function load_lang_file() 
	{ 
 
		if(file_exists('./lang/'.$this->prefix.'.lang'.EXT)) 
		{ 
			//echo 'istnieje';
			include './lang/'.$this->prefix.'.lang'.EXT;
			$this->lines = $lang; 
			echo '<pre>';
			print_r($this->lines); 
		} 
	}	// end load_lang_file
 
	public static function get($key = false)
	{
	//var_dump($key);
		//echo '<span style=color:red>'.$key.'</span>';
		//return vsprintf(self::instance()-> lines[$key], $params);
 
		if(isset(self::instance()->lines[$key]))
		{
			return vsprintf(self::instance()-> lines[$key], false);
		}
		else
		{
  			return false;
  		}
 
	}// end get
 
} // end class Lang
[PHP] pobierz, plaintext 

Dlaczego kod

[PHP] pobierz, plaintext 
echo '<pre>';
print_r($this->lines); 
[PHP] pobierz, plaintext 

w metodzie load_lang_file() wyświetla dwa razy tą samą tablice?

Plik z tłumaczeniem wygląda tak:

[PHP] pobierz, plaintext 
$lang = array(
	'mobile_version' => 'wersja mobilna',
	'site_map' => 'mapa witryny',
	'contact' => 'kontakt'
);
[PHP] pobierz, plaintext 

Ten post edytował !*! 8.09.2011, 08:05:08

[Daiquiri]

A co zawiera $lang powyżej?

[!*!]

Tablicę $lang z pliku.

[Daiquiri]

OK, a jak robisz print_r($lang) w miejscu print_r($this->lines) to też jest podwójnie?

[!*!]

Tak, nadal są dwie. hmm może coś jest nie tak z self::instance()->lines[$key] i vsprintf, tylko za bardzo nie wiem co.

Ten post edytował !*! 8.09.2011, 08:15:46

[Daiquiri]

[PHP] pobierz, plaintext 
 if (self::$instance->lines[$key]) {
    return vsprintf(self::$instance->lines[$key], FALSE);
}
[PHP] pobierz, plaintext 

To powinno być ok. Pytanie skąd bierzesz $lang ($this->lines = $lang;)? Na Twoim miejscu "wracałabym" się o krok, żeby wyłapać gdzie jest błąd.

@Nospor:
Co racja to racja

Powód edycji: [Daiquiri]:

[nospor]

A może rozwiązanie jest banalne, jak to bywa w 99% takich przypadków na forum:
odpalasz ten kod dwukrotnie, to się wyświetla podwójnie

[!*!]

Daiquiri - $this->lines = $lang; $lang to tablica w pliku jaki pobieram.
nospor - faktycznie też to przed chwilą zauważyłem

Kod

new Lang('pl');
echo Lang::get('mobile_version');

Spowoduje wyświetlenie dwa razy, jeśli pominę new Lang będzie ok, jednak zdawało mi się że właśnie od tego jest $instance aby nie "ładowało" klasy 2 razy.

[nospor]

jednak zdawało mi się że właśnie od tego jest $instance aby nie "ładowało" klasy 2 razy.

Bo i do tego jest, ale trzeba to tez odpowiednio używać.
A skoro ty ręcznie robisz NEW no to pomijasz swoje instance. Potem w instance znowu robisz NEW i masz dwa razy.

Jak już używasz instance to nie rób już
new Lang('pl');
bo przecież tu ręcznie tworzysz nowy obiekt.

[Daiquiri]

Może zrób prywatny konstruktor i nie będziesz mógł się "pomylić".