PHP OOP i strona wielojęzyczna, pytanie początkującego Opcje

[szczypior]

index.php

[PHP] pobierz, plaintext 
include 'classes/lang.class.php';
 
 
$lang = new lang_set($_GET['lang']);
 
include_once 'lang/'.$lang;
[PHP] pobierz, plaintext 

plik lang.class.php

[PHP] pobierz, plaintext 
<?php
class lang_set{
 
private $lang;
 
	public function __construct($lang_in){
 
		// gdy mamy zmienną, to znaczy że użytkownik chce zmienić obecny język
		If(!empty($lang_in))
		{
 
			switch ($lang_in) {
			  case 'en':
			  $this->lang = 'en';
			  break;
 
			  case 'pl':
			  $this->lang = 'pl';
			  break;
 
			  default:
			  $this->lang = 'pl';
			}
			$_SESSION['lang'] = $lang;
			setcookie('lang', $lang, time() + (3600 * 24 * 30));
		}
		else if(isSet($_SESSION['lang']))
		{
			//gdy mamy zmienną sesyjną, to ją odczytujemy
			$this->lang = $_SESSION['lang'];
		}
		else if(isSet($_COOKIE['lang']))
		{
			// gdy użytkownik ma ciasteczko to odczytujemy zapisany w nim język i zapisujemy dodatkowo do zmiennej sesyjnej
			$this->lang = $_COOKIE['lang'];
			$_SESSION['lang'] = $_COOKIE['lang'];
		}
		else
		{
			//domyślny język
			$this->lang = 'pl';
		}
	}
 
	public function __toString()
    {
       return $this->lang.'.lang.php';
    }
}
?>
[PHP] pobierz, plaintext 

Ten post edytował szczypior 12.08.2016, 13:00:46

[nospor]

Komentarze zaczyna sie poprzez
//
a nie
'

Twoj kod z obiektowka nie ma zadnego zwiazku - przenosze.

Trzymanie jezyka w sesji czy w ciachu to nieporozumienie. Jezyk ma byc w url, najlepiej w ladnym url.
I tak strona pl moze wykladac tak:
http://mojastrona.pl/

zas wersja angielska juz tak:
http://mojastrona.pl/en/

Odpalaj swoj kod na wlaczonym wyswietlaniu wszystkich bledow

ps:
'lang/pl.lang.php'
czy naprawde plik z jezykiem, ktory lezy w katalog LANG musi miec jeszcze sufix LANG? Czyz nie jest to dosc oczywiste z faktu ze lezy w katalogu LANG i ma nazwe PL czy EN ?

[szczypior]

Dzięki za opinię.

Dlaczego uważasz, że nie jest on w ogóle obiektowy? Co powinienem zmienić, żeby go "uobiektowić"?

[nospor]

Wsadzenie kawalka kodu w klase, a tak naprawde w konstruktor, nie czyni z niego OOP tylko poprostu kod w klasie

[szczypior]

Czego brakuje temu kodowi, by faktycznie był obiektowy?

Wszystko jest w konstruktorze, bo ta klasa wydawała mi się na tyle mało rozbudowana, że nie potrzebuje dodatkowych funkcji. Jak mniemam, myliłem się

Będę bardzo wdzięczny za sugestie

Abstrahując od tego, że faktycznie zmianę języka można rozwiązać w lepszy sposób (tak jak sugerujesz przez htaccess), to czy możesz wskazać kierunek, jak należałoby poprawić ten kod, by stał się obiektowym.
Czy właśnie należałoby go podzielić na funkcje?

Z góry dzięki za odpowiedź

Ten post edytował szczypior 12.08.2016, 13:13:20

[nospor]

Wybacz, moze troche wprowadzilem cie w blad.
OOP - Object Oriented Programming
W dziale OOP zajmujemy sie (a przynajmniej powinnismy) troche bardziej powazniejszymi sprawami jak klasa, ktorej jedyne zadanie to ustawienie jezyka na podstawie parametru w GET. To z OOP nie ma o tyle zwiazku, ze tutaj zastosowanie klasy nic nie w nosi. Od kod strukturalny wlozyles w klase. Zasadza dzialania kodu sie nie zmienila.

Na chwile obecna masz klase, masz obiekt - starasz sie pisac obiektowo a nie strukturalnie - to juz cos.

Nie mniej jednak wypadaloby sie troche zapoznac z roznymi konwencjami, np PSR i nazewnictem klass i plikow klas.
Ot prosty przyklad:
Twoj plik z klasa nazywa sie lang.class.php a sama klasa lang_set
1) wyrzuc class z nazwy pliku
2) Plik ma sie nazywac tak samo jak klasa
3) plik Lang.php, klasa Lang

Jak zaczniesz lekture PSR to wpadniesz na cos takiego jak autoload - przydatne cudo

[viking]

W samej klasie masz co najmniej kilka funkcjonalności. Pierwsze to przekazanie i odczyt wartości języka. Tu dodatkowo jeśli byś miał 20 języków to taki switch mocno by sie rozrosł. Nie można z tej klasy wyciągnąć bieżącej wartości języka. Drugi to adapter storage który akurat jest sesją domyślną ale gdybyś chciał zmienić na jakiś komponent zewnętrzny wtedy cały kod jest do przepisania a nie tylko dany fragment. Dodatkowo na sztywno czas sesji którego nie można zmienić przez konfigurację. O fatalnym nazewnictwie nospor już pisał.

[kpt_lucek]

[PHP] pobierz, plaintext 
class Lang
{
	/** @var LangResolverInterface  */
	protected $langResolver;
 
	/** @var  string */
	protected $lang;
 
	/**
	 * Lang constructor.
	 *
	 * @param LangResolverInterface $langResolver
	 */
	public function __construct(LangResolverInterface $langResolver)
	{
		$this->langResolver = $langResolver;
		$lang               = $langResolver->getDefaultLang();
	}
 
	/**
	 * @param string $lang
	 *
	 * @return $this
	 */
	public function changeLang($lang)
	{
		$resolvedLang = $this->langResolver->resolve($lang);
		if (null === $resolvedLang) {
			throw new \InvalidArgumentException(sprintf('Lang "%s" is not supported.', $lang));
		}
		$this->lang = $resolvedLang;
 
		return $this;
	}
}
[PHP] pobierz, plaintext 

Możesz coś podobnego przekminić, wydziel logikę odpowiedzialną za "sprawdzenie" języka gdzieś na zewnątrz, dispatchuj event na zmianę języka.

Opcji są miliony

gdy użytkownik ma ciasteczko to odczytujemy zapisany w nim język i zapisujemy dodatkowo do zmiennej sesyjnej

Wywal to do osobnego obiektu, który w razie potrzeby odpali

[PHP] pobierz, plaintext 
->setLang($lang);
[PHP] pobierz, plaintext 

na klasie Lang, z założenia Lang nie musi wiedzieć kto, skąd i po co zmienia język, ma to tylko ogarnąć po swojej stronie i tyle (plus ewentualnie powiadomić jakimś eventem że zmiana miała miejsce)

gdy mamy zmienną, to znaczy że użytkownik chce zmienić obecny język

Może abstrakcja, ale... co jeżeli pewna część kontentu będzie w innym języku? W teorii muszę utworzyć nową instancję klasy Lang po to, żeby w tym momencie ten język zmienić?

[PHP] pobierz, plaintext 
return $this->lang.'.lang.php';
[PHP] pobierz, plaintext 

Może potraktuj tłumaczenia (zakładam strukturę key=>value) raczej jako config? W sensie *.ini, *.yml, xml itd.
Dodatkowo, możesz napisać sobie coś w rodzaju Translatora, co za tym idzie... mówić mu aby tłumaczył klucze na uprzednio załadowanej konfiguracji języka.
Do tego musisz oczywiście pamiętać o fallback'u - w momencie gdy dane tłumaczenie (klucz) nie istnieje.


P.S. taki troche Symfony-way

Ten post edytował kpt_lucek 13.08.2016, 02:55:57

[IAmBoskiM]

[PHP] pobierz, plaintext 
else if
[PHP] pobierz, plaintext 

To nie JS.

[PHP] pobierz, plaintext 
elseif ($warunek) {
}
// albo
else {
   if ($warunek) {
   }
}
[PHP] pobierz, plaintext 

@Down Huh, no rzeczywiście. Myślałem, że to zawsze jest błąd. Ale osobiście i tak odradzam stosowanie tego jako dwa słowa, bo przy klamrach działa, a przy dwukropkach nie, a elseif działa przy wszystkim.

Ten post edytował IAmBoskiM 14.08.2016, 16:24:30

[Comandeer]

No to przecież else if jest równoznaczny z Twoim drugim kodem bez klamerek → http://php.net/manual/pl/control-structures.elseif.php