Refactoring: int -> enum in `HBCICallback`

hbci4j / hbci4java

Java-based FinTS protocol implementation that supports all features (chipTAN, pushTAN, HHD, SEPA, PSD2,...)

GNU Lesser General Public License v2.1

147 stars 49 forks source link

Refactoring: int -> enum in `HBCICallback` #60

Closed ruderphilipp closed 2 years ago

ruderphilipp commented 2 years ago

Erhöhe Typsicherheit, indem die Konstanten fachlich in Enums gebündelt werden. Dadurch muss keine Prüfung auf fehlerhafte Eingabewerte (< 0, invalider Wert, etc.) erfolgen und für den Entwickler ist besser ersichtlich, was als Parameter erwartet wird (--> Autovervollständigung der IDE).

Innerhalb der aufrufenden Klassen wird der Code dadurch ebenfalls sprechender/ besser verständlich.

willuhn commented 2 years ago

Das kann ich nicht übernehmen. Du zerbrichst damit alle hbci4java-Integrationen. Hibiscus ist nur ein Programm, welches hbci4java nutzt. Daneben gibt es noch etliche weitere - oft auch Eigenentwicklungen, die von Unternehmen intern verwendet werden. Bitte erstelle keine reinen Refactoring-PRs. Die damit verbundenen Aufwände für alle anderen stehen oft in keinem Verhältnis zum Nutzen.