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