bridgecommand / bc

Bridge Command ship simulator
https://www.bridgecommand.co.uk
Other
56 stars 24 forks source link

Housekeeping #30

Closed m6c7l closed 3 years ago

m6c7l commented 3 years ago
bridgecommand commented 3 years ago

Thanks. I'll look at this over the coming week.

dpservis commented 3 years ago

Hi everyone,

I have a private repo of BridgeCommand and I have completely restructured the code by completely separating the business from visualization (there's somewhere a mention to MVC but really there's no MVC there...). One of the reasons that I attempted this was that I wanted to separate my own ship code and unit test it, the same for other classes. So I have several smaller projects now: OwnShip, Scene (everything GUI related), World, Assets, and Initializers (everything needed to initialize business from files). I have also isolated irr and removed the usage of irr typed in business so that it is not needed when testing and it is also possible to replace it eventually with a live library. Would such a structure be interesting for you?

m6c7l commented 3 years ago

Hi there,

I think MVC can definitely help to make things easier from several points of view in the future in the BC project... I agree that there is nothing like this nor another pattern implemented in the codebase - at least not as obvious as it would be helpful in some way. So yes, I think it would be a good step in the right direction to have that kind of separation and guiding pattern in the codebase.

As a first step, could you somehow grant access to your repo for an initial investigation about the changes you made in particular?

Am Freitag, dem 22.01.2021 um 13:35 -0800 schrieb dpservis:

Hi everyone, I have a private repo of BridgeCommand and I have completely restructured the code by completely separating the business from visualization (there's somewhere a mention to MVC but really there's no MVC there...). One of the reasons that I attempted this was that I wanted to separate my own ship code and unit test it, the same for other classes. So I have several smaller projects now: OwnShip, Scene (everything GUI related), World, Assets, and Initializers (everything needed to initialize business from files). I have also isolated irr and removed the usage of irr typed in business so that it is not needed when testing and it is also possible to replace it eventually with a live library. Would such a structure be interesting for you? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

bridgecommand commented 3 years ago

Yes, I would be interested in being able to see your changes, thanks.

dpservis commented 3 years ago

Hi,

thanks for the answer! Since there is interest in this, I will clean up my repo and make it public.

Kind regards

Dimitris