Гафурова Софья#79
Conversation
|
🍅 Пройдено тестов 58 из 74 |
|
🍅 Пройдено тестов 59 из 74 |
|
🍅 Пройдено тестов 59 из 74 |
|
🍅 Пройдено тестов 59 из 74 |
|
🍅 Пройдено тестов 60 из 74 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Пройдено тестов 61 из 74 |
|
🍅 Пройдено тестов 62 из 74 |
| throw new RangeError(); | ||
| } | ||
|
|
||
| return Math.trunc(year / 100) + 1; |
There was a problem hiding this comment.
обрати внимание на крайний случай при переходе к другому веку
| throw new RangeError(); | ||
| } | ||
|
|
||
| if (!/#[a-f|A-F|0-9]+/.test(hexColor)) { |
There was a problem hiding this comment.
эта регулярка проверит не всю строку целиком, а есть в ней такая подстрока
например, #11XXXX ее пройдет, хотя не должен, так как сматчится подстрока #11
|
|
||
| let matrixResult = []; | ||
|
|
||
| for (let x = 0; x < matrix[0].length; x++) { |
There was a problem hiding this comment.
матрица необязательно квадратная, размерность может быть например 3*5
судя по этому циклу ты это не учитываешь
| * @returns {String} Число n в системе счисления targetNs | ||
| */ | ||
| // eslint-disable-next-line | ||
| function numberSystemProblem(n, targetNs) { |
There was a problem hiding this comment.
число n может быть дробным и отрицательным
| } | ||
|
|
||
| const abc = ['0', '1', '2', '3', '4', '5', '6', '7', '8', '9', 'A', 'B', | ||
| 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'Q', |
| */ | ||
| function phoneProblem(phoneNumber) { | ||
| // Ваше решение | ||
| return /8-800-\d\d\d-\d\d-\d\d/.test(phoneNumber); |
There was a problem hiding this comment.
тут снова не учитывается, что это строка полностью: после валидного номера я могу добавить в конце или в начале что угодно и проверка пройдет
| @@ -72,7 +182,7 @@ function numberSystemProblem(n, targetNs) { | |||
| * @returns {Boolean} Если соответствует формату, то true, а иначе false | |||
| */ | |||
| function phoneProblem(phoneNumber) { | |||
There was a problem hiding this comment.
если аргумент приходит не в формате string, то надо бросить TypeError
| */ | ||
| function smilesProblem(text) { | ||
| // Ваше решение | ||
| const matchResult = text.match(/(:-\))/g); |
There was a problem hiding this comment.
в условии два типа улыбающихся смайликов
не учитываешь (-:
|
🍅 Пройдено тестов 63 из 74 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Пройдено тестов 64 из 74 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Пройдено тестов 63 из 74 |
|
🍅 Пройдено тестов 66 из 74 |
|
🍅 Пройдено тестов 67 из 74 |
|
🍅 Пройдено тестов 67 из 74 |
|
🍅 Пройдено тестов 68 из 74 |
|
🍅 Пройдено тестов 69 из 74 |
|
🍅 Пройдено тестов 69 из 74 |
|
🍅 Пройдено тестов 69 из 74 |
|
🍅 Пройдено тестов 69 из 74 |
|
🍅 Пройдено тестов 70 из 74 |
|
|
||
| if (matrix.length === 0) { | ||
| return []; | ||
| } |
There was a problem hiding this comment.
Надо все же еще проверить что внутри массива тоже массивы
| throw new RangeError(); | ||
| } | ||
|
|
||
| let [int, d] = n.toString().split('.'); |
There was a problem hiding this comment.
а не надо сначала сделать n положительным? у тебя сейчас с отрицательными не работает
|
🍅 Пройдено тестов 70 из 74 |
|
🍅 Пройдено тестов 71 из 74 |
3 participants
No description provided.