История изменений
Исправление zg, (текущая версия) :
Я люблю настолько очевидный код, насколько это возможно. Переменных мне тоже не жалко. Зато теперь я работаю с editingUser и даже не пытаюсь держать в голове, как он мог меняться и какие у requestedUser были умолчания.
Скорее всего ты любишь многословный, то есть менее читабельный код. В итоге то, что можно сделать меньшим количеством строк ты делаешь их бОльшим количеством. Если ты не один в проекте, вероятность спутать requestedUser с editingUser ниже по коду увеличилась. А ведь логика кода очень простая - если нам не дали пользователя, мы просто берём его сами из другого места. Нет никакого смысла держать две переменные под это. А представь, что таких объектов несколько или что ты получил varargs таких объектов. Ты будешь весь массив дублировать для этого?
Если тебе кажется, что будет лучше сделать по-другому — пожалуйста. Твой пример кода тоже вполне адекватен ситуации. Мы ж тут собрались не религиозные войны вести. Я просто предлагаю свой вариант и говорю, что мне в нём нравится. 🤷♂️
Религиозную войну объявил какой-то самодур (или их несколько) который заставил весь наш R&D в трёх странах использовать maven-checkstyle-plugin с требованием объявлять все аргументы методов как final и валящий билд в противном случае.
Исходная версия zg, :
Я люблю настолько очевидный код, насколько это возможно. Переменных мне тоже не жалко. Зато теперь я работаю с editingUser и даже не пытаюсь держать в голове, как он мог меняться и какие у requestedUser были умолчания.
Скорее всего ты любишь многословный, то есть менее читабельный код. В итоге то, что можно сделать меньшим количеством строк ты делаешь их бОльшим количеством. Если ты не один в проекте, вероятность спутать requestedUser с editingUser ниже по коду увеличилась. А ведь логика кода очень простая - если нам не дали пользователя, мы просто берём его сами из другого места. Нет никакого смысла держать две переменные под это. А представь, что таких объектов несколько или что ты получил varargs таких объектов. Ты будешь весь массив дублировать для этого?
Если тебе кажется, что будет лучше сделать по-другому — пожалуйста. Твой пример кода тоже вполне адекватен ситуации. Мы ж тут собрались не религиозные войны вести. Я просто предлагаю свой вариант и говорю, что мне в нём нравится. 🤷♂️
Религиозную войну объявил какой-то самодур (или их несколько) который заставил весь наш R&D в трёх странах использовать maven-checkstyle-plugin с требованием объявлять все аргументы методов final и валящий билд в противном случае.