[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]

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