Изменение ссылки на аргумент js-функции

В последнее время я пишу приложения response-redux, и как разработчик React я пишу чистый, функциональный и предсказуемый код. Несмотря на то, что мне нравится этот опыт, я сомневаюсь, красив ли мой код или нет.

Итак, у меня есть дерево в моем состоянии, и мне нужно обновить кучу узлов в дереве. Допустим, API дерева предоставляет метод pureUpdate(path, newNode, tree) => newTreeчистый, который возвращает новое дерево с обновленным узлом. В этом случае мой метод редуктора может выглядеть как

function updateNodes(tree, updateRules) {
    updateRules.forEach(updateRule => {
        const { path, node } = updateRule;
        tree = pureUpdate(path, node, tree);
    });
    return tree;
}

Но я не уверен, что можно было бы сделать лучшее.

Первое, что выглядит неприятно, - это tree = pureUpdate(path, node, tree);. Похоже на изменение параметра, что не рекомендуется, но я просто переназначаю ссылка, не так ли? Это объясняется здесь во второй части ответа. Но хотя этот трюк может быть приемлемым, в этом обсуждении сказал, что такой код может быть неоптимизированным, и переназначение параметров может вызвать проблемы с производительностью (больше информации с примерами). Самое простое исправление, которое пришло мне на ум, - использовать дополнительную переменную, которая будет клоном дерева.

function updateNodes(tree, updateRules) {
    let newTree = someCloneFunc(tree);
    updateRules.forEach(updateRule => {
        const { path, node } = updateRule;
        newTree = pureUpdate(path, node, newTree);
    });
    return newTree;
}

Вопрос в том, ничего ли я не пропускаю, а мой код остается чистым, красивым и не вызовет никаких проблем.

Если вы хотите заниматься чисто функциональным программированием, никогда используйте forEach.

Bergi 19.11.2018 20:46
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
2
1
89
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Если вас вообще беспокоит производительность, я бы не стал клонировать tree только для того, чтобы не переназначать параметр.

Хотя здесь вы можете использовать forEach и переназначить параметр, reduce - это правильная функциональная абстракция для вашего варианта использования, и, как правило, это лучшая и более полезная абстракция, чем forEach, поскольку ее можно (и нужно) использовать в чистом виде, тогда как forEach всегда на стороне. последствия.

Решение, основанное на reduce, также делает вопрос о том, следует ли клонировать и / или переназначать параметр функции, полностью спорным.

Вот рабочее решение для reduce - без переназначения параметров, без побочных эффектов forEach и без причин для клонирования tree:

const updateNodes = (tree, updateRules) =>
  updateRules.reduce(
    (acc, { path, node }) => pureUpdate(path, node, acc),
    tree // initialize acc (the accumulator)
  )

Другие вопросы по теме