Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring of string-based identifiers #33

Open
nicl-nno opened this issue Feb 13, 2023 · 0 comments
Open

Refactoring of string-based identifiers #33

nicl-nno opened this issue Feb 13, 2023 · 0 comments

Comments

@nicl-nno
Copy link

Сейчас большая часть логики завязана на строковые идентификаторы:

image
image
image

Но это порождает многочисленные ситуации, когда есть различия в регистре и возникают .lower() и прочие неэстетичные штуки.

Я бы предложил два варианта:

  1. ту часть "категориальных" идентификаторов, которая не видна конечному пользователю, заменить на Enum-ы;
  2. или хотя бы заменить все эти строковые идентификатора на константы.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants