Co źle robie?

Question

function translate($text)
{
	$jezyk_uzytkownika = substr($_SERVER['HTTP_ACCEPT_LANGUAGE'], 0, 2);   // naucz sie tych 0,2
	if($jezyk_uzytkownika == "pl")
	{
		$jezyk_uzytkownika_nazwa = "Polski";
	}elseif($jezyk_uzytkownika == "de")
	{
		$jezyk_uzytkownika_nazwa = "Deutsch";
	}else
	{
		$jezyk_uzytkownika_nazwa = "English";
	}

	$strefa = date("e");
	$pl = [
        'witaj' => 'Witaj',
        'wybierz_plik' => 'Wybierz plik do wczytania.',
		 'historia_plikow' => 'Historia wczytanych plików ',
		   'nazwa' => 'Nazwa Pliku',
		     'czas_wczytania' => 'Czas wczytania pliku [s]',
			   'wybierz_jezyk' => '<b>Wybierz język</b>',
			     'wyslij' => 'Wyślij plik',
				   'strefa' => '<b>Twoja strefa czasowa: </b>'.$strefa,
				     'jezyk_uzy' => '<b>Twoj język: </b>'.$jezyk_uzytkownika_nazwa
    ];
    $en = [
       'witaj' => 'Hello',
        'wybierz_plik' => 'Select the file to load',
		  'historia_plikow' => 'History of loaded files',
		    'nazwa' => 'File name',
		     'czas_wczytania' => 'Time to load the file [s]',
			   'wybierz_jezyk' => '<b>Choose language</b>',
			     'wyslij' => 'Send a file',
				   'strefa' => '<b>Your time zone: </b>'.$strefa,
				     'jezyk_uzy' => '<b>Your language: </b>'.$jezyk_uzytkownika_nazwa
    ];
	if($_GET['lang'] == "pl")
	{
        return $pl[$text];
    }
	elseif($_GET['lang'] == "en")
	{
		return $en[$text];
	}
	elseif(substr($_SERVER['HTTP_ACCEPT_LANGUAGE'], 0, 2) == "pl")
	{
		return $pl[$text];
	}
	elseif(substr($_SERVER['HTTP_ACCEPT_LANGUAGE'], 0, 2) == "en")
	{
		return $en[$text];
	}
	else
	{
		return $en[$text];
	}
   
    return $text;
}

Ta jak w temacie. Program ma tłumaczyć strone ale chciałem też tak ustawić że jak ktoś nie kliknie przycisku z językiem to automatycznie mu pobiera jego jezyk i ustawia na jego. Ale coś mi nie działa.. a błąd to

"

Notice: Undefined index: lang in D:\xampp\htdocs\Czytnik\index.php on line 63

"

Answer 1

Troche przekombinowałeś.

<?php

function getTranslations(): array
{

    return [
        'pl-PL' => [
            'witaj' => 'Witaj',
            'wybierz_plik' => 'Wybierz plik do wczytania.',
            'historia_plikow' => 'Historia wczytanych plików ',
            'nazwa' => 'Nazwa Pliku',
            'czas_wczytania' => 'Czas wczytania pliku [s]',
            'wybierz_jezyk' => '<b>Wybierz język</b>',
            'wyslij' => 'Wyślij plik',
            'strefa' => '<b>Twoja strefa czasowa: </b>', //nie możemy tego ustalić z poziomu php
            'jezyk_uzy' => '<b>Twoj język: </b>Polski',
        ],
        'en-US' => [
            'witaj' => 'Hello',
            'wybierz_plik' => 'Select the file to load',
            'historia_plikow' => 'History of loaded files',
            'nazwa' => 'File name',
            'czas_wczytania' => 'Time to load the file [s]',
            'wybierz_jezyk' => '<b>Choose language</b>',
            'wyslij' => 'Send a file',
            'strefa' => '<b>Your time zone: </b>',//nie możemy tego ustalić z poziomu php
            'jezyk_uzy' => '<b>Your language: </b>English',
        ],
    ];
}

function translate($text): string
{
    $lang = $_GET['lang'] ?? $_SERVER['HTTP_ACCEPT_LANGUAGE'] ?? '';
    $translations = getTranslations();
    switch ($lang) {
        case 'pl-PL':
            return $translations['pl-PL'][$text] ?? '';
        case 'en-US':
        case 'en-GB':
        default:
            return $translations['en-US'][$text] ?? '';
    }
}

 

Comments

echo translate($text):

 

---

Jest taki problem. Gdyż miało być tak, że jak ktoś wchodzi pierwszy raz to ustawia mu język taki jaki mu pobierze z $_SERVER['HTTP_ACCEPT_LANGUAGE'] a później jeśli sam klinie w button to mu się zmieni. I przy "Twój język" wlasnie tam miało być jego jakby wczytany a nie aktualny. z $_SERVER['HTTP_ACCEPT_LANGUAGE']. Da się jakoś to ustawić?

---

Proszę przykład: http://sandbox.onlinephpfunctions.com/code/8a12473a14fac6813ec4defa52e6065b2a59648d

---

Nie rozumiem go trochę. Na czym polega te 3 echo?

---

1. to przypadek, gdy nie masz ustawionego GET, ani nie ma headera ACCEPT_LANGUAGE

2. to przypadek z ustawionym tylko headerem

3. to przypadek z ustawionym GETem i headerem

Simple as far?

Answer 2

nie sprawdzasz czy  w ogóle istnieje lang

Answer 3

przed ciągiem if i else if daj if(isset($_GET['lang']))

problem polega na tym, że zawsze próbujesz wyciągnąć wartość z GET, a nie zawsze ona tam jest

+dlaczego nie połączysz tego z bazą i tekst trzymasz w zmiennych?
+mógłbyś tutaj użyć instrukcji switch() zamiast else if w kilku miejscach

+polecam zajrzenie do zasad SOLID, np zasady pojedynczej odpowiedzialności
Spokojnie można to rozdzielić na metodę sprawdzającą jaki język jest wybrany i metodę wybierającą odpowiedni tekst
Wtedy mógłbyś z wielu miejsc w kodzie sprawdzić jaki język jest na stronie oraz z wielu miejsc mógłbyś wyświetlić tekst w określonym języku, nie wiadomo jakie będą potrzebne aktywności w aplikacji za kilka tygodni
+dużo prościej to przetestować jednostkowo

Comments

heh, to samo mu napisałem :|

Baza jest niepotrzebna. Może tworzy aplikacje bez połączenia z bazą? Taki WP opiera się o l18n, ale niektóre frameworki tworzą arraye z tłumaczeniami.

---

Isset to zły pomysł. Lepiej użyć array_key_exists albo filter_Input.

---

Ehlert, dlaczego tak sądzisz?

---

Get to tablica i isset dla nulla zwraca przekłamania. Zawsze jej unikałem i z powodzeniem można ją zastąpić.

---

dlatego powinien użyć !empty :)

---

Zmieniam zdanie. Powinien być filter_input. jest to funkcja przeznaczona do pracy ze zmiennymi od strony klienta i obsługuje przypadku braku ustawienia itp. 

Answer 4

Dziękuje wam wszystkim za pomoc!