Почему ошибка при запуске?

Question

[Antonio09]

1) Почему ошибка при запуске? Что поправить?
2) И почему при объявлении переменных пишут var max = 0;
var maxName = null;

а не к примеру

var max = null;
var maxName = null;

https://jsfiddle.net/q9zpv7n5/1/

Comments

[Виктор Л]

Если про ошибку в jsfiddle.net/q9zpv7n5/1/ то там проблема в области видимости переменных var maxName только в функции

Answer 1

[Виктор Л]

var maxName = null;
var max = 0; 
var salaries = {
  "John": 100,
  "Pete": 300,
  "Mary": 250
};
console.log(maxName);
(function topSalary() {  
    for(const [name, salary] of Object.entries(salaries)) {
      if (max < salary) {
        max = salary;
        maxName = name;
      }
    } 
    return maxName;
  })()
console.log(maxName);

Comments

[Antonio09]

Виктор Л, спасибо
а без немедленного вызова функции можно?

[Сергей Мелодин]

Antonio09, можно в одну строку:

Object.entries({
  "John": 100,
  "Pete": 300,
  "Mary": 250
}).reduce((acc, [name, salary]) => (salary > acc.salary ? { name, salary } : acc), { name: '', salary: 0 })

[Виктор Л]

Antonio09, Уберите конструкцию (function topSalary(){})()
и вызовите topSalary()

Answer 2

[Сергей Мелодин]

Попробуйте свою функцию всё-таки вызвать :)

При объявлении переменных лучше указывать тот тип, который там будет содержаться - для числовых 0, для строчных - пустая строка.

PS
Ещё погуглите проблематику глобальных переменных. Весь этот код может вполне без них обойтись