Refactor `Sat` - Githubissues

agathazeren / cuneiforbits

𒆩𒀩𒂷𒂷𒂷𒀖

1 stars 1 forks source link

Open agathazeren opened 4 years ago

agathazeren commented 4 years ago

There are a couple problems with the current implementation

enum Sat {
    CubeSat(CubeSat),
    LargeSat(LargeSat),
    ArraySat(ArraySat),
    Station(Station),
}

It feels nonuniform, and it probably can be refactored into a more unified implementation.
Also, there is a question on whether the variants should have Sat in their names.

agathazeren commented 4 years ago

I really am feeling the clunkiness of the current implementation. Refactoring this will definitely make our lives easier.