completed_exercises/maxchar/index.js

[amcereijo]

The solution should not avoid the second loop?

function maxChar(str) {
  const charMap = {};
  let max = 0;
  let maxChar = '';

  for(let element of str) {
    charMap[element] = charMap[element] ? charMap[element] + 1 : 1;

    if(charMap[element] >  max) {
      max = charMap[element];
      maxChar = element
    }
  }

  return maxChar;
}

instead of

function maxChar(str) {
  const charMap = {};
  let max = 0;
  let maxChar = '';

  for (let char of str) {
    if (charMap[char]) {
      charMap[char]++;
    } else {
      charMap[char] = 1;
    }
  }

  for (let char in charMap) {
    if (charMap[char] > max) {
      max = charMap[char];
      maxChar = char;
    }
  }

  return maxChar;
}