Propozycja zmiany nazw wartości typu "place"

Qziem / eloapp

Aplikacja do wyznaczania rankingów. Napisana w ReasonML i PHP

4 stars 1 forks source link

Open jazithedev opened 5 years ago

jazithedev commented 5 years ago

W chwili obecnej mamy tak:

type place =
  | RANK_AND_STATS
  | OPERATIONS
  | STATISTICS
  | NOT_FOUND;

Proponuję tak:

type place =
  | PLACE_RANK_AND_STATS
  | PLACE_OPERATIONS
  | PLACE_STATISTICS
  | PLACE_NOT_FOUND;

Wartości te będą o wiele lepiej rozróżnialne w kodzie aplikacji. Wiążę się z tym oczywiście refaktor. Co o tym sądzisz, @kamilkuz?

Qziem commented 5 years ago

Jak najbardziej. Myślę że dobry pomysł.