Каковы особенности переменных в ES6, в чем ошибка?

Question

[Егор Чеботарев]

...
 ).then(
   menu => {
      let a = menu;
      let b = func(menu);

      // здесь почему то а = b = reverseMenu
      // и menu = reverseMenu

      function func(m) {
         ...
         return reverseMenu;
      }
   }
)

Есть идеи?

Comments

[Mikhail Osher]

А что вместо троеточия?

[Егор Чеботарев]

Mikhail Osher: Преобразования меню.
Естественно в самой функции я нигде не пишу menu = m (или menu = reverseMenu), просто в конце возвращаю преобразованное значение

[Николай]

Егор Чеботарев: ну вы покажите код, скорее всего дело в том, что где-то вы это делаете : )

[Lynn «Кофеман»]

Егор Чеботарев: Ну так напишите уже что там.
Вангую, что menu это объект или массив и вы изменяете его, а не делаете копию. Судя по названию, вы используте метод reverse у массива который изменяет массив (а не только возвращает «перевёрнутый» массив).

[Егор Чеботарев]

function mergerMenu(menu) {
    let depthPath = {};

    depthPath.quantity = 0;
    depthPath.products = [];

    try {
        for (let i = 0; i < menu.subCategoryLocal.length; i++) {
            let item = menu.subCategoryLocal[i];

            if (item.name == 'Коврики для ванной' || item.name == 'Шторы для ванной' || item.name == 'Полотенца') {

                depthPath.name = 'Текстиль для ванной';
                depthPath.pathCategory = '/departments/bathroom/bathroom_textiles_depth';

                item.products = item.products.map(clientPath);
                item.pathCategory = clientPath(item.pathCategory);

                depthPath.products.push(item);
                depthPath.quantity = (depthPath.quantity + item.quantity);
                menu.subCategoryLocal.splice(i, 1);

                i--;
            }
        }
    }
    catch (err) {
        if (err) {
            console.log(err, 1107);
        }
    }

    menu.subCategoryLocal.forEach(item => {
        item.pathCategory = clientPath(item.pathCategory);

        item.products = item.products.map(clientPath);
        item.products.sort(sortString);
    });

    menu.subCategoryLocal.sort(sortString);
    menu.subCategoryLocal.push(depthPath);
    return menu;

    function clientPath(item) {
        let path;
        let link = '';

        if (typeof item == 'object') path = item.path;
        else path = item;

        path = path.split('/');
        path.splice(0, 1);

        path.forEach(item => {
            if (item == 'simsCategory') item = 'departments';

            if (item != 'Users' &&
                item != 'che' &&
                item != 'Desktop' &&
                item != 'WEB' &&
                item != 'node-js' &&
                item != 'categories') {

                link += '/' + item;

            }

        });

        if (typeof item == 'object') {
            item.path = link;
            return item;
        }
        else {
            return link;
        }
    }

    function sortString(a, b) {
        if (a.name > b.name) return 1;
        if (a.name < b.name) return -1;
    }

}

К сожалению вряд ли, товарищи, тем более что эта функция находится вне промиса и не имеет доступа к menu

[Lynn «Кофеман»]

Ну и, никакого отношения к ES6 вопрос не имеет.

Изучайте learn.javascript.ru/object-reference

[Егор Чеботарев]

Алексей Тен: скопируйте этот комментарий в ответы, чтобы можно было его отметить. Пришлось помучаться с глубоким копированием, но вы оказались правы.

Answer 1

[DenJel]

Нужен код функции func, скорее всего Вы меняете исходный объект, а не его копию.

UPDATE: А все нашел у вас комментариях, ну разумеется вы меняете входной объект меню, в начале функции mergerMenu необходимо сделать чтото типо :

const newMenu = Object.assign( {}, menu )

и потом его уже возвращать из функции

return newMenu;

ну или если уж вы любитель es6
const newMenu = { ...menu }

UPDATE: все ключи объекта с таким подходом копируются по ссылке, если вам нужно абсолютно новое меню, нужно делать глубокое копирование, а это процесс довольно ресурсоемкий. Советую посмотреть на Immutable.js