Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Добрый день!
Основное задание выполнено в файле main.js
Так же добавил для разбора угадайку (numbers.js) и "быки-коровы" (cows.js), вопросы по cows интересующие в самом низу написал.
Так же возник вопрос по правильному пониманию и использованию стрелочных функций в различных методах (часто используют в методах для работы с массивами, например) - подробно описал в файле method.js
К индексу изначально подключен main.js, остальные перечисленные файлы закомментированы следом за main.js
По main.js
Так же возник еще ряд теоретических вопросов:
cows.js:
В целом по всему коду написал комментарии к тем или иным действиям, в отладке все, вроде бы, правильно. Использовал методы преобразования строк и массивов. В большей степени интересует, насколько верны использованные логические выражения для проверок и методов обработки массивов, а также использование return и break (что может пойти не по запланированному сценарию). Ну и что можно было бы еще оптимизировать, какие методы были бы более эффективны. Плюс под самим кодом написал, на какие warnings указал phpstorm (ошибок нет, при этом, только предупреждения, и вообще все функционирует), хотелось бы разобраться, что они означают.
numbers.js аналогично, warnings тоже перечислил в комментарии в конце файла