[PHP]Skracanie wyrazenia if Opcje

[JamalBIG]

Witam

Czy mozna 'jakos' skrocic takie wyrazenie?:

[PHP] pobierz, plaintext 
<?php
if ($ocena == 1) {
  $checked1 = 'checked';
} elseif ($ocena == 2) {
  $checked2 = 'checked';
} elseif ($ocena == 3) {
  $checked3 = 'checked';
} elseif ($ocena == 4) {
  $checked4 = 'checked';
} elseif ($ocena == 5) {
  $checked5 = 'checked';
}
?>
[PHP] pobierz, plaintext 

[Lejto]

użyć porostu OR?

[v1t4n]

Przypuszczam ze robisz to do selecta. Jesli tak to spróbój na tablicach (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[.radex]

switch!

[PHP] pobierz, plaintext 
<?php
switch($ocena)
{
case 1: $checked1 = 'checked'; break;
 
case 2: $checked2 = 'checked'; break;
 
//itd.
}
?>
[PHP] pobierz, plaintext 

Edit:

Panowie! Jeśli to ma być tylko te 5 wartości to po kiego grzyba syfić kod małoczytelnym kodem?

Ten post edytował .radex 7.05.2009, 19:53:14

[kirex]

[PHP] pobierz, plaintext 
<?php
${'checked'.$ocena} = 'checked';
?>
[PHP] pobierz, plaintext 

[Lion_87]

A może tak?

[PHP] pobierz, plaintext 
<?
$ocena="4";
for($a=1;$a<=5;$a++){
 if($ocena==$a){${checked.$a}="checked";}
}
 
?>
<input type="checkbox" <? echo $checked1; ?>>
<input type="checkbox" <? echo $checked2; ?>>
<input type="checkbox" <? echo $checked3; ?>>
<input type="checkbox" <? echo $checked4; ?>>
<input type="checkbox" <? echo $checked5; ?>>
[PHP] pobierz, plaintext 

[decha-design]

Trochę warningów Ci parser PHP wysypie przy tym twoim kodzie, @Lion_87

[JamalBIG]

Sporo tych propozycji, dzeki za zainteresowanie ale teraz sam nie wiem ktory wybrac bo np ze switch raczej odpada bo 'niezmniejsza' mi kodu ale petla z uzyciem funkcji for wyglada interesujaco

[kirex]

@JamalBIG a po jakiego grzyba Ci ten for ? masz jedną zmienną $ocena, która zawiera wartości od 1 do 6 czyli w zupełności wystarczy Ci samo

[PHP] pobierz, plaintext 
<?php
${'checked'.$ocena} = 'checked';
?>
[PHP] pobierz, plaintext 

powiedzmy, że ocena bedzie miała wartość 4 to z w/w wyrażenia dostaniesz $checked4 = 'checked'


Ps. no chyba, że od razu chcesz utworzyć pozostałe chcecked.. z pustymi wartościami (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował kirex 7.05.2009, 20:18:14

[Lion_87]

Trochę warningów Ci parser PHP wysypie przy tym twoim kodzie, @Lion_87

no całe 0 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

racja racja MOJA wina (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) no może 1

Ten post edytował Lion_87 7.05.2009, 21:04:53

[kirex]

@Lion_87 wywali 6 NOTICE (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

i błąd przy ${checked.$a} (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował kirex 7.05.2009, 20:20:28

[decha-design]

to jeden piernik (IMG:http://forum.php.pl/style_emoticons/default/haha.gif) ważne, że wywali =)

[Crozin]

Jak rozumieim masz zadeklarowaną stałą: checked (o wartości checked) tylko nam jej nie pokazujesz?

Albo zapomniałeś włączyć raportowania błędów typu E_NOTICE?

[JamalBIG]

Dzieki @kirex

[PHP] pobierz, plaintext 
<?php
${'checked'.$ocena} = 'checked';
?>
[PHP] pobierz, plaintext 

Dziala pieknie i byl bym bardzo wdzieczny za 'wytlumaczenie' mi tego kodu

[Crozin]

PHP to taki język, który umożliwia dynamiczne tworzenie nazw zmienych. Innymi słowy utworzy zmienną o nazwie checked + wartość ze zmiennej $ocena o wartości checked

[Lion_87]

@Lion_87 wywali 6 NOTICE tongue.gif

i błąd przy ${checked.$a} winksmiley.jpg

hmmmm jednak nadal nierozumiem...mozesz bardziej wytłumaczyć kirex?

Ten post edytował Lion_87 7.05.2009, 20:54:54

[kirex]

@Lion_87 checked w nawiasie klamrowym musi być w "ciapkach", czyli łączysz string ze zmienną 'chceked' . $ocena i to podstawiasz pod ${ ... }

Jeśli zaś w Twoim przypadku checked było stałą, to powinieneś napisać ją dużymi literami (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował kirex 7.05.2009, 21:03:17

[Lion_87]

@Lion_87 checked w nawiasie klamrowym musi być w "ciapkach", czyli łączysz string ze zmienną 'chceked' . $ocena i to podstawiasz pod ${ ... }

Jeśli zaś w Twoim przypadku checked było stałą, to powinieneś napisać ją dużymi literami winksmiley.jpg

No to kapuje ale czemu 6 NOTICE?

EDIT
domyślam sie może ze przez to?

Kod

<? echo $checked1; ?>

itd.

Ten post edytował Lion_87 7.05.2009, 21:14:35

[kirex]

A może tak?

[PHP] pobierz, plaintext 
<?
$ocena="4";
for($a=1;$a<=5;$a++){
if($ocena==$a){${checked.$a}="checked";}
}
 
?>
<input type="checkbox" <? echo $checked1; ?>>
<input type="checkbox" <? echo $checked2; ?>>
<input type="checkbox" <? echo $checked3; ?>>
<input type="checkbox" <? echo $checked4; ?>>
<input type="checkbox" <? echo $checked5; ?>>
[PHP] pobierz, plaintext 

Pomyliło mi się, będą 4 NOTICE, czyli $checked(1,2,3,5) ponieważ ich nie zadeklarowałeś