PHP 7.4: как проверить, что типизированное свойство инициализировано?

Question

[Посторонним В.]

В пхп 7.4 если мы пытаемся получить доступ к типизированному свойству, которому не засетано значение, всё падает с ошибкой.

Стоит задача - проверить, иницилаизированно ли свойство объекта каким-то значением, включая null.

class A
{
    private ?string $typed;

    public function isTypedInitialized(): bool
    {
        // ???
    }
}

как это сделать? isset возвращает false для null, в итоге мы не можем понять, то ли оно не инициализированно, то ли инициализированно наллом.

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

Comments

[nokimaro]

Почему не приводить null из источника к "пустая строка" при инициализации и тогда можно избежать извращений с reflection и try/catch ?

[Посторонним В.]

nokimaro, в реале там налл или обьект. Строку для простоты привел в примере

[Ипатьев]

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

[Посторонним В.]

Роман Юрьевич Ипатьев,
может не так поняли.
У нас есть например CurrentUserProvider, который возвращает текущего юзера или null, если текущий юзер ID не найден в базе. И вот чтобы каждый раз не искать в базе, делаем в этом классе свойство $currentUser и при первом запросе в него записываем результат поиска в базе - либо налл, либо юзера. При последующих запросах, когда свойство проинициализированно уже, отдаем то что записалось в него.

[Ипатьев]

Не понимаю. Почему нельзя полезть в базу в конструкторе?
Тогда все проблемы снимаются автоматом.

Answer 1

[Alexey Hog]

Ещё такой вариант сработает ))

public function isTypedInitialized(): bool
{
    return (new ReflectionProperty($this, 'typed'))
        ->isInitialized($this);
}

Comments

[Посторонним В.]

Не слишком изящно, но лучше чем ничего.

Answer 2

[Посторонним В.]

Нашел вариант, но это дикость...

public function isTypedInitialized(): bool
{
        try {
            $this->typed;
            return true;
        } catch (Error $e) {
            return false;
        }
}

Тем более что ошибка общая бросается, не специфическая

Comments

[Bone]

Можно завести еще одно свойство флаг, например private $isInitialized = false и в сеттере для $typed устанавливать флаг в true.

[Vitsliputsli]

Ошибки и исключения - это исключительные ситуации, нельзя их использовать как стандартную логику.

Answer 3

[Bone]

Еще как вариант использовать класс:

class StringValue
{
    public ?string $value;
    
    public function __construct(?string $value)
    {
        $this->value = $value;
    }
}

И потом вместо
private ?string $typed;
Использовать
private ?StringValue $typed;

Answer 4

[Vitsliputsli]

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