Javascript: Инкрементное имя, если существует

Question

Я хочу вставить имя в массив, имя будет именем по умолчанию.
Если имя уже существует, мне нужно добавить целое число перед нажатием, что-то вроде увеличения, если экземпляр существует.

Если он не существует, он должен создавать экземпляр. Будет делать то же самое во всем процессе, но он должен увеличиваться.

var arr = ['team banana', 'team melon', 'team melon 2', 'team orange'] 
var name = 'team apple'; // this is default team name
var xarr = [];

if(arr.length){
    // store the all instances of 'team apple'
    for(var c = 0; c <= arr.length; c++){
        if(arr[c] !== undefined){
            if(arr[c].indexOf('team apple')  > -1){
                xarr.push(arr[c].replace(/['0-9']/g,''))
            }
        }
    }
    // start the iterration
    for (var k=0; k<=arr.length; k++){
        // if existing
        if(arr[k] !== undefined){
            if(name === arr[k]) {
                arr.push(name+' ' + ( xarr.length))
            }else{
                // if not existing creat an instance of 'team apple'
                // arr.push(name) <--- this cause the browser to crash
            }
        }
    }
    xarr = []
}else{
    arr.push(name )
}

console.log(arr)

То, что я ожидаю в результате, выглядит примерно так:

['team banana', 'team melon', 'team melon 2', 'team orange','team apple','team apple 1', 'team apple 2' ...]

Блок кода уже работает, за исключением того, что если в массиве нет экземпляра 'team apple', он начинает падать

Если массив выглядит так, все работает хорошо:

['team banana', 'team melon', 'team melon 2', 'team orange','team apple']

Но если это выглядит так:

['team banana', 'team melon', 'team melon 2', 'team orange']

Это браузер начинает падать.

Это та часть кода, которая его вызывает, я понятия не имею.

if(name === arr[k]) {
    arr.push(name+' ' + (xarr.length))
}else{
    // if not existing creat an instance of 'team apple'
    // arr.push(name) <--- this cause the browser to crash
}

Я даже не уверен, является ли это правильной реализацией для этого, где я действительно хотел, это увеличивающееся имя. любое предложение будет огромной помощью. Также я ограничен только ES5.

Заранее спасибо

Answer 1

Гораздо проще просто использовать фильтр и принимать полученную длину в качестве постфикса.В вашем коде много циклов, и ни один, кроме одного, не понадобится

const arr = ['team banana', 'team melon', 'team melon 2', 'team orange']
const name = 'team apple';

function addToArr(arr, name) {
  const postfix = arr.filter(e => e.replace(/\s\d+$/, '') === name).length || '';
  arr.push(`${name} ${postfix}`.trim());
  return arr;
}

console.log(addToArr(arr, name));
console.log(addToArr(arr, name));
console.log(addToArr(arr, name));

Версия ES 5

var arr = ['team banana', 'team melon', 'team melon 2', 'team orange']
var name = 'team apple';

function addToArr(arr, name) {
  var postfix = arr.filter(function(e) { return e.replace(/\s\d+$/, '') === name}).length || '';
  arr.push((name + ' ' + postfix).trim());
  return arr;
}

console.log(addToArr(arr, name));
console.log(addToArr(arr, name));
console.log(addToArr(arr, name));

Answer 2

Вам нужно только определить, сколько экземпляров команды для добавления существует в массиве. Для этого вы можете использовать Array.filter ( см. MDN ) или свой собственный метод фильтрации. Что-то вроде:

(() => {
  let arr = ['team banana', 'team melon', 'team melon 2', 'team orange'];
  const teamApple = "team apple";

  addTeam(teamApple, arr);
  addTeam(teamApple, arr);
  addTeam(teamApple, arr);
  addTeam(teamApple, arr);
  addTeamAlternative(teamApple, arr);
  console.log(arr);
  
  // will it work for an empty array?
  let newArr = [];
  addTeamAlternative(teamApple, newArr);
  addTeamAlternative(teamApple, newArr);
  addTeam("team kiwifruit", newArr);
  addTeam("team kiwifruit", newArr);
  console.log(newArr);
  
  // Array.filter and ES20xx
  function addTeam(team, someArr) {
    const nExisting = someArr.filter(teamFromArray => teamFromArray.startsWith(team)).length;
    someArr.push(team + (nExisting == 0 ? "" : ` ${nExisting + 1}`));
    return someArr;
  }
  
  // Your own method (a more 'classic' alternative)
  function addTeamAlternative(team, someArr) {
    var arrSearch = someArr.slice(0);
    var value = arrSearch.shift();
    var nExisting = 1;
    while(value) {
      nExisting += +RegExp("^" + team).test(value);
      value = arrSearch.shift();
    }
    someArr.push(team + (nExisting > 1 ? (" " + nExisting) : ""));
    return someArr;
  }
})()