Как суммировать значения всех свойств объекта, не перечисляя каждое?

Question

[annaozi]

Как переписать total обькта salaries,
чтобы вне зависимости от количества сотрудников, он имел возможность считать зп всех людей?

const salaries = {
  frontend: 12000,
  backend: 10000,
  designer: 8000,
  dayPay() {
    alert('We must pay salary on Tuesday!');
  },
  total() {
    let sum = this.frontend + this.backend + this.designer;
    console.log(sum);
  }
};

salaries.total(); // 3000

salaries.manager = 5000;
salaries.total(); // 35000

Comments

[Elvis]

Вы нашего одобрения хотите получить?

[annaozi]

Мне ресурс порекомендовал вас как експерта. Если что зарание Извиняюсь.

[Станислав Макаров]

Довольно скверная практика - с одним и тем же объектом работать и как со словарём, и как с объектом, занимающимся инкапсуляцией. Если бы вы так не делали, вопроса бы не возникло. И уж тем более всяких адовых костылей, вроде того что предлагает Vancho84

Answer 1

[Vancho84]

Нужно отделить данные о ЗП от методов, например:

total() {
    let sum = Object.values(this)
      .filter((prop) => {
        return typeof prop !== "function";
      })
      .reduce((acc, emp) => {
        return (acc = acc + emp);
      }, 0);
    console.log(sum);
  },

Answer 2

[Евгений Фролов]

Можно переписать в таком виде

const employees = {
    frontend: 12000,
    backend: 10000,
    designer: 8000,
};

const total = (employees = {}) =>
    Object.values(employees).reduce((salary, employeeSalary) => salary + employeeSalary, 0);

console.log(total(employees))
employees.manager = 5000;
console.log(total(employees))

Или если хотите придерживаться вашего варианта то так:

const employeesList = ['frontend', 'backend', "designer"];
const salaries = {
    frontend: 12000,
    backend: 10000,
    designer: 8000,
    total: (staff = []) => staff.reduce((salary, employeeSalary) => salary + employeeSalary, 0),
    dayPay() {
        alert('We mush pay salary on Tuesday!');
    },
};

salaries.total(employeesList) // 30000
employees.manager = 5000;
employeesList.push('manager');
salaries.total(employeesList) // 35000

Comments

[annaozi]

Добрый день спасибо за ответ . В варианте последнем когда пишу в консоль не выводятся цифри там выводится только название.
А первом варианте нарушена структура там daypay нельзя трогать и total нельзя убирать .

[Евгений Фролов]

Поправил, вот так должно работать.

const employeesList = ['frontend', 'backend', "designer"];
const salaries = {
    frontend: 12000,
    backend: 10000,
    designer: 8000,
    total(staff = []) {
        return staff.reduce((salary, employee) => salary + this[employee], 0)
    },
    dayPay() {
        alert('We mush pay salary on Tuesday!');
    },
};

console.log(salaries.total(employeesList)) // 30000
salaries.manager = 5000;
employeesList.push('manager');
console.log(salaries.total(employeesList)) // 35000

Answer 3

[ThunderCat]

total(){ 
  return Object.values(this).reduce((sum, a) => sum + a, 0);
}