ledgerloops / strategy-pit

Testing ground for LedgerLoop strategies
Apache License 2.0
0 stars 0 forks source link

Inheritence is messing up with various message types #11

Closed michielbdejong closed 5 months ago

michielbdejong commented 5 months ago

The receiveMessage method in base class Node wants to deal with Meet messages but leave other handle___Message functions as abstract.

But now I'm adding new message types for the Butterfly strategy, I need to edit this method in the base class. I guess a clean way to do it is not to use abstract methods for each method types, but just empty methods for unused method types in the base class