Несколько придирок к коду в первом примере. Вы неправильно используете строковую функцию trim(). Строки никогда не изменяются, поэтому каждая строковая функция всегда возвращает новую строку. Другими словами, вызов someString.trim() без присваивания не будет иметь никакого эффекта. Он обрежет строку, а затем отбросит результат. Чтобы обрезать строку, вы должны присвоить обрезанную строку где-нибудь, например, someString = someString.trim();.
Вместо того, чтобы разбивать строку ввода на '|' несколько раз, вы можете просто разделить ее один раз, например так: let [,,,,timestamp,,,name] = line.split( '|' ); Чтобы избежать постоянного вызова функции trim, вот трюк: let [,,,,timestamp,,,name] = line.split( /[\s\uFEFF\xA0]*\|[\s\uFEFF\xA0]*/ ); Это удалит все пробелы по обе стороны от оператора конвейера | и вам не нужно будет обрезать, хотя вы можете захотеть сделать одну обрезку на всей строке ввода, если вы подозреваете, что она может начинаться или заканчиваться пробелом.
Кроме того, размещение имен в массиве довольно затратно, и вам не нужно этого делать. Просто используйте имена по мере их поступления, чтобы вычислить то, что вам нужно. То же самое для пожертвований по дате. Например, для дат вы собираете их все, а затем используете их для подсчета пожертвований в месяц. Дорого содержать их всех вокруг. Вместо этого просто добавляйте их по мере поступления в функцию чтения строк:
const yrMnth = timestamp( /^(\d{4})(\d\d)/, '$1-$2' );
dateDonations[ yrMnth] = ( dateDonations[ yrMnth] || 0 ) + 1;
Выполнение Object.entries() само по себе дорого, но создание такого массива еще дороже. На самом деле, вас вообще не волнует сортировка, вам просто нужен максимум, так что const popular = Object.entries( dupeNames ).reduce( ( winner, cur ) => winner[ 1 ] >= cur[ 1 ] ? winner : cur, [ '', Number.NEGATIVE_INFINITY ] );
Вы также создаете выходной поток, который никогда не используется. Вам это не нужно. Просто создайте считыватель только для ввода: readline.createInterface( { input: fs.createReadStream( 'test.txt' ) } ); и все готово.
Наконец, вы никогда больше не должны использовать var. От него (почти) никогда нет никакой пользы, и он ведет себя довольно причудливо и подвержено ошибкам. Если вы не записываете какой-либо идентификатор, используйте const, так как он не только поясняет читателю, что он никогда больше не будет записываться, но и ускоряет ваш код, поскольку теперь компилятор знает, что ему не нужно беспокоиться о тип (или значение) постоянно меняется. Для других переменных используйте let.
В общем, хорошая статья о распространенной задаче, с которой время от времени сталкивается большинство программистов. Честно говоря, я думаю, что было бы очень полезно больше статей такого рода. Спасибо, что поделился. :)
