Как упростить данный код?

Question

[Михаил]

module.exports = function (names) {
	var numArr;
	var busNames = [];

	for (var i = 0; i <= 9; i++) {
		numArr = names.filter( function(word) {
			var firstLetter = word.split('')[0];
			return firstLetter == i;
		});
        
		numArr.unshift(''+ i);
		if (numArr.length > 1) busNames.push(numArr);
	};


	var letArr = names.filter(function(word) {
		var firstLetter = word.split('')[0];
		return (firstLetter >= 'а' && firstLetter <= 'я') || (firstLetter >= 'А' && firstLetter <= 'Я');
	});
    
	letArr.unshift('А-Я');


	busNames.push(letArr);
	return busNames;
};

Код принимает массив названий и фильтрует их по первому символу.
Буквы к буквам, цифра к цифре.

Спасибо.)

Comments

[profesor08]

А для кого код должен стать проще?

Answer 1

[0xD34F]

Object.values(names.reduce((acc, n) => {
  const g = /^\d/.test(n[0]) ? n[0] : /^[А-Я]/i.test(n[0]) ? 'А-Я' : null;
  g && (acc[g] = acc[g] || [ g ]).push(n);
  return acc;
}, {}))

Comments

[forspamonly2]

[ g ] - тут g не лишняя?

и номера с нулей начинающиеся оно пропустит. у автобусов такое бывает.

[0xD34F]

forspamonly2,

[ g ] - тут g не лишняя?

Не лишняя. У автора же выполняется numArr.unshift(''+ i), letArr.unshift('А-Я') - первым элементом должно идти имя группы.

номера с нулей начинающиеся оно пропустит

Не пропустит, !!"0" === true.