search

nofaceinbook / muxp

Mesh Updater X-Plane
GNU Lesser General Public License v3.0
18 stars 3 forks source link

muxp_file module functions could be implemented as a class #26

Open marangonico opened 4 years ago

marangonico commented 4 years ago

From what I see, the muxp_file could be re-written as a class.

nofaceinbook commented 4 years ago

Yes, but why?

marangonico commented 4 years ago

It's subjective, I know, but I tend to see the config file as an object, and so I'm prone to gather the related methods (read, parse, validate, write, etc) into a class. This can lead to many benefits IMHO. But this is not an exact science.. it's also a matter of personal taste.

nofaceinbook commented 4 years ago

Understood. I will assign nice2have label for such issues, as in my view we have more urgent topics to solve.