-
Notifications
You must be signed in to change notification settings - Fork 0
SergiiShcherbakov/TestTaskForCompanyX
Folders and files
Name | Name | Last commit message | Last commit date | |
---|---|---|---|---|
Repository files navigation
Все тестовые задания выполнены. Есть следующие комментарии к реализации: Общее замечание: старался делать самодокументирующийся код и писать поменьше комментариев. В результате из названия метода понятно его назначение, хоть и само название получается довольно длинным и хуже воспринимается. Замечания к заданиям Задача 1 - о скобочных выражениях Запуск решения - класс main в пакете ua.com.shcherbakov.sergii.brackets Задача 2 - о минимальной стоимости пути Запуск решения - класс main в пакете ua.com.shcherbakov.sergii.way Вижу, что данный код можно улучшать. Учитывая учебное задание и отсутствие времени, лишь укажу некоторые моменты, в которые бы доработал: -добавить проверку на совпадение названий городов при вводе списка городов -добавить проверку на корректность всех введенных соседей -добавить проверку на совпадение начальной и конечной точки маршрута -вынес бы проверку ввода из класса ConsoleViewer и использовал выброс исключений и обработку в методах -используется интерфейс Viewer а его реализация по тексту программы называется console. С точки зрения данной задачи использование переменной console увеличивает читабельность кода, хотя при расширении програмы вносило бы путаницу. Учитывая, что задача учебная с четко ограниченным способом вывода, выбираю читабельность как более приоритетную. -добавить расчет в фоновом режиме в другом потоке взвешенного графа по алгортму Дейкстры, сразу после ввода данных Задача 3 - о факториалах Запуск решения - класс main в пакете ua.com.shcherbakov.sergii.factorial
About
Тестовое задание для компании Х
Resources
Stars
Watchers
Forks
Releases
No releases published
Packages 0
No packages published