search

LaurensWeyn / Spark-Reader

A tool to assist non-naitive speakers in reading Japanese
GNU General Public License v3.0
30 stars 7 forks source link

Moving over to MVC #26

Open LaurensWeyn opened 6 years ago

LaurensWeyn commented 6 years ago

So the code is becoming a bit of a mess - the mix of graphics and UI code inside classes that the deconjugator deals with is making things difficult to deal with. Some features, like a headless text analyser, or scrolling through all definitions continuously, can't really work well with how things are now.

So I've been thinking to convert things over to something like MVC. This is a non-trivial rewrite, which is why I'm putting up this issue as a warning of sorts.

Since exams are in full swing where I live, I won't be able to start on this until early December at the soonest.

It would take some work, but I think it would be worth it in the long run. Many of the features often suggested to me would suddenly become a lot easier, code maintenance would be simpler, providing more UI options becomes far simpler and code could be reused more easily for other projects (e.g. a general purpose offline dictionary tool).

wareya commented 6 years ago

I'm not sure whether it should be MVC or something else, but anything to restructure the program to fix the UI code leakage problem would be a huge plus. I'm all for it.