1

Подскажите пожалуйста как можно рефакторить этот код чтоб обойтись без switch

public function getCodDoc($kod_doc)
{
    switch ($kod_doc) {
        case "retail":
            $kod_doc = $this->retail;
            break;
        case "bn":
            $kod_doc = $this->bn;
            break;
        case "bnfab":
            $kod_doc = $this->bnfab;
            break;
        case "ya":
            $kod_doc = $this->ya;
            break;
        case "ret":
            $kod_doc = $this->ret;
            break;
    }

    return $kod_doc;
}
7
  • 4
    return $this->{$kod_doc} Commented 30 мар. 2021 в 8:31
  • 1
    Используйте match =) Если пока не можете, оставьте как есть, незачем рефакторить вполне понятный код. Ну или заведите массив с отображением.
    – vp_arth
    Commented 30 мар. 2021 в 8:37
  • @DanielProtopopov, это не эквивалентный код, нужен белый список или хотя бы property_exists.
    – vp_arth
    Commented 30 мар. 2021 в 8:46
  • @vp_arth все зависит от ситуации.
    – teran
    Commented 30 мар. 2021 в 8:52
  • @DanielProtopopov я бы действительно добавил кейс по умолчанию, return $this->{$kod_doc} ?? $kod_doc; Commented 30 мар. 2021 в 8:54

0

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Посмотрите другие вопросы с метками или задайте свой вопрос.