Переключить улучшение

Question

У меня есть следующий случай переключения, но на данный момент он содержит 10 условий.

public function convertParameters(string $key, $value)
{
    switch ($key) {
        case "office":
        case "name":
        case "first_name":
        case "last_name":
            $value = ucfirst($value);
            break;
        case "email":
            $value = strtolower($value);
            break;
        case "street":
        case "city":
            $value = ucwords($value);
            break;
        case "start":
        case "end":
            $value = \DateTime::createFromFormat("Y-m-d H:i:s", $value);
            break;
    }
    return $value;
}

Кто-нибудь знает, как я могу улучшить эту функцию, чтобы у нее было меньше условий? (или другие улучшения)

Answer 1

Ваш метод выглядит хорошо, но если вы ищете альтернативы, вы можете использовать массив, который отображается из ключей в функции.

$ops = [
    "office" => "ucfirst",
    "name" => "ucfirst",
    "first_name" => "ucfirst",
    "last_name" => "ucfirst",
    "email" => "strtolower",
    "street" => "ucwords",
    "city" => "ucwords",
    "start" => function($d) { return DateTime::createFromFormat("Y-m-d H:i:s", $d); },
    "end" => function($d) { return DateTime::createFromFormat("Y-m-d H:i:s", $d); }
];

if (isset($ops[$key])) {
    return $ops[$key]($value);
} else {
    return $value;
}