Bessere if-Abfrage

[inkamina]

Liebe Kollegen,

ich habe folgende if-Abfrage:

foreach(Item::getAllItemnumbers() as $i) :
    $a = $item->get($i->item_number);
    if (!empty($a)) { $items[] = $a; }
endforeach; 

Es werden sich demnach alle verfügbaren Artikelnummern geholt und in der folgenden Schleife werden die Artikeldaten zusammengesetzt um daraus letztlich ein großes Array mit allen Artikeln zu machen.
Was mich stört ist der Syntax der if-Abrage, die null Werte herausfiltern soll. Schöner wäre aus meiner Sicht in etwa soetwas hier:

foreach(Item::getAllItemnumbers() as $i) :
    if(!empty($a = $item->get($i->item_number)) : $items = $a; endif;
endforeach; 

Das lässt sich aber nicht ohne Syntax Fehler in PHP darstellen. Wie geht ihr da vor, wie könnte man das noch lösen?

Besten Dank für ein paar knackige Hinweise.

[BlackScorp]

foreach(Item::getAllItemnumbers() as $i) :
    $a = $item->get($i->item_number);
    if(in_array(trim($a),[NULL,false,''])){ continue; } //wenn a null/false oder empty ist, dann überspringe die schleife
   $items[] = $a;
endforeach; 

[protestix]

$items = [];

foreach( Item::getAllItemnumbers() as $i ) {

    if( !empty($item->get($i->item_number)) ) {

        $items[] = $item->get($i->item_number)

    }

} 

Es sollte leserlich und verständlich sein und nicht kurz und kryptisch.

[Dormilich]

Andere Variante mit array-Funktionen:

$items = array_map(function ($i) use ($item) {
    return $item->get($i->item_number);
}, Item::getAllItemnumbers());

$items = array_filter($items); 

Aber ich sehe da eher ein Problem bei der Item-Klasse, da diese ja für die Items verantwortlich sein sollte.

[inkamina]

Danke für die unerwartet schnellen Rückmeldungen

$items = [];

foreach( Item::getAllItemnumbers() as $i ) {

if( !empty($item->get($i->item_number)) ) {

$items[] = $item->get($i->item_number)

}

} 

Es sollte leserlich und verständlich sein und nicht kurz und kryptisch.

Das Problem bei der Lösung ist, dass alle Funktionen und Queries die hinter $item->get($i->item_number) liegen, dann erneut, d.h. insgesamt zwei mal, aufgerufen werden. Das kann bei einer höheren Anzahl items (> 1 Mill.) deutliche Unterschiede ausmachen.

Andere Variante mit array-Funktionen:

$items = array_map(function ($i) use ($item) {
return $item->get($i->item_number);
}, Item::getAllItemnumbers());

$items = array_filter($items); 

Aber ich sehe da eher ein Problem bei der Item-Klasse, da diese ja für die Items verantwortlich sein sollte.

Das ist in diesem Fall absolut korrekt. Und so werde ich es auch lösen. Btw. dein Lösungsansatz ist sehr interessant.

[BlackScorp]

Das Problem bei der Lösung ist, dass alle Funktionen und Queries die hinter $item->get($i->item_number) liegen, dann erneut, d.h. insgesamt zwei mal, aufgerufen werden. Das kann bei einer höheren Anzahl items (> 1 Mill.) deutliche Unterschiede ausmachen.

$items = [];

foreach( Item::getAllItemnumbers() as $i ) {
$itemValue = $item->get($i->item_number);
    if( !empty($itemValue ) ) {
        $items[] = $itemValue;
    }

} 

damit hast du das Problem gelöst

[Dormilich]

Btw. dein Lösungsansatz ist sehr interessant.

Orientiert sich ein wenig an Funktionaler Programmierung.

[jspit]

Die Lösung #6 die ja nichts anderes als eine "sprechende" Variante der Ausgangslösung #1 ist halte ich in Punkto Lesbarkeit + Geschwindigkeit für besser geeignet.

Edit: Da wohl sichergestellt ist dass das Resultat von $item->get($i->item_number existiert kann die Variante #6 noch etwas verkürzt werden:

$items = [];

foreach( Item::getAllItemnumbers() as $i ) {
   if( $itemValue = $item->get($i->item_number) )  $items[] = $itemValue;
} 

[lottikarotti]

Ich hätte gerne sowas in PHPs stdlib:

$items = new Stream($itemNumbers);
  ->map([$repo, 'getByNumber'])
  ->filter(function($item){
    return $item instanceof Item;
  })
  ->getElements(); 

Funktionen wie array_map, array_filter, array_reduce usw. sind unnötig unhandlich und klobig. imho

[Dormilich]

Funktionen wie array_map, array_filter, array_reduce usw. sind unnötig unhandlich und klobig. imho

Genau. Deswegen hab ich mir genau so ein Helferlein mal erstellt. (https://github.com/Dormilich/array-list)

[Arne Drews]

Finde ich nicht, ich nutze die häufig lieber, wie Schleifen...

[lottikarotti]

Finde ich nicht, ich nutze die häufig lieber, wie Schleifen...

Ich nutze sie auch häufig, finde sie aber trotzdem hässlich. Da gefällt mir selbst JavaScript besser:

items = itemNumbers
  .map(repo.getItem)
  .filter(item => item instanceof Item) 

[Arne Drews]

Jo, JS macht das schon schick mit seinem Objekt-Model...
Ist am Ende ja auch Geschmacksache, ich finde bspw. PHP-Code mit häufigen IF oder FOR nicht sonderlich schön und versuche vieles auf andere Weise zu lösen, sofern es geht.
Performance o.ä. hat damit absolut nichts zu tun, geht dabei nur um meinen persönlichen Geschmack...