[PHP]wybieranie języka wyświetlenia strony. Pomoc w zaznaczeniu który język wybrany. Opcje

[casperii]

Witam Panowie, mam kod:

[PHP] pobierz, plaintext 
<?php
						//============== *** JEZYKI *** ==============
						$jezyki = '';
						$jezyki_array = array('pl','ua');
						if(isset($lang)){
							$lang = $_GET['lang'];
						}else{
							$lang = 'pl';
						}
						$language_get = $lang;
						if(in_array($language_get, $jezyki_array)){
						$language_page = $language_get; }
						else{
						$language_page = 'pl';
						}
						/*if(isset($lang)=='pl'){
									$classy = '_act';
						}else{
									$classy = '';
						}
						*/
						switch ($language_page){
							case 'pl':
										include_once('language/polish.php');
										$kodowanie = "text/html; charset=UTF-8";
										break;
 
							case 'ua':
										include_once('language/ukrainian.php');
										$kodowanie = "text/html; charset=UTF-8";
										break;
 
							default:
										include_once('language/polish.php');
										$kodowanie = "text/html; charset=UTF-8";
						}
 
						$adres_url = "http://".$_SERVER["SERVER_NAME"].$_SERVER["SCRIPT_NAME"];
						$adres_zmn = explode("&", $_SERVER["QUERY_STRING"]);
 
						for($jl=0;$jl<count($jezyki_array);$jl++){
 
							$adres_zmienne = NULL;
							$lang_tak = FALSE;
 
								for($jz=0;$jz<count($adres_zmn);$jz++){
 
									$and_nie = TRUE;
 
										if(preg_match('/lang=+/i', $adres_zmn[$jz])){
 
											if($lang_tak){
												$and_nie = FALSE;
											}else{
												$adres_zmienne .= 'lang='.$jezyki_array[$jl];
												$lang_tak = TRUE;
												}
										}else{
												$adres_zmienne .= $adres_zmn[$jz];
										}
 
										if($jz==(count($adres_zmn)-1) AND !$lang_tak){
 
											if($adres_zmn[$jz]!=NULL){
												$adres_zmienne .= '&lang='.$jezyki_array[$jl];
											}else{
												$adres_zmienne .= 'lang='.$jezyki_array[$jl];
											}
 
												$and_nie = FALSE;
										}
 
										if($jz!=(count($adres_zmn)-1) AND $and_nie){
											$adres_zmienne .= '&'; 
										}
 
									}
 
								$adres_href = $adres_url.'?'.$adres_zmienne;
								$jezyki .= '<div style="padding:0px; margin:0px;"><a href="'.$adres_href.'" class="lang'.$classy.'"><img src="/images/flag/'.$jezyki_array[$jl].'.png" border="0"></a></div>';
						}
						//============================================
						?>
						<?= $jezyki;?>
[PHP] pobierz, plaintext 

Proszę o sprawdzenie czy kod jest dobrze napisany i jeżeli zawiera niepotrzebne lub przestarzałe funkcje to proszę o naprowadzenie mnie.
Ponadto chciałbym dodać do classy zmienną $classy jeżeli wybrałem dany język.
Reasumując wybrałem język polski dodaje mi do class="lang_act" pozostałe języki class="lang"

[KsaR]

[PHP] pobierz, plaintext 
<?php
						//============== *** JEZYKI *** ==============
						$jezyki = '';
						$jezyki_array = array('pl','ua');
						if(isset($lang)){
[PHP] pobierz, plaintext 

Mi się tylko zdaje czy korzystasz z REGISTER_GLOBALS (IMG:style_emoticons/default/questionmark.gif) (IMG:style_emoticons/default/ohmy.gif)
explode, in_array, count, preg_match. Wedlug mnie nie sa przestarzale (IMG:style_emoticons/default/tongue.gif) ,
Yyy. Poczytaj o DRY. Bo powtarzasz kod gdzies gdzie doslownie bylby zbedny np.
$kodowanie...
poza tym:

[PHP] pobierz, plaintext 
	for($jl=0;$jl<count($jezyki_array);$jl++){
 
							$adres_zmienne = NULL;
							$lang_tak = FALSE;
 
								for($jz=0;$jz<count($adres_zmn);$jz++){
[PHP] pobierz, plaintext 

Count powinienes miec wpierwszym argumencie zapisany jesli ilosc eleementow tablicy sie nie zmeinia co 1 petle. - a u ciebie sie nie zmiena.

Tzn w pierwszym argumencie for mozesz zapisywac nowe zmienne po porzecinku, wiec mozesz tak:

for ($x=0, $ct=count($cos); $x<$ct ; ++$x)

Dla przykladu.

(IMG:style_emoticons/default/tongue.gif)

Poza tym tworzysz duzo zmienmych.
$language_get = $lang;
(..ciach..)
$language_page = $language_get;

[PHP] pobierz, plaintext 
			if(isset($lang)){
							$lang = $_GET['lang'];
						}else{
							$lang = 'pl';
						}
						$language_get = $lang;
						if(in_array($language_get, $jezyki_array)){
						$language_page = $language_get; }
						else{
						$language_page = 'pl';
						}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$language_page = isset($_GET['lang']) ? $_GET['lang'] : 'pl';
if (!in_array($language_page, $jezyki_array))
 $language_page='pl';
[PHP] pobierz, plaintext 

Tak chyba lepiej?..

@@ widze ze nie moge zedytowac postu bo go na html zamieni.
Co do register_globals; no chyba ze includujesz ten plik.. ; (IMG:style_emoticons/default/tongue.gif)

[KsaR]

1. $lang = 'pl'; skasuj ta zmienna bo jej nawet nie uzywasz.
2. $kodowanie = "text/html; charset=UTF-8"; skasuj z switcha bo robisz w kazdym case to samo(więc mozesz raz za switchem)
3. Uzyj funkcji ktora ci pokazalem w innym temacie parse_str()
Zamiast tej kombinacji str_replace.
4. Razi tak czy inaczej:
for($jl=0;$jl<count($jezyki_array);
Lepiej:
for($jl=0,$jct=count($jezyki_array);$jl<$jct;
Bo inaczej liczy od nowa za kazdym obiegiem pętli.
Ale z tego co widze i tak mozesz ta petle pominac jak sie nie myle, gdy uzyjesz funkcji z pkt. 3.
Edit. nie chce mi sie kombinowac bo dzis nie mysle.
Mam gorączkę to moglem cos pomylic (IMG:style_emoticons/default/wink.gif)

Edit. Po godzinie (IMG:style_emoticons/default/biggrin.gif) , zrob tak jak w manualu.

[PHP] pobierz, plaintext 
$params = parse_str($_SERVER['QUERY_STRING']);
[PHP] pobierz, plaintext 

I z tego wydobadz jaki język (IMG:style_emoticons/default/wink.gif) , (oczywiscie zrob print_r($params) zeby zobaczyc jak wyglada tablica zeby wiedziec jak sie odwolac)

I potem mozesz np.

$wybrany_jezyk = isset($params['lang']) ? $params['lang'] : 'pl';
I switch tego wybranego języka. Oczywiscie klucz lang to ma byc klucz pod ktorym bedzie język.
A for mozesz pominąć tzn to kasowanie przez str_replace itp. Zbuduj z tego co wyzej odpowiedni url.

Ten post edytował KsaR 17.10.2015, 20:58:33

[casperii]

@KsaR zrobiłem trochę po swojemu :-) ale działa. Napisz mi w jaki sposób mam zaznaczyć który język jest wybrany. Np dopisanie czegoś do class

[PHP] pobierz, plaintext 
 
$jezyki = '';
$jezyki_array = array('pl','ua');
 
$language_page = isset($_GET['lang']) ? $_GET['lang'] : 'pl';
if(!in_array($language_page, $jezyki_array))
$language_page='pl';
 
switch ($language_page){
	case 'pl':
	include_once('language/polish.php');
	break;
 
	case 'ua':
	include_once('language/ukrainian.php');
	break;
 
	default:
	include_once('language/polish.php');
}
$kodowanie = "text/html; charset=UTF-8";
 
$count_lang = count($jezyki_array);
for ($jl=0; $jl <$count_lang; $jl++){						
	$jezyk = NULL;
	$jezyk .= ''.$jezyki_array[$jl];
 
	$cos=$_SERVER['REQUEST_URI'];
	$cos2 = str_replace($language_page, '', $cos);
	$cos2 = str_replace('/', '', $cos2);
	$adres_href = NULL;
	$adres_href .= '/'.$jezyk.'/'.$cos2;
	$jezyki .= '<div style="padding:0px; margin:0px;"><a href="'.$adres_href.'" class="lang"><img src="/images/flag/'.$jezyk.'.png" border="0"></a></div>';
}
[PHP] pobierz, plaintext 

Musiałem użyć str_replace ponieważ parse_str nic nie wyświetlał.
Wstawienie w switch zmiennej class i później jej dopisanie w hrefie nic nie daje.

Ten post edytował casperii 18.10.2015, 18:06:28

[KsaR]

Bo ja wycielem fragment ktory trzeba zamienic a nie cala (IMG:style_emoticons/default/tongue.gif) ,
A ogolem to miales bez petli z tym co pisalem poprobowac hm... I wedlug tego link zbudowac.

Ten post edytował KsaR 17.10.2015, 22:08:40

[casperii]

Ma ktoś jakiś pomysł jak mam zaznaczyć który język wybrano?

[Pyton_000]

A czego Ty tu nie umiesz? Przecież masz w zmiennej wybrany język, teraz trzeba tylko coś z tym zrobić.

[casperii]

@Pyton_000 to ja wiem, że za wybrany język odpowiada mi zmienna $language_page.
ale chciałbym to jakoś wkomponować w pętle for gdzie mam wyświetlone flagi i ten język który został wybrany to flaga będzie miała inną classe.

[PHP] pobierz, plaintext 
if($language_page='pl'){
$classaNowa ='opacity:0.5';
}
[PHP] pobierz, plaintext 

tylko, przecież tu nie może być pl bo to przecież może tyczyć się każdego języka. a nie nas*am ifów (IMG:style_emoticons/default/tongue.gif)

Drugim problemem jest to, że chce do flagi do adresu dodać index.html jeśli ktoś wszedł główną domenę. Bo to teraz wygląda tak domena.pl/pl <- a tu muszę dopisać index.html , ale nie może być na stałe, bo jak ktoś wybierze start to adres jest domena.pl/index.html :-)

[casperii]

@Pyton_000 to ja wiem, że za wybrany język odpowiada mi zmienna $language_page.
ale chciałbym to jakoś wkomponować w pętle for gdzie mam wyświetlone flagi i ten język który został wybrany to flaga będzie miała inną classe.

[PHP] pobierz, plaintext 
if($language_page='pl'){
$classaNowa ='opacity:0.5';
}
[PHP] pobierz, plaintext 

tylko, przecież tu nie może być pl bo to przecież może tyczyć się każdego języka. a nie nas*am ifów (IMG:style_emoticons/default/tongue.gif)

Drugim problemem jest to, że chce do flagi do adresu dodać index.html jeśli ktoś wszedł główną domenę. Bo to teraz wygląda tak domena.pl/pl <- a tu muszę dopisać index.html , ale nie może być na stałe, bo jak ktoś wybierze start to adres jest domena.pl/index.html :-)

[Pyton_000]

if($jezyk == $language_page)