red-life-project / red-life

A simple game about surviving on mars
GNU General Public License v3.0
18 stars 1 forks source link

Replace Event name with EventMessage #275

Closed Eraden closed 1 year ago

Eraden commented 1 year ago

In Event#action we have match which uses string slice as a value. As far as this string slice should always be taken from translations this still can be error-prone area which will be hard to debug and is really sub-effective solution.

Number of messages is really limited and there's no space for "custom" one so every single one must be implemented and hard-coded.

I suggest to change it to enum so match will be much simpler and number of lookup for translations will be greatly reduced