You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Admittedly, Solve methods are all quite small, but it appears that some of them duplicate code in others, whereas they can be just calling the overload which code it's duplicating.
Ideally only a single overload should call Search (and BuildInternalStructure) and the reset should call this overload (possibly via another overload call, in a chain).
Overall I salute you for solution meticulously crafted, very well done, but this thing stood out for me, may be there is a reason for this, that I failed to see?
The text was updated successfully, but these errors were encountered:
Admittedly,
Solve
methods are all quite small, but it appears that some of them duplicate code in others, whereas they can be just calling the overload which code it's duplicating.Ideally only a single overload should call
Search
(andBuildInternalStructure
) and the reset should call this overload (possibly via another overload call, in a chain).Overall I salute you for solution meticulously crafted, very well done, but this thing stood out for me, may be there is a reason for this, that I failed to see?
The text was updated successfully, but these errors were encountered: