search

anrypwnz / validation

0 stars 0 forks source link

Яндекс.Практикум:Код-ревью #3

Open zlocate opened 4 years ago

zlocate commented 4 years ago

Здравствуйте. Благодарю за проделанную работу.

Отлично: Отлино Все ветки присутствуют в репозитории. Отлично Соблюдена версионность в проекте.

К сожалению текущая реализация содержит ряд недостатков, не позволяющих мне засчитать работу на данной итерации:

Репозиторий: Можно лучше файлы logo.svg и cover.jpg, расположенные в репозитории не используются на страничке (загружаются с серверов Яндекса). Просьба либо задействовать их либо удалить

Нужно исправить: README файл должен содержать номер версии (ненулевой), ссылку на проект (github pages) и заголовок отражающий назначение проекта. В дополнение к ссылкам из практики рекомендую почитать статью "Как написать хороший README?". Ссылка на статью:

https://medium.com/astrolabe/%D0%BA%D0%B0%D0%BA-%D0%BD%D0%B0%D0%BF%D0%B8%D1%81%D0%B0%D1%82%D1%8C-%D1%85%D0%BE%D1%80%D0%BE%D1%88%D0%B8%D0%B9-readme-159f88076b26

Примеры шаблонов для README:

Можно лучше: .gitignore должен содержать правила, исключающие попадание временных и\или системных файлов в репозиторий.

Регулярные выражения: Нужно исправить: каждое регулярное выражение должно начинаться со знака ^ и заканчиваться знаком $, чтобы предотвратить частичное совпадение вводимого значения и пропускать ввод только при полном совпадении с регулярным выражением. Данные замечания предназначены для регулярных выражений, которым уже проставили в начале квантификатор ^, а в конце $:

Имя: Нужно исправить не проходят валидацию имена, начинающиеся с буквы Ё. Например: Ёлка

Email: Нужно исправить не проходят валидацию следующие корректные адреса: 2@2domains.ru, 2@2domains.ru - цифры и - могут содержаться в имени пользователя и домене. something@ya.academy - не проходят валидацию адреса с длинной доменной зоной. Я рекомендую убрать ограничение на длину доменной зоны сверху. Информация о данной домененной зоне: https://www.reg.ru/domain/new/ACADEMY/

Нужно исправить: некорректные адрес проходят валидацию _@ya.ru - имя пользователя не может состоять только из одного спецсимвола.

Номер телефона: Нужно исправить:Данные форматы номеров телефонов проходят валидацию, хотя с точки зрения постановки задачи они являются невалидными.

+7(925 900-90-90 +7925900-90-90 +792590090-90 +7925900909-0 - +7-925-900-90-90 +7 (925) 900-90-90 +7 (925) 9009090 +7(925) 9009090 +7 (925)9009090 +7(925)9009090 +7 925 900-90-90

Адрес сайта: Регулярное выражение содержит ошибку (отображается в консоли браузера), из-за чего валидация не работает

Pattern attribute value ^((https?:\/\/)?)(www\.)?(([\da-z\.-]+)\.([a-z\.]{2,6})([\/\w \.-]*)*|(([0-9]{1,3}[\.]){4}))\#?$ is not a valid regular expression: Uncaught SyntaxError: Invalid regular expression: /^((https?:\/\/)?)(www\.)?(([\da-z\.-]+)\.([a-z\.]{2,6})([\/\w \.-]*)*|(([0-9]{1,3}[\.]){4}))\#?$/: Invalid escape

Обратите внимание на экранирование спецсимволов

zlocate commented 4 years ago

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

zlocate commented 4 years ago

Чтобы минимизировать число ошибок разметки рекомендую использовать вкладку Preview или https://stackedit.io/ :) Также если вы создали коммит, а потом поняли что хотите дополнить/изменить можно воспользоваться amend https://git-scm.com/book/ru/v2/%D0%98%D0%BD%D1%81%D1%82%D1%80%D1%83%D0%BC%D0%B5%D0%BD%D1%82%D1%8B-Git-%D0%98%D1%81%D0%BF%D1%80%D0%B0%D0%B2%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5-%D0%B8%D1%81%D1%82%D0%BE%D1%80%D0%B8%D0%B8

zlocate commented 4 years ago

Нужно исправить второе имя также может начинаться с буквы Ё (сейчас данном случае имя не проходит валидацию :( ) Пример: Яков-Ёлка

Почта: Нужно исправить: валидацию проходят адреса электронной почты начинающиеся и/или заканчивающиеся пробелами image Нужно исправить валидацию не проходят адреса электронной почты, доменное имя которых состоит из одного символа. Пример: some@x.com (кстати домен x.com принадлежит Илону Маску :) https://twitter.com/elonmusk/status/885776126148083712 Нужно исправить валидацию проходят адреса с кирилицей в доменном имени. Пример: some@кирилица.ru Нужно исправить валидацию проходят адреса c несколькими спецсимволами подряд mysit.........e123@gmail.bg. Не может идти вподряд несколько точек. Адрес сайта: Нужно исправить валидацию не проходят адреса с однобуквенными доменными именами https://x.com Нужно исправить валидацию проходят адреса содержищие пробелы на конце https://prnt.sc/ru8o9m Нужно исправить валидацию не проходят коррректные адреса IP+порт https://192.168.1.1:90 Нужно исправить валидацию не проходят коррректные адреса домен+порт https://ya.ru:90 http://ya.ru:10000 Нужно исправить валидацию не проходят коррректные адреса домен+путь Пример: https://192.168.1.1:90 Нужно исправить валидацию проходят некорректные адреса с множеством слешей на конце Пример:https://ya.ru////////////////////// После исправлений не забудьте проверить что кейсы IP+порт+путь, домен+порт+путь отрабатывают корректно Нужно исправить проверку проходят следующие некорректные адреса: http://www.ya. (нет доменной зоны)

zlocate commented 4 years ago

Нужно исправить не проходят валидацию корректные адреса сайтов состоящие из домена и порта и домена, порта и пути Примеры: https://ya.ru:90 https://ya.ru:90/some/path#

Можно лучше some_@ya.ru - имя ящика не может начинаться и заканчиваться на спецсимвол my---mail@site.ru - адрес электронной почы, содержащий несколько символов '-' подряд является некорректным

zlocate commented 4 years ago

Добрый день. Работа принята. Желаю вам успехов в дальнейшем обучении. :)