zaretskaya/directedgraph

directedgraph's Issues

Замечания

1. Почему вес дуги Вы проверяете на не равно 0, а не на больше 0?
2. Если Вы переопределили equals, то нужно переопределить hashCode.
3. Конструкцию if (x) return true; else return false; можно упростить до return x;
4. Не знаю, зачем Вам понадобилось делать полем класса keys, но в любом случае, нет необходимости его обновлять при изменении Map, потому что keySet() возвращает множество-отображение, которое меняется вместе с исходной Map.
5. С точки зрения пользователя класс Edge выглядит странно: вес и исходящая вершина. Тем более в методе, который возвращает все исходящие вершины: там нельзя понять, куда они идут.
6. Не стоит возвращать коллекции, хранящиеся в полях класса, лучше сделать копию.
7. Методам hasEdge и deleteEdge логичнее принимать начальную и конечную вершины, а не дугу и вершину (вес дуги указывать незачем).
8. В addEdge незачем перекладывать список Map.
9. Вместо итерации по keys лучше использовать итерацию сразу по entrySet() (там, где потом всё равно берется значение по ключу).
10. В deleteVertex во-первых, тоже не нудно перекладывать список в Map, во-вторых, Вы пытаетесь удалить вместо найденной дуги имя вершины, а в третьих, вообще пытаетесь удалять из списка по которому итерируетесь в цикле.
11. В deleteEdge Вы зачем-то сделали цикл по ключам, вместо того, чтобы просто получить список по ключу.
12. Ваши тесты должны использовать assert... методы, а не кидать исключения.