0

Есть php код:

public static function load() : void
{
    $config = new ...;
    $config = $config->getData();

    $throw = function() : void
    {
        throw new \ConfigException ("Invalid config.");
    };

    self::$param1 = $config["core"]["param1"] ?? ($throw)();
    self::$param2 = $config["core"]["param2"] ?? ($throw)();
    self::$param3 = $config["module"]["param3"] ?? ($throw)();
    self::$param4 = $config["module"]["param4"] ?? ($throw)();
}

public static function getParam1() : bool
{
    return self::$param1;
}

public static function getParam2() : string
{
    return self::$param2;
}

public static function getParam3() : int
{...}

/*и т.д.*/

Он проверяет значения в массиве, и если их нету, то выбрасывает исключение ($throw)(), если есть, то это значение присваивается в статическое свойство класса.

На мой взгляд, сейчас проверка значения и выбрасывание исключения выглядит ужасно. Как это можно оформить получше?

6
  • ну сделайте через конфиг какой-нибудь типа [ 'param1' => "core.param1", ... ]
    – teran
    31 окт 2017 в 15:01
  • 1
    В чём смысл этого статического класса? У вас же уже есть класс конфига с методом getData...
    – vp_arth
    31 окт 2017 в 15:01
  • @vp_arth тот видимо чтением занимается исключительно. вообще самое стрёмное тут пока что названия параметров param1 и т.д.
    – teran
    31 окт 2017 в 15:02
  • Названия параметров, надеюсь, тут только для примера. Меня смущает type-hinting в геттерах и отсутствие проверки типов при разборе)
    – vp_arth
    31 окт 2017 в 15:03
  • инспекция кода 31 окт 2017 в 15:05

1 ответ 1

4

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

Сделайте сообщение об ошибке более информативным (какой ключ отсутствует, в каком конфиге). Убедитесь что ваши проверки действительно необходимы в этом подозрительном классе. Избавтесь от лишних зависимостей. Убедитесь что сам подозрительный класс необходим.

1
  • Начав с последнего предложения, можно сэкономить кучу времени)
    – vp_arth
    31 окт 2017 в 15:54

Ваш ответ

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

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