Kemsekov / GraphSharp

GraphSharp is a tool to study and use knowledge of graph theory in C#!
MIT License
37 stars 5 forks source link

Divide GraphStructure to functional classes #37

Closed Kemsekov closed 2 years ago

Kemsekov commented 2 years ago

Your current impl of GraphStructure require a lot of lambda-configurations for it to work properly, meanwhile a lot of them may be not used in some cases of creating / altering nodes. This mean you break SRP in your GraphStructure instance. Fix it by dividing this big classes GraphStructure to separate more contained classes.

Kemsekov commented 2 years ago

I decided do not bother myself with this for now. I changed configuration of graph and refactored namespaces and location of functions and currently it seems pretty much fine for me