Skip to main content
Code Review

Return to Answer

deleted 14 characters in body
Source Link
Jamal
Jamal
  • 34.9k
  • 13
  • 133
  • 237

The method:Optimize Optimize isn't working fine.It It doesn't make sense to connect two randomly selected points and then connect other points by these two's IncommingConnectionIncommingConnection.

The method:Optimize isn't working fine.It doesn't make sense to connect two randomly selected points and then connect other points by these two's IncommingConnection.

The method Optimize isn't working fine. It doesn't make sense to connect two randomly selected points and then connect other points by these two's IncommingConnection.

deleted 149 characters in body
Source Link
Stanislav Chankov
Stanislav Chankov
  • 11
  • 2

CreateNewRouteTo and InsertIntoExistingRoute aren't The method:Optimize isn't working fine, because they add Incoming and Outgoing connections which isn't right.

The IncomingIt doesn't make sense to connect two randomly selected points and Outgoing connections which addthen connect other points by these 2 methods break all the logic whentwo's GetCostOfWholeRoute method is trying to calculate the cost of the whole routeIncommingConnection.

CreateNewRouteTo and InsertIntoExistingRoute aren't working fine, because they add Incoming and Outgoing connections which isn't right.

The Incoming and Outgoing connections which add these 2 methods break all the logic when GetCostOfWholeRoute method is trying to calculate the cost of the whole route.

The method:Optimize isn't working fine.It doesn't make sense to connect two randomly selected points and then connect other points by these two's IncommingConnection.

added 23 characters in body
Source Link
Mast ♦
Mast ♦
  • 13.5k
  • 12
  • 54
  • 119

CreateNewRouteToCreateNewRouteTo and InsertIntoExistingRoute isn'tInsertIntoExistingRoute aren't working fine,because because they adds Incomingadd Incoming and OutgoingOutgoing connections which isn't right.The Incoming

The Incoming and Outgoing connectionOutgoing connections which addsadd these 2 methods break all the logic when GetCostOfWholeRouteGetCostOfWholeRoute method is trying to calculate the cost of the whole route.

CreateNewRouteTo and InsertIntoExistingRoute isn't working fine,because they adds Incoming and Outgoing connections which isn't right.The Incoming and Outgoing connection which adds these 2 methods break all the logic when GetCostOfWholeRoute method is trying to calculate the cost of whole route.

CreateNewRouteTo and InsertIntoExistingRoute aren't working fine, because they add Incoming and Outgoing connections which isn't right.

The Incoming and Outgoing connections which add these 2 methods break all the logic when GetCostOfWholeRoute method is trying to calculate the cost of the whole route.

Source Link
Stanislav Chankov
Stanislav Chankov
  • 11
  • 2