История изменений
Исправление witaway, (текущая версия) :
Какая религия
Никакая.
И короче и понятнее ведь.
Поэтому и говорю: «сугубо моё личное ИМХО», «вопрос вкуса». Ни в коем случае не утверждаю, что это самое правильное лучшее решение.
Я просто ощущаю, что после if-а семантика переменной самую-самую капельку меняется. Это два очень похожих, но разных по смыслу объекта. Они вполне могут иметь два разных имени.
Я люблю настолько очевидный код, насколько это возможно. Переменных мне тоже не жалко. Зато теперь я работаю с editingUser
и даже не пытаюсь держать в голове, как он мог меняться и какие у requestedUser
были умолчания.
Есть до и есть после, о них можно думать полностью независимо. Ещё чуть вырастет количество логики — возможно, будет правильнее вынести в метод.
Пример правда очень упрощенный… Но если придумать что-то более объёмное и неочевидное, падение когнитивной нагрузки становится ощутимым. Опять-таки, это ИМХО.
Если тебе кажется, что будет лучше сделать по-другому — пожалуйста. Твой пример кода тоже вполне адекватен ситуации. Мы ж тут собрались не религиозные войны вести. Я просто предлагаю свой вариант и говорю, что мне в нём нравится. 🤷♂️
Исходная версия witaway, :
Какая религия
Никакая.
И короче и понятнее ведь.
Поэтому и говорю: «сугубо моё личное ИМХО», «вопрос вкуса». Ни в коем случае не утверждаю, что это самое правильное лучшее решение.
Я просто ощущаю, что после if-а семантика переменной самую-самую капельку меняется. Это два очень похожих, но разных по смыслу объекта. Они вполне могут иметь два разных имени.
Я люблю настолько очевидный код, насколько это возможно. Переменных мне тоже не жалко. Зато теперь я работаю с editingUser
и даже не пытаюсь держать в голове, как он мог меняться и какие у requestedUser
были умолчания.
Есть до и есть после, о них можно думать полностью независимо. Ещё чуть вырастет количество логики — возможно, будет правильнее вынести в метод.
Пример правда очень упрощенный… Но если придумать что-то более объёмное и неочевидное, падение когнитивной нагрузки становится ощутимым. Опять-таки, это ИМХО.
Если тебе кажется, что будет лучше сделать по-другому — пожалуйста. Твой пример кода тоже вполне адекватен ситуации.