alinpiaa / js-assignments Goto Github PK

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/02-numbers-tasks.js#L187

Math.pow(10, pow) вычисляется дважды, обычно в таких случаях считают один раз и сохраняют в переменную.

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/03-date-tasks.js#L80-L93

Опять немного прямолинейно. Мне нравиться более простой вариант:

return new Date(endDate - startDate).toISOString().slice(...); 

parseDataFromRfc2822 и parseDataFromIso8601 проходят тесты только потому что тесты кривоватые.

Функции просят вернуть объект Date, но возвращаются миллисекунды Эпохи, ибо Date.parse возвращает их.

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/01-strings-tasks.js#L204

Нельзя такое писать в одну строчку, пока поймешь что написано, то глаза сломаешь.
Тут логично на три строки разбить, чтобы все было один под одним, тогда с первого взляда будет понятно.

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/02-numbers-tasks.js#L166

Есть готовая стандартная функция, которая делает то же самое.

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/01-strings-tasks.js#L148

По условию в теге угловые скобки только в начале и в конце, поэтому эффективнее будет использовать slice, а не replace

findFirstSingleChar может быть реализовано за O(N), а не за O(N^2). Кроме того, str.indexOf(value) выглядит не очень в цикле по символам, можно завести счётчик и не выяснять этого каждый раз.

reverseInteger — можно не гонять число в строку и обратно заиспользовав немного математики (остаток от деления на 10 даёт самую правую цифру, деление на 10 и округление сдвигает число вправо). То же и к getDigitalRoot.

В getDigitalRoot использован китайский цикл на 2. Не проходит тест, если передать, например, 991. Сначала получает 19, переходит к второму этапу, получает 10. Должно продолжить и получить 1.

isBracketsBalanced — слишком много дублирующегося кода.

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/01-strings-tasks.js#L72

Тут короче будет:

return value.slice(7, -1);

У slice довольно лаконичные аргументы, надо уметь этим пользоваться.

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/02-numbers-tasks.js#L76

RTFM https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Math/hypot

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/02-numbers-tasks.js#L132-L133

Неэффективное решение. Операции с числами всегда будут быстрее операций со строками.
RTFM https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Arithmetic_Operators#Remainder

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/01-strings-tasks.js#L270-L274

Достаточно прямолинейное решение. Это задачка на строки, а не на массив. Можно попробовать чуть хитрее и эффективней:

return '♣♦♥♠'.indexOf(value.slice(-1)) * 13 + ...

getMovingSum имеет вычислительную сложность больше чем O(N) из-за просмотра массива от начала до текущего элемента каждый раз, как мы меняем элемент. Это можно сделать эффективнее.

В propagateItemsByPositionIndex я не понимаю, зачем в .fill(elem, 0, i*2+1) передаются второй и третий аргументы, если нам нужно заполнить массив целиком. Более того, i*2+1 это больше, чем длина массива.

get3TopItems работает за O(N log N), может за O(3 * N) → O(N).

Можно не создавать лишний массив.. Ещё здесь, здесь.

Линейным поискам в массивах нужно предпочитать поиски в хештаблицах. Видел ещё здесь.

В getIdentityMatrix двойной map излишен, достаточно пройтись по диагонали, а не по всем элементам матрицы.

В distinct сложность O(N^2) вместо O(N) из-за постоянной пробежке по массиве при каждом элементе.

https://github.com/RumataDeEstor/js-assignments/blob/44ed3159cb9acbedd6f517837c5b0dcfc2e64e80/task/02-numbers-tasks.js#L114-L116

В этом модуле задачи не на математику, а на знание стандартных математических методов javascript :) Есть стандартный метод, с помощью которого это задание решается просто, не надо изобретать велосипед.

get99BottlesOfBeer — if (bottle == 1) можно вынести за while сделав while (bottle > 1), if (bottle == 0) тогда тоже пропадает, в цикле остаётся только два yield'а и один декремент.

Этот .pop можно сделать выше при присваивании, ничего не изменится, но на строку меньше.

getPolynom выглядит по-китайски. Не работает на тестах с четырьмя и более аргументами. Можно сделать универсальным.

memoize сразу же выполняет переданную функцию, должно только после первого вызова сконструированной.