Как можно оптимизировать данную функцию?

Question

[Andy Oker]

Функция places создает новый массив, сортирует объекты внутри по алфавиту и разбивает их на подмассивы по первой букве в параметре свойства name.
Пример результата:

[
  [
    {name: Архангельск},
    {name: Астрахань}
  ],
  [
    {name: Барнаул},
  ],
  [
    {name: Вологда},
    {name: Владимир},
  ]
]

PlaceList - одномерный массив объектов, который не должен быть изменен

places() {
      let places = PlacesList.slice()

      places.sort(function (a, b) {
        return a.name.localeCompare(b.name)
      })

      let arrays = []

      while (places.length > 0) {
        arrays.push(places.splice(0, places.filter(el => el.name[0] === places[0].name[0]).length))
      }

      return arrays
    },

Comments

[Lynn «Кофеман»]

А что она делает?

[Andy Oker]

Lynn «Кофеман», Функция places создает новый массив и разбивает его на подмассивы с объектами по алфавиту.
Пример результата:

[
  [
    {Архангельск},
    {Астрахань}
  ],
  [
    {Барнаул},
  ],
  [
    {Вологда},
    {Владимир},
  ]
]

[Сергей Сергей]

Можно просто взять алфавит и для каждой буквы нафильтровать значений из PlacesList, потом вычеркнуть пустые.

'АБВГДЕЁЖЗИЙКЛМНОПРСТУФХЦЧШЩЪЫЬЭЮЯ'.split('').map( abl => PlacesList.filter( p => abl === p.name[0] ) ).filter( l => l.length )

типа так.

[NR]

Сергей Сергей, такая себе оптимизация

[0xD34F]

Как сгруппировать и отсортировать массив имён?

[Сергей Сергей]

aedam, для списка городов России подойдёт, не в mousemove же это толкать.

Answer 1

[NR]

function places() {
  const places = [...PlacesList];

  const sortedPlaces = {};

  for (const place of places) {
    const letter = place.name[0];

    if (!(letter in sortedPlaces)) {
      sortedPlaces[letter] = [];
    }

    sortedPlaces[letter].push(place);
  }

  return Object.values(sortedPlaces);
}