Java Core senior
В чём суть проблемы?
Ответ
Это «магическое» условие — фильтр, смысл которого непонятен из кода. «Берём только нечётные цены» — это бизнес-правило или баг? Если правило — оно должно быть задокументировано или вынесено в именованный предикат с говорящим названием, чтобы намерение стало явным.
Разбор: Главная беда — «магическое» бизнес-правило: фильтр «только нечётные цены» непонятен из кода, неясно, правило это или баг, и его следует задокументировать либо вынести в именованный предикат с говорящим названием. Дистракторы уводят в сторону второстепенных или выдуманных проблем (производительность, NPE, стиль if/тернарник), игнорируя суть — потерю смысла и неявность намерения.
Хочешь так же по своей компании — с вопросами по грейдам и задачами? Закажи гайд или забери свежее в Telegram.