Замечания по URLParser

MorlakDzuu / OOP_Labs

OOP labs

0 stars 0 forks source link

Замечания по URLParser #9

Open alexey-malov opened 5 years ago

alexey-malov commented 5 years ago

[x] Класс URL не моделирует сущность "URL".

alexey-malov commented 5 years ago

[x] Из метод разбора URL-а выделить методы для разбора отдельных его частей

alexey-malov commented 5 years ago

[x] Класс URL нарушает принцип единой ответственности. Если поменять формат вывода информации об URL-е, то его тесты придётся переписать

alexey-malov commented 5 years ago

[x] Неверно парсятся URL-ы без слэша в имени домена. Нет тестов на проверку данной ситуации
[ ] Не обрабатываются ограничения на диапазон значений номера порта [1, 65535]. Нет тестов на проверку данной ситуации

alexey-malov commented 5 years ago

[x] Следующий URL ошибочно счиатется правильным: http://:1/1.txt

alexey-malov commented 5 years ago

[ ] Сделать так, чтобы для извлечения нескольких характеристик URL-а нужно было вызывать один метод:

URLComponents components = URLParser.parseURL("http://google.com:435/search");
System.out.println(components.host);
System.out.println(components.scheme);
System.out.println(components.document);
System.out.println(components.port);

alexey-malov commented 5 years ago

[ ] Допустимые значения номера порта - от 1 до 65535. Покрыть тестами. Сейчас 0 порт программа считает валидным