aimclub / aimclub/BAMT

Refactoring of string-based identifiers

Offen
#33 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
refactoring
Vorherrschende Sprache
Python
Sterne
134
Forks
23
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

![image](https://user-images.githubusercontent.com/5711814/218581526-83fe96a2-c0f1-44f8-ae64-5ce90f389e5d.png)
![image](https://user-images.githubusercontent.com/5711814/218581856-6d040660-8d3b-4be9-94dd-b727414d41a2.png)
![image](https://user-images.githubusercontent.com/5711814/218581344-529f4650-f764-4f04-9451-d3c8974421ab.png)

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

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

1) ту часть "категориальных" идентификаторов, которая не видна конечному пользователю, заменить на Enum-ы;
2) или хотя бы заменить все эти строковые идентификатора на константы.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.