Нужно больше функций #3
Conversation
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
js/function.js
Outdated
|
|
||
| function isPalindrome(word) { | ||
| if (!word.length) { | ||
| return; |
There was a problem hiding this comment.
Хорошая проверка, но давай функция будет возвращать только boolean. Это и из логики, и из названия следует. Сейчас ты undefined вернешь
js/function.js
Outdated
| } | ||
| word = word.toLowerCase(); | ||
| word = word.replaceAll(' ', ''); | ||
| let lastLetter = word.length - 1; |
There was a problem hiding this comment.
Хорошее решение за неполный проход слова 👍
Один вопрос: зачем эта переменная? Кажется, можно обойтись без нее и сразу индекс отнимать. Но если хочешь оставить, то лучше переименовать. Сейчас кажется, что ты хранишь тут букву, а не длину. Я сходу не понял, как ты букву используешь вместо индекса, пришлось вчитываться)
There was a problem hiding this comment.
Переменную убрал, правда была лишней
js/function.js
Outdated
| isPalindrome('Кекс'); // false | ||
| isPalindrome('Лёша на полке клопа нашёл '); | ||
|
|
||
| function takeNum(string) { |
There was a problem hiding this comment.
Имя функции не совсем отражает суть) Тут лучше что-то от слова "извлекать" или "получить"
В аргументах string выглядит логичным, но потом может быть не удобно в TypeScript из-за типов, получится что-то вида (string: string)=>. Поэтому можно сразу задумываться об этом, чтобы не привыкать) Функция палиндрома хороший пример с аргументом word
Это усложнение будешь делать?)
имяФункции(2023); // 2023
имяФункции(-1); // 1
имяФункции(1.5); // 15
There was a problem hiding this comment.
Имя поправил)
Даже не подумал о ts, сейчас исправил на word и дописал функцию
|
Можно настроить линтер под себя в файле .eslintrc. |
keksobot
merged commit 1ad4b2d
into
htmlacademy-univer-javascript-1:master
🎓 Нужно больше функций
💥 https://htmlacademy-univer-javascript-1.github.io/2579201-kekstagram-5/3/