Какая или какие строки кода изменяют массив

@zorin1497 · Регистрация: 22.06.2022

Author24 — интернет-сервис помощи студентам

Добрый день! Не могу разобраться, какая строка или строки кода изменяют массив cid, переданный в качестве аргумента функции и как можно это исправить.
Пробовал внутри функции копировать массив с помощью slice и [...arr], оригинальный массив все равно изменяется.

Javascript

let nominal = {"ONE HUNDRED": 100, "TWENTY": 20, "TEN": 10, "FIVE": 5, "ONE": 1, "QUARTER": 0.25, "DIME": 0.1,  "NICKEL": 0.05, "PENNY": 0.01};
let key_nominal = Object.keys(nominal);
function findIndex(name, arr) {
    for (let i in arr) {if (arr[i][0] == name) {return i}};
}
function calculate(change, cid) {
    let arr_nom = [];
    for (let i in key_nominal) {
        let sum = 0;
        while(change >= nominal[key_nominal[i]] && cid[findIndex(key_nominal[i], cid)][1] > 0) {
            change = (change - nominal[key_nominal[i]]).toFixed(2);
            cid[findIndex(key_nominal[i], cid)][1] -= nominal[key_nominal[i]];
            sum += nominal[key_nominal[i]];
        }
        if (sum > 0) {arr_nom.push([key_nominal[i], sum]);}
    }
    return arr_nom;
}
function rest(change, cid) {
    for (let i in key_nominal) {
        while(change >= nominal[key_nominal[i]] && cid[findIndex(key_nominal[i], cid)][1] > 0) {
            change = (change - nominal[key_nominal[i]]).toFixed(2);;
            cid[findIndex(key_nominal[i], cid)][1] -= nominal[key_nominal[i]];
        }
    }
    return change != 0 ? true : false;
}
function checkCashRegister(price, cash, cid) {
    let change = cash - price;
    let av_nominal = cid.filter((item) => nominal[item[0]] <= change);
    let all_cash = av_nominal.reduce((sum, item) => sum + item[1], 0);
    if (change == all_cash) {return {status: "CLOSED", change: cid};}
    else if (all_cash < change || rest(change, cid)) {
        return {status: "INSUFFICIENT_FUNDS", change: []};
    }
    return {status: "OPEN", change: calculate(change, cid)};
}
console.log(checkCashRegister(3.26, 100, [["PENNY", 1.01], ["NICKEL", 2.05], ["DIME", 3.1], ["QUARTER", 4.25], ["ONE", 90], ["FIVE", 55], ["TEN", 20], ["TWENTY", 60], ["ONE HUNDRED", 100]]));

Добавлено через 9 минут
В каждой функции сокпировал массив в новую переменную, но оригинальный массив все равно изменяется...

Javascript

let nominal = {"ONE HUNDRED": 100, "TWENTY": 20, "TEN": 10, "FIVE": 5, "ONE": 1, "QUARTER": 0.25, "DIME": 0.1,  "NICKEL": 0.05, "PENNY": 0.01};
let key_nominal = Object.keys(nominal);
function findIndex(name, arr) {
    for (let i in arr) {if (arr[i][0] == name) {return i}};
}
function calculate(change, cid) {
    let arr_nom = [];
    let newcid = cid.slice();
    for (let i in key_nominal) {
        let sum = 0;
        while(change >= nominal[key_nominal[i]] && newcid[findIndex(key_nominal[i], newcid)][1] > 0) {
            change = (change - nominal[key_nominal[i]]).toFixed(2);
            newcid[findIndex(key_nominal[i], newcid)][1] -= nominal[key_nominal[i]];
            sum += nominal[key_nominal[i]];
        }
        if (sum > 0) {arr_nom.push([key_nominal[i], sum]);}
    }
    return arr_nom;
}
function rest(change, cid) {
    let newcid = cid.slice();
    for (let i in key_nominal) {
        while(change >= nominal[key_nominal[i]] && newcid[findIndex(key_nominal[i], newcid)][1] > 0) {
            change = (change - nominal[key_nominal[i]]).toFixed(2);;
            newcid[findIndex(key_nominal[i], newcid)][1] -= nominal[key_nominal[i]];
        }
    }
    return change != 0 ? true : false;
}
function checkCashRegister(price, cash, cid) {
    let change = cash - price;
    let av_nominal = cid.filter((item) => nominal[item[0]] <= change);
    let all_cash = av_nominal.reduce((sum, item) => sum + item[1], 0);
    if (change == all_cash) {return {status: "CLOSED", change: cid};}
    else if (all_cash < change || rest(change, cid)) {
        return {status: "INSUFFICIENT_FUNDS", change: []};
    }
    return {status: "OPEN", change: calculate(change, cid)};
}
console.log(checkCashRegister(3.26, 100, [["PENNY", 1.01], ["NICKEL", 2.05], ["DIME", 3.1], ["QUARTER", 4.25], ["ONE", 90], ["FIVE", 55], ["TEN", 20], ["TWENTY", 60], ["ONE HUNDRED", 100]]));

krvsa · 18.07.2022, 16:00

Сообщение от zorin1497

Пробовал внутри функции копировать массив с помощью slice и [...arr], оригинальный массив все равно изменяется.

Такое клонирование клонирует только сам массив. Сложные элементы ссылаются на те же элементы.
Как вариант - превращать все в джейсон-строку и потом парсить. Так у тебя получится какой/никакой, а клон.

@zorin1497 · 18.07.2022, 16:06 **[ТС]**

Есть вариант обойтись без json?

krvsa · 18.07.2022, 16:18

Сообщение от zorin1497

Есть вариант обойтись без json?

Писать свою функцию...
Типа такой

Javascript

const Y = deepClone(Z);
function deepClone(obj) {
  const clObj = {};
  for(const i in obj) {
    if (obj[i] instanceof Object) {
      clObj[i] = deepClone(obj[i]);
      continue;
    }
    clObj[i] = obj[i];
  }
  return clObj;
}
Y.addnlProp = { fd: 45 };
console.log('Z object after cloning: ', Z);
console.log('Y object: ', Y);

https://habr.com/ru/post/480786/

@Balanaar · 18.07.2022, 16:19

А ещё лучше не костылить и продумать архитектуру программы более детально.

@zorin1497 · 18.07.2022, 16:21 **[ТС]**

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

Javascript

function rest(change, cid) {
    for (let i in key_nominal) {
        let amount = cid[findIndex(key_nominal[i], cid)][1];
        while(change >= nominal[key_nominal[i]] && amount > 0) {
            change = (change - nominal[key_nominal[i]]).toFixed(2);;
            amount -= nominal[key_nominal[i]]}}
    return change != 0 ? true : false;

krvsa 3499 / 1336 / 368 Регистрация: 14.03.2022 Сообщений: 3,211
	18.07.2022, 16:00	2
	Сообщение от zorin1497 Пробовал внутри функции копировать массив с помощью slice и [...arr], оригинальный массив все равно изменяется. Такое клонирование клонирует только сам массив. Сложные элементы ссылаются на те же элементы. Как вариант - превращать все в джейсон-строку и потом парсить. Так у тебя получится какой/никакой, а клон. 0

@zorin1497 0 / 0 / 0 Регистрация: 22.06.2022 Сообщений: 13
	18.07.2022, 16:06 [ТС]	3
	Есть вариант обойтись без json? 0

@Balanaar 2463 / 1769 / 625 Регистрация: 11.07.2016 Сообщений: 4,067
	18.07.2022, 16:19	5
	А ещё лучше не костылить и продумать архитектуру программы более детально. 0

	18.07.2022, 16:21

Опции темы