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

В настоящий момент получила широкое распространение практика Continuous integration (CI) (а также практики Continuous Deployment и Continuous Delivery, которые ей сопутствуют), которая позволяет запускать различные проверки кода и тесты автоматически. Тем самым гарантируется непрерывный контроль качества и работоспособности кода, в отличие от избирательных ручных запусков тестов и проверок.