Ist das ein schlechtes Muster? (Schalter innen für / foreach Schleife)

Ich schreibe Code wie:

foreach($array as $key => $value) {
    switch($key) {
        case 'something':
            doSomething($value);
            break;
        case 'somethingelse':
            doSomethingElse($value);
            break;
    }
}

Gibt es einen besseren Weg, dies zu tun? Scheint mir dreckig, aber ich könnte einfach darüber nachdenken.

Die einzige Alternative, die ich mir vorstellen kann, ist eine if-Anweisung für jeden Schlüssel, die nicht besser zu sein scheint. Das heißt :

if($array[0] == 'something') {
    doSomething($array[0]);
}
if($array[1] == 'somethingelse') {
    doSomethingElse($array[1]);
}

(oder sowas ähnliches)

Ich kann bei Bedarf genauen Code posten, aber dies ist der allgemeine Überblick darüber, was passiert. Bitte Kritik weg, aber denk daran, dass ich hier Hilfe suche. Wenn ich also etwas ungeheuerliches falsch mache, dann weise darauf hin.

Antworten auf die Frage(6)

Ihre Antwort auf die Frage