srb2thepast / osu-alltrick

A trainer for every skill in the rhythm game osu!
MIT License
12 stars 0 forks source link

Restructure skill calculation system with inheritance in mind #14

Open srb2thepast opened 1 year ago

srb2thepast commented 1 year ago

There are have been instances where two skills end up having very similar calculation variables (ex. Tapping Stamina and Flow Aim for Strain-based Stream Length and all aim skill for Aim Difficulty).

Preferably there should be no need to rewrite the whole calculation set all over again. One of the ways to achieve that would be to have those similar methods either: A: Inherited or B: Used in a similar fashion to the Shared Methods class.

Documentation should also reflect this.