$objectId = $this->repository->getObjectId($name);
if (!$objectId) {
continue;
}
Часто встречаю на ревью (для кода выше) комментарий, что нужно заменить отрицание строгим сравнением: null === $objectId.
Я не понимаю зачем это делать, ведь парадоксальным образом, эта замена делает условие менее строгим. Менее строгим в том
смысле, что, если метод getObjectId вернет по какой-то причине false или 0, то условие не сработает, хотя это
ожидается. Ведь нас интересуют исключительно валидные id, а все остальное можно смело пропускать.