[php] zobtymalizoawać skrypt Opcje

[3miel]

mam taki ecoś. ale troche tego za dużo i chiałbym się dowiedzieć czy można i w jaki sposób go zobtymalizowac.

[PHP] pobierz, plaintext 
<?
if($pole['model']=='Nokia')	 { echo '<option value=Nokia selected>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Siemens')	 { echo '<option value=Nokia>Nokia<option value=Siemens selected>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Sony Ericsson')	 { echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\" selected>Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Samsung')	 { echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung selected>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Alcatel')	 { echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel selected>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Ericsson')	 { echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson selected>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Motorola')	 { echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola selected>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Sagem')	 { echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem selected>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Sendo')	 { echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo selected>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic>Panasonic';	} 
elseif($pole['model']=='Mitsubishi')	 { echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi selected>Mitsubishi<option value=Panasonic>Panasonic';	} 
else	{echo '<option value=Nokia>Nokia<option value=Siemens>Siemens<option value=\"Sony Ericsson\">Sony Ericsson<option value=Samsung>Samsung<option value=Alcatel>Alcatel<option value=Ericsson>Ericsson<option value=Motorola>Motorola<option value=Sagem>Sagem<option value=Sendo>Sendo<option value=Mitsubishi>Mitsubishi<option value=Panasonic selected>Panasonic';	} 
?>
[PHP] pobierz, plaintext 

[cim]

[PHP] pobierz, plaintext 
<?php
$telefony = new Array('Nokia', 'Siemens', 'Sony Ericsson', 'Samsung', 'Alcatel', 'Ericsson', 'Motorola', 'Sagem', 'Sendo', 'Mitsubishi', 'Panasonic');
 
foreach($telefony as $telefon){
if($telefon == $pole['model']){
 print(&#092;"<option value=\"$telefon\" selected>$telefon</option>\");
}else{
 print(&#092;"<option value=\"$telefon\">$telefon</option>\");
}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował cim 25.09.2004, 15:45:49

[rogrog]

zapisz sobie telefony w tablicy

[PHP] pobierz, plaintext 
<?php
 
 
/**
Zbiór telefonów.
*/
 
$telefony = new Array('Nokia', 'Siemens', 'Sony Ericsson', 'Samsung', 'Alcatel', 'Ericsson', 'Motorola', 'Sagem', 'Sendo', 'Mitsubishi', 'Panasonic');
 
 
/**
Wyświetlanie
*/
 
foreach ( $telefony as $telefon ){ //pętla po telefonach
 
	echo	'<option
			 value=\"'. $telefon.'\"'
			 . ( $telefon == $pole['model'] ) ? ' selected=\"selected\"' : '' . //sprawdzenie warunku
			 '>'.$telefon.'</option>';
 
}
 
 
?>
[PHP] pobierz, plaintext 

[3miel]

niestety bład W OBU PRZYPADKACH (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) :

Parse error: parse error, unexpected T_ARRAY, expecting T_STRING or T_VARIABLE or '$' in /home/gsm/public_html/admin/viewa.php(91) : eval()'d code on line 125


PS. to jest w WHILE.

Ten post edytował 3miel 25.09.2004, 15:54:03

[cim]

a możesz wkleić cały kod? masz średnik na końcu linii poprzedzającej deklaracje zmiennej z nazwami telefonów?

[3miel]

[PHP] pobierz, plaintext 
<?
$zapytanie=&#092;"SELECT * FROM sms_ng WHERE ng='$ng'\";
$wykonaj=mysql_query($zapytanie);
while($pole = mysql_fetch_array($wykonaj))
{
echo '<tr><td>Ng: </td><td><input type=hidden name=ng value='.$pole['ng'].'><input type=text name=ng2 maxlength=3 size=3 value='.$pole['ng'].'></td></tr>';
echo '<tr><td>Model:</td><td><select name=model>';
 
 
/**
Zbiór telefonów.
*/
 
$telefony = new Array('Nokia', 'Siemens', 'Sony Ericsson', 'Samsung', 'Alcatel', 'Ericsson', 'Motorola', 'Sagem', 'Sendo', 'Mitsubishi', 'Panasonic');
 
 
/**
Wyświetlanie
*/
 
foreach ( $telefony as $telefon ){ //pętla po telefonach
 
	echo	'<option
			 value=\"'. $telefon.'\"'
			 . ( $telefon == $pole['model'] ) ? ' selected=\"selected\"' : '' . //sprawdzenie warunku
			 '>'.$telefon.'</option>';
 
}
 
}
?>
[PHP] pobierz, plaintext 

i bład:

Parse error: parse error, unexpected T_ARRAY, expecting T_STRING or T_VARIABLE or '$' in /home/gsm/public_html/admin/viewa.php(91) : eval()'d code on line 117

Ten post edytował 3miel 25.09.2004, 16:15:43

[sf]

Następnym razem napisz w php Początkujący bo usunięcie jednego słowa 'new' by to zadziałało to naprawdę jest ... eh, bez komentarza.

Masz odrazu działający kod :

[PHP] pobierz, plaintext 
<?php
 
$pole['model']='Siemens';
 
$telefony = array('Nokia', 'Siemens', 'Sony Ericsson', 'Samsung', 'Alcatel', 'Ericsson', 'Motorola', 'Sagem', 'Sendo', 'Mitsubishi', 'Panasonic');
?>
<form action=\"<?php echo $_SERVER['PHP_SELF']; ?>\" method=\"post\">
<select name=\"telefon\">
<?
foreach($telefony as $telefon){
	if($telefon == $pole['model']){
		echo '<option value=\"'.$telefon.'\" selected=\"selected\">'.$telefon.'</option>';
	}else{
		echo '<option value=\"'.$telefon.'\">'.$telefon.'</option>';
	}
}
?>
</select>
</form>
[PHP] pobierz, plaintext 

Ten post edytował sf 25.09.2004, 16:17:19

[3miel]

ok działa ale tylko ten 1 kod:) dzieki