[PHP]Skrócenie if i else if Opcje

[JamalBIG]

Witam

Jest jakiś sposób na skrócenie poniższej 'formułki'?

[PHP] pobierz, plaintext 
if ($row['wersja']) {
	$id_kategorii = $row['wersja'];
} else if ($row['marka']) {
	$id_kategorii = $row['marka'];
} else if ($row['model']) {
	$id_kategorii = $row['model'];
} else if ($row['rodzaj']) {
	$id_kategorii = $row['rodzaj'];
}
[PHP] pobierz, plaintext 

Z góry dzięki za odpowiedz
Pozdrawiam

[gino]

manual

gino

[Crozin]

Mógłbyś tutaj z jakąś pętlą wyskoczyć typu:

[PHP] pobierz, plaintext 
foreach (array('wersja', 'marka', 'model', 'rodzaj') as $abc) {
  if ($row[$abc]) {
    $id_kategorii = $row[$abc];
    break;
  }
}
[PHP] pobierz, plaintext 

Ale szczerze powiedziawszy zostawiłbym to tak jak jest, bo jest czytelniej i prościej.

@mat-bi: Pomijając fakt, że jest to błędny kod... weź nawet takich potworków nie proponuj.

Ten post edytował Crozin 19.03.2011, 20:57:28

[mat-bi]

Gino, radzę najpierw spojrzeć w kod

Można uzżyć skróconej wersji if

[PHP] pobierz, plaintext 
$id_kategorii = ($row['wersja']) ? $row['wersja']; : ($row['marka']) ? $row['marka']; :  ($row['model']) ?  $row['model']; : ($row['rodzaj']) ? $row['rodzaj'];
[PHP] pobierz, plaintext 

[JamalBIG]

dzięki wszystkim za odpowiedź ale jeżeli nie ma jakiegoś fajnego rozwiązania to rzeczywiście zostanę przy tym co już zrobiłem

[gino]

Gino, radzę najpierw spojrzeć w kod

spojrzałem, czytelniej jest tak jak jest a bardzej elegancko z switch ... case

gino

PS. pomijając fakt błędnego kodu

Ten post edytował gino 19.03.2011, 21:12:55

[mat-bi]

Gino, ale za każdym razem sprawdzany jest inny element tablicy, w jaki więc sposób chcesz to przerobić przez switch?

[gino]

ooo sory, już nie widzę od kompa, najwyższa pora skończyć na dzisija pracę moje posty prosze pominąć ...


gino

[everth]

@mat-bi: da się

[PHP] pobierz, plaintext 
switch(true) {
   case isset($row['wersja']):
      //jakis kod
      break;
   case isset($row['marka']):
      //jakis kod
      break;
}
[PHP] pobierz, plaintext 

Pytanie co będzie bardziej zrozumiałe, kod w obecnej postaci czy przepisany z użyciem switcha. Ja wolę obecny.

[mat-bi]

W sumie ja też. (nawet nie wiedziałem o takiej konstrukcji switch )

A co do @Crozin, rzeczywiście, przesadziłem z potworkiem.

[desperat666]

Nie jestem do końca pewien czy złapałem

[PHP] pobierz, plaintext 
<?php
function SetCategoryID($rowarray) {
$CategoryID = "";
  if is_array($rowarray) {
   // Zgodnie z zamierzoną kolejnością, sprawdza czy jest taki klucz i przypisuje zmiennej CategoryID wartość klucza
   // uprzednio sprawdzając czy wartość klucza nie jest NULL, bo jeśli tak to CategoryID pozostanie domyślnie puste.
   // Rozumiem że tylko jeden z kluczy w $rowarray będzie w danym momencie miał przypisaną wartość, którą autor chciał przypisać zmiennej 
   // id_kategorii, bo jeśli nie, to id_kategorii zostanie przypisana wartość z ostatniego sprawdzanego klucza  
   if array_key_exists('wersja',$rowarray) { if ($rowarray['wersja']!="") { $CategoryID=$rowarray['wersja']; } }
   if array_key_exists('marka',$rowarray) { if ($rowarray['marka']!="") { $CategoryID=$rowarray['marka']; } }
   if array_key_exists('model',$rowarray) { if ($rowarray['model']!="") { $CategoryID=$rowarray['model']; } }
   if array_key_exists('rodzaj',$rowarray) { if ($rowarray['rodzaj']!="") { $CategoryID=$rowarray['rodzaj']; } }
  }
// CategoryID pozostanie puste jeżeli żadna z wartości kluczy nie będzie ustawiona
return $CategoryID;
}
 
if (SetCategoryID($row)!="") {$id_kategorii = SetCategoryID($row); } else { printf("Nie można ustawić id_kategorii\n"); print_r($row); }
// no i nie powinno się krzaczyć..
?>
 
[PHP] pobierz, plaintext