id - в чем проблема? А он нужен часто, так что, уже вообще нельзя что ли получать чисто одно необходимое здесь и сейчас свойство? Имхо, нужно - значит добавлю метод, и ничего не вижу в этом, сеттера нет, есть геттер, ну назови ты иначе если не нравится префикс get )), это генерируемый идентификатор , это как тот же инкремент получить от запроса
Ну setStatus не обязательно просто сетит, внутри могут быть какие-то проверки. Но то, что setStatus неинформативный и скорее всего его можно рефакторнуть в несколько методов это, кажется, другая проблема уже
Сеттеры ВСЕГДА были пустыми во всех проектах, что я видел «Модно инкапсулировать» слышал тоже всегда в их защиту
оно и понятно почему всегда пустые: вот добавил логику для некоторых кейсов, а юзается в хвост и гриву в 100 мест этот сеттер, как поймать те юзкейсы, где не надо 🙂
Обсуждают сегодня