Chciałbym zmieniać język na stronie z POL na ENG i odwrotnie.

1. Jak byście doradzali aby się do tego zabrać ?

2. Czy mój POMYSŁ jest OK?
POMYSŁ: jedyne co na razie przyszło mi do głowy to:
Zmiana jeżyka odbywa się po wciśnieciu jednego z dwóch przycisków POL lub ENG na stronie głównej o przenosi do pliku w którym wykonuje się skryp gdzie łączzy się z bazą danych i w zależności od wybranego języka pobiera wszystkie wiersze z bazy mysql i przypisuje w zmienne sesyjne, które są ustawiane na stronie gdzie powinien być tekst.

I teraz zmieniasz jakis tekst bo cos tam i user zobaczy zmiane dopiero na drugi dzien jak mu sesja wygasnie...

Tekst ma byc pobierany w momencie wyswietlania a nie w zadnej sesji. Mozesz oczywiscie trzymac go w jakims cache by za kazdym razem nie latac do bazy.
Zas samo info o jezyku ma byc zawarte w linku a nie jednorazowej akcji. np:
en.mojastrona.pl/
lub
mojastrona.pl/en

ok, to popraw mnie jeśli źle rozumiem

[HTML] pobierz, plaintext 
<a href="polski.php">Zmień na polski </a>
[HTML] pobierz, plaintext 

i tak samo z ang
a w plikach polski.php etc. trzymać skrypt który pobiera z bazy słówka i wyświetla w cache ?
nigdy nie bawiłem sie z cache to nie wiem jak to wygląda dodaamm.

Jeżeli mogę się wtrącić, to uważam, że adres powinien przekazywać tylko jaki to jest język, a skrypt uniwersalny.

@emstawicki w ogole nie powinno byc zadnego skryptu.

@djtomaszq napisalem przeciez ze nie ma byc twojego zadnego skryptu na zmiane jezyka.
Jezyk masz miec w adresie strony zas potem gdzies tam w kodzie funkcja, ktora zwraca ci tekst dla danego jezyka

[PHP] pobierz, plaintext 
blabla <?php echo trans('jakisklucz') ?>blabla
[PHP] pobierz, plaintext 

I funkcja trans() na podstawie jezyka w linku zwroci ci tekst dla klucza "jakisklucz". I czy ona to pobierze z bazy czy z cache to juz jej sprawa, ciebie nie interesuje

Łapie o co chodzi ale powoli...

w bazie mysql tabeli translator kolumny:
klucz | PL | ENG
gracz | zawodnik | player

[PHP] pobierz, plaintext 
function ($tekst)
{
  <a href="index.php/?lang=PL">PL</a>
 
  //laczenie z baza itp
  $zapytanie = $pol->query("SELECT '$_GET['lang']' FROM translator WHERE klucz = '$tekst'");
  //przypisuje
  ...
  <?php echo trans('gracz') ?>
}
[PHP] pobierz, plaintext 

.. coś trafiłem czy dalej leżakuje w tym przedszkolu?

Napisz to porzadnie a nie jakis pseudo kod ktory teraz wyglada fatalnie i trudno stwierdzic czy zalapales czy nie

Nie PL a pl

I tabela nie ma miec kolumn PL EN tylko ma miec kolumny
key | lang | text

przykladowe rekordy
key1 | pl | klucz1
key1 | en | key1
key2 | pl | klucz2
key2 | en | key2

[PHP] pobierz, plaintext 
<?php
	function trans($text)
	{
		require_once "connect.php";
 
		try{
			$polaczenie = new mysqli($host, $db_user, $db_password, $db_name);
			if($polaczenie->connect_errno !=0)
			{
				throw new Exception($polaczenie->mysqli_connect_errno());
			}else
			{
				$lang = $_GET['strona'];
				if(!$zapytanie = $polaczenie->query("SELECT text FROM translate WHERE lang = '$lang' AND key = '$text'"))
				{
					throw new Exception($polaczenie->error);
				}else
				{
					$wiersz = $zapytanie->fetch_assoc;
				}
 
				$polaczenie->close();
			}
		}catch(Exception $e)
		{
			echo $e;
		}
 
		return($wiersz['text']);
	}
 
	echo trans('key1');
?>
 
 
[PHP] pobierz, plaintext 

I za każdym razem wykonujesz zapytanie do bazy, a co gorsze za każdym razem otwierasz połączenie z bazą.
Wyobraź sobie, że na stronie masz do przetłumaczenia chociażby 100 etykietek.

Lepiej pobierz raz wszystkie tłumaczenia do tabeli i odczytuj w razie potrzeby już właśnie z niej.

@nospor
Chodziło mi bardziej o to żeby skrypty/modele, które wczytują dane były uniwersalne - tym bardziej jeżeli chodzi potem o wielojęzykowość innych treści.
@trueblue ma rację. Pobierz wszystkie zmienne językowe

@ djtomaszq tak, teraz lepiej i teraz juz tylko to optymalizowac
Jak juz wspomniano dane masz pobrac raz a potem ciagnac je z tabeli juz pobranej by nie latac do bazy po kazdy klucz.
Skoro korzystasz z funkcji, to mozesz zapisac to w zmiennej statycznej w funkcji za pierwszym razem, a potem odwolywac sie juz do tej zmiennej
Mozesz tez zapisac to w cache by przy kazdym request do strony nie wczytywac tego i tak ponownie z bazy.
Jesli tekstow bedziesz mial duzo mozesz pokusic sie podzielic to na moduly jeszcze i wowczas zamias
key
bedziesz mial
module.key
I zamiast pobierac wszystkie teksty bedziesz pobieral tylko dla danego modulu

@emstawicki wybacz, zrozumialem ze mowisz o tych dwoch konkretnych skryptach polski.php oraz english.php

ok czyli tak by to wyglądało lepiej ?

[PHP] pobierz, plaintext 
<?php
		require_once "connect.php";
 
		try{
			$polaczenie = new mysqli($host, $db_user, $db_password, $db_name);
			if($polaczenie->connect_errno !=0)
			{
				throw new Exception($polaczenie->mysqli_connect_errno());
			}else
			{
				$lang = $_GET['strona'];
				if(!$zapytanie = $polaczenie->query("SELECT text, klucz FROM translate WHERE lang = '$lang'"))
				{
					throw new Exception($polaczenie->error);
				}else
				{
					$ile = $zapytanie->num_rows;
 
					for($i=0; $i < $ile; $i++)
					{
						$wiersz[$i] = $zapytanie->fetch_assoc();
					}
				}
 
				$polaczenie->close();
			}
		}catch(Exception $e)
		{
			echo $e;
		}
 
		function trans($text)
		{
			global $ile;
			global $wiersz;
 
			for($i=0; $i < $ile; $i++)
			{
				if($wiersz[$i]['klucz'] == $text) return $wiersz[$i]['text'];
			}
		}
 
	echo trans('key1');
?>
 
 
[PHP] pobierz, plaintext 

tylko jak to teraz zrobić żebyna całej stronie gdzie jest kilka podstron zmieniał się jezyk ?
narazie mam tak:

[HTML] pobierz, plaintext 
 
                                <div class="przyciski_logowania">
					<a class="link" href="index.php?strona=en">Angielski</a>
				</div>
 
				<div class="przyciski_logowania">
					<a class="link" href="index.php?strona=pl">Polski</a>
				</div>
[HTML] pobierz, plaintext 

i ten plik z funkcją muszę includować do każdej podstrony ?

kluczem twojej tablicy ma nie byc $i tylko key z bazy. Wowczas by dobrac sie do danego klucza robisz poprostu $tablica[$klucz] i juz.
Poza tym pobieranie z bazy mialo byc w funkcji a nie gdzies oddzielnie. Mowilem bys uzyl zmiennej statycznej w funkcji i jesli ona jest pusta to masz pobrac dane z bazy i zapisac do tej tablicy.

Jak teraz? ...

[PHP] pobierz, plaintext 
<?php
	function trans($text)
	{
		static $tablica;
 
		if(empty($tablica))
		{
			require_once "connect.php";
			mysqli_report(MYSQLI_REPORT_STRICT);
 
			try
			{
				$polaczenie = new mysqli($GLOBALS['host'], $GLOBALS['db_user'], $GLOBALS['db_password'], $GLOBALS['db_name']);
 
				if($polaczenie->connect_errno != 0)
				{
					throw new Exception($polaczenie->mysqli_connect_errno());
				}else
				{
					$lang = $_GET['strona'];
 
					if(!$zapytanie = $polaczenie->query("SELECT klucz, text FROM translate WHERE lang = '$lang'"))
					{
						throw new Exception($polaczenie->error);
					}else
					{
						while ($wiersz = $zapytanie->fetch_assoc())
						{
							$tablica[$wiersz['klucz']] = $wiersz['text'];
						}
					}
 
					$polaczenie->close();
				}
			}catch(Exception $e)
			{
				echo $e;
			}
		}
 
		return $tablica[$text];
	}
 
	echo trans('key1')."<br />";
	echo trans('key2')."<br />";
	echo trans('key3')."<br />";
?>
 
 
[PHP] pobierz, plaintext 

Coraz lepiej

Jeszcze tylko nie tworz do funkcji oddzielnego obiektu bazy danych, tylko przekaz obiekt, ktory juz kiedys tam utworzyles - bo na pewno juz nie raz na stronie utworzyles taki obiekt? Dotyczy to nie tylko tej jednej funkcji ale wszystkiego innego - obiekt bazy danych masz tworzyc tylko raz i przekazywac wszedzie tam gdzie jest potrzebny a nie za kazdym razem tworzysz kolejny obiekt bazy danych

.. hm czyli jeśli dobrze rozumiem mam plik index.php i tam tylko RAZ tworze połączenie a dalej w tym pliku tylko się odwołuje i mam plik główna.php to też musze RAZ utworzyć obiekt i dalej w tym pliku tylko się odwoływać? Czy może wystarczy w index.php utworzyć obiekt a w główna.php już tylko się odwoływać ?

no ale tej ostatniej rzeczy czyli zrobienie odwołania nie wiem jak się zabrać wiem że chodzi o linijkę

[PHP] pobierz, plaintext 
$polaczenie = new mysqli($GLOBALS['host'], $GLOBALS['db_user'], $GLOBALS['db_password'], $GLOBALS['db_name']);
[PHP] pobierz, plaintext 

tak? czyli pierwsze tworzenie wyglądałoby tak jak na powyższym skrypcie a kolejne w tym pliku zamiast tej linijki muszę się ("jakoś" bo jeszcze nie wiem jak) się odwołać?

ok, z tym jakoś sobie może poradzę.

.. ale wracając do problemu z tematu zostało mi odwołanie od kliknięcia wyboru języka, mam to teraz zrobione tak:

[HTML] pobierz, plaintext 
<div class="przyciski_logowania">
	<a class="link" href="pl">Polski</a>
</div>
<div class="przyciski_logowania">
	<a class="link" href="en">Polski</a>
</div>
[HTML] pobierz, plaintext 

no i z powyższym skryptem działa po kliknięciu tłumaczenie ale mam też inne strony i podstrony na portalu i jak kliknę w menu w inny link i się otworzy inna strona to tłumaczenie znika a nie chce za każdym razem na każdej stronie klikać pl lub en tylko żeby odrazu dla całej witryny tłumaczyło język.

Czy mam to zrobić podczepiając tego $_GET['lang'] do każdego linku na stronie ?

No mowilem ci na samym poczatku, ze kazdy link ktory generujesz ma zawierac jezyk

Wiec jesli do tej pory generowales link np tak:

<a class="link" href="costam.php">Costam</a>

To teraz ma wygladac tak:
<a class="link" href="costam.php?lang=LANG KTORY AKTUALNIE OBOWIAZUJE">Costam</a>

Zadanie byloby banalnie proste jakbys od samego poczatku uzywal funkcji do generowania linkow, np:

<a class="link" href="<?=linkGenerator('costam')?>">Costam</a>

I funkcja sama za ciebie by dodala teraz jezyk. A ze nie robiles tego to teraz trzeba pocierpiec. Ale nie martw sie, kazdy przez to przechodzil

Ok rozumiem, myślałem linkach ale zrobię to w liście wyboru.

A czy w skrypcie trans ten get jest bezpieczny?

[PHP] pobierz, plaintext 
$lang = $_GET['strona'];
if(!$zapytanie = $polaczenie->query("SELECT klucz, text FROM translate WHERE lang = '$lang'")
[PHP] pobierz, plaintext 

.. A co do generatora linków pewnie sam tego teraz nie zrobię? bo w googlach nic nie znalazłem albo źle szukalem

Wpisz se w adresie
?strona=' OR '1
to będziesz wiedział

Poczytaj o SQLInjection. A skoro uzywasz mysqli to poczytaj o bindowaniu

Co do generowania to zalezy jakie masz linki i czy wszystkie na jeden wzor.

Linki mam takie

[HTML] pobierz, plaintext 
<a href="index.php?strona=podstrona"></a>
[HTML] pobierz, plaintext 

i w przyszłości Będzie na pewno coś takiego

[HTML] pobierz, plaintext 
<a href="index.php?strona=coś&gra=3&id=1">mecz</a>
 
[HTML] pobierz, plaintext 

No i używam przyjaznych linków

No to twoja funkcja jest banalnie prosta do napisania. Tak na baaaardzo szybko:

[PHP] pobierz, plaintext 
function linkGenerator($param) {
$lang = isset($_GET['lang']) ? $_GET['lang'] : 'pl';
if (!in_array($lang, array('pl', 'en'))) {
$lang = 'pl;'
}
 
return "index.php?strona={$param}&lang=$lang";
}
 
[PHP] pobierz, plaintext 

A potem w kodzie

<a class="link" href="<?=linkGenerator('podstrona')?>">Costam</a>
<a class="link" href="<?=linkGenerator('strona=cos&gra=3&id=1')?>">Costam</a>

A czy ma znaczenie jeśli pozamieniam miejscami
?strona=coś&lang=pl
Na
?lang=pl&strona=coś
?

No widac jakies ma skoro chcesz zamieniac

tylko po to żeby wygladalo tak host.pl/en/strona a nie host.strona/en
czyli język zawsze był pierwszy

hm, no to teraz mnie zagiales. To albo generujesz index.php?..&...&... albo strona.pl/en/sth. Sie zdecyduj

No nauczyłem się przyjaznych linków i w .htaccess robie przyjazne linki i używam w linku

[HTML] pobierz, plaintext 
<a href="zaloguj">Logowanie</a>
<a href="glowna">Strona startowa</a>
 
a w przyszłości
<a href="pl/glowna">Strona startowa</a>
<a href="pl/mecze/1liga">1 liga</a>
[HTML] pobierz, plaintext 

ups...

no to jak zaczniesz uzywac przyjaznych linkow to wtedy zmienisz. Teraz to nie ma znaczenia.
A najlepsze jest to, ze jak juz zrobisz to na tej funkcji to pozniej zmiana na przyjazne linki to bedzie poprawka w tej jednej funkcji a nie w calym projekcie

No to już w sumie zmieniłem wszystko na przyjazne linki

.htaccess

[PHP] pobierz, plaintext 
RewriteEngine On
 
RewriteRule ^([a-z]+)$ index.php?strona=$1
[PHP] pobierz, plaintext 

samą funkcje trans mam w osobnym pliku trans.php (czy tak powinno być czy w głównym index.php ją trzymać?)

podstrony includuje przez

[PHP] pobierz, plaintext 
SWITCH($_GET['strona'])
			{
				case "logowanie":
					require_once "logowanie.php";
				break;
 
				case "rejestracja":
					require_once "rejestracja.php";
				break;
 
				case "glowna":
					require_once "pliki/glowna.php";
				break;
 
				case "pl" or "en":
					require_once "trans.php";
				break;
 
				default:
					require_once "pliki/glowna.php";
				break;
			}
[PHP] pobierz, plaintext 

case "pl" or "en":
require_once "trans.php";

A po co ci to?

To było tylko tak do sprawdzenia czy działa funkcja tłumaczenia osobną stronę zrobiłem do tego ale już to usuwam tego nie ma...

No dobra, ale wtwoim przyjaznym linku brakuje teraz lang a przynajmniej tak wynika z htaccess

Właśnie do tego doszedłem i widzę problem ...
a wszystko przez głupotę bo na szybko sprawdzałem to jako $_GET['strona'] ze wszystkimi innymi podstronami w tym SWITCH.

teraz czy mam zmiane języka robić w przycisku jako..?

[PHP] pobierz, plaintext 
<a href="trans.php?lang=pl">Polski</a>
[PHP] pobierz, plaintext 

EDIT a linijke w htaccess

[PHP] pobierz, plaintext 
RewriteRule ^([a-z]+)/([a-z]{2})$ index.php?strona=$1&lang=$2
[PHP] pobierz, plaintext 

EDIT 2: Czy może być tak?

Linki do zmiany języka:

[PHP] pobierz, plaintext 
<div class="przyciski_logowania">
						<a class="link" href="test-en">Angielski</a>
					</div>
 
					<div class="przyciski_logowania">
						<a class="link" href="test-pl">Polski</a>
[PHP] pobierz, plaintext 

EDIT 2: Czy może być tak?

Linki do zmiany języka:

[PHP] pobierz, plaintext 
<div class="przyciski_logowania">
						<a class="link" href="test-en">Angielski</a>
					</div>
 
					<div class="przyciski_logowania">
						<a class="link" href="test-pl">Polski</a>
[PHP] pobierz, plaintext 

EDIT 2: Czy może być tak?

Linki do zmiany języka:

[PHP] pobierz, plaintext 
                                        <div class="przyciski_logowania">
						<a class="link" href="test-en">Angielski</a>
					</div>
 
					<div class="przyciski_logowania">
						<a class="link" href="test-pl">Polski</a>
[PHP] pobierz, plaintext 

.htacces

[PHP] pobierz, plaintext 
RewriteRule ^([a-z]+)-([a-z]{2})$ index.php?strona=$1&lang=$2
[PHP] pobierz, plaintext 

i funkcję trzymam samą w pliku trans.php a require_once dołączam go na górze do index.php

Czy tak będzie dobrze?

EDIT 3: a robię tak: test-en zamiast test/en bo wtedy psuje mi css'a i nie pamiętam jak to naprawić

Nie zadne test-en tylko test/en a najlepiej by jezyk byl pierwszy czyli en/test

Co do problemu z css i js to sciezki do css daje sie pelne a nie niepelne - temu ci nie dziala
Nie: style.css
tylko: http://mojastrona.pl/style.css
Od biedy: /style.css

Ok wszystko fajnie działa tylko jak wchodzę na stronę tak: localhost/mojastrona to wyskakuje błąd że nie ma zmiennej GET. Klikając linki to już wszystko działa tylko przy tym pierwszym zalogowaniu jest błąd. Czy wystarczy jak.do index.php dodam że jeśli nie istnieje $_GET['lang'] to przypisuje jej 'pl'?

... albo jeśli taka zmienna nie istnieje przekieruj na adres tylko że z zmienną get o danej wartości.

Przeciez w kodzie dalem sprawdzanie czy zmienna w get istnieje czy nie
$lang = isset($_GET['lang']) ? $_GET['lang'] : 'pl';

cos mnie wacpan sciemniasz

Ja nie, chyba że mój kod...

index.php (fragment)

[PHP] pobierz, plaintext 
<?php
	session_start();
 
	require_once "connect.php";
	require_once "trans.php";
 
	$http = 'http://localhost/strona';
 
	if(!isset($_GET['strona']))
	{
		$_GET['strona'] = "";
	}
 
 
 
	function linkGenerator($param) 
	{
		global $http;
 
		$lang = isset($_GET['lang']) ? $_GET['lang'] : 'pl';
		if (!in_array($lang, array('pl', 'en'))) {
		$lang = 'pl';
		}
 
		return "$http/{$param}/$lang";
	}
?>
<!DOCTYPE HTML>
<html lang="pl">
[PHP] pobierz, plaintext 

i wtedy jak wpsze: localhost/strona/
to wyswietla:

[PHP] pobierz, plaintext 
Notice: Undefined index: lang in C:\xampp\htdocs\strona\trans.php on line 22
[PHP] pobierz, plaintext 

a jak dodam do index.php

[PHP] pobierz, plaintext 
if(!isset($_GET['lang']))
	{
		$_GET['lang'] = "pl";
	}
[PHP] pobierz, plaintext 

To działa i błędu nie ma

A co ty mi index.php pokazujesz jak blad wyraznie mowi ze to trans.php.... czytaj uwaznie komunikaty bledow,

Do trans tez masz dodac tem kawalek kodu
$lang = isset($_GET['lang']) ? $_GET['lang'] : 'pl';
if (!in_array($lang, array('pl', 'en'))) {
$lang = 'pl';
}

A nalepiej jezyk masz ustalac w jednym miejscu i przekazywac gdzie trzeba

jak dodam ten kawałek do trans.php to dalej ten sam błąd

linijka błędu w trans.php

[PHP] pobierz, plaintext 
$lang = mysqli_real_escape_string($polaczenie, $_GET['lang']);
[PHP] pobierz, plaintext 

skoro LANG po moim kodzie znajduje sie w $lang to masz go juz nie szukac w $_GET['lang']...

rozumiesz w ogole co robi ten kod
$lang = isset($_GET['lang']) ? $_GET['lang'] : 'pl';
if (!in_array($lang, array('pl', 'en'))) {
$lang = 'pl';
}

?
Bo jesli nie to usiadz nad nim na 5 minut i go przeanalizuj bo latasz teraz z totalnymi podstawami.

Tak, wiem co on oznacza tylko że bez tej linijki dalej jest błąd

[PHP] pobierz, plaintext 
Notice: Undefined [b]variable[/b]: lang in C:\xampp\htdocs\strona\trans.php on line 29
[PHP] pobierz, plaintext 

tym razem w tej linijce

[PHP] pobierz, plaintext 
if(!$zapytanie = $polaczenie->query("SELECT klucz, text FROM translate WHERE lang = '$lang'"))
[PHP] pobierz, plaintext 

To mi juz na dzisiaj wystarczy. Przeczytaj jeszcze raz moich pare ostatnich postow i zastanow sie nad nimi dluzej bo ewidentnie gubisz bardzo istotny a zarazem prosty fakt.