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.

Sign up for GitHub

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

Jump to bottom

Verständlichkeit erhöht in GVTAN2Step und HBCIJobImpl #66

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Verständlichkeit erhöht in GVTAN2Step und HBCIJobImpl #66

wants to merge 15 commits into from

Conversation

ruderphilipp
Copy link
Contributor

Im Zuge meiner Einarbeitung in den Code sind mir einige Dinge aufgefallen. Dieser Pullrequest konzentriert sich auf die Klasse GVTAN2Step und die folgenden Anpassungen daran:

  • Override-Annotation
  • Überarbeitung der Javadoc-Texte
  • Lesbarkeit/ Verständlichkeit der Methode extractResults

Zusätzlich wurden in der Klasse HBCIJobImpl folgende Anpassungen vorgenommen:

  • Sternchen sodass Kommentar zu Javadoc wird (und somit von IDE angezeigt wird)
  • Zeilenumbruch bei Parameterliste einer Methode entfernt
  • Leerzeilen entfernt

@ruderphilipp
ergänze Override und entferne dadurch redundanten JavaDoc-Text
2213316
@ruderphilipp
JavaDoc korrigiert
0cfeb16
@ruderphilipp
Leerzeilen entfernt
a344ec4
@ruderphilipp
Umbruch in Methodenparametern entfernt
7c05502
@ruderphilipp
konsistenter Codestil: this ergänzt
917ae47 
In der Klasse wird in allen Methoden explizit unterschieden, ob es
sich um eine lokale Variable handelt oder um eine, die der Variable
der Objektinstanz (mit `this`). Außer an den zwei Stellen.
@ruderphilipp
statischer Import von KnownReturncode
c016e20
@ruderphilipp
statischer Import von KnownTANProcess
ea51905
@ruderphilipp
Hilfsmethode KnownReturncode::isIn ergänzt
2f5292e
@ruderphilipp
statischer Import von StringUtil.toInsCode()
f9b9e24
@ruderphilipp
x.equals(y) -> y.equals(x)
1c8e5fe 
Zum besseren Verständnis die Konstante (innerhalb der Methode) nach
vorn gestellt, statt sie weit hinten im Code zu "verstecken".
@ruderphilipp
JavaDoc überarbeitet
7129aa3
@ruderphilipp
JavaDoc: logisch zusammengehörigen Bereich als region markiert
691db2d
@ruderphilipp
Konstanten für Status-Objekte aus HBCIMsgStatus für bessere Lesbarkeit
6e347b6
@ruderphilipp
Zeilenumbruch für bessere Lesbarkeit
9c98add
@ruderphilipp
JavaDoc: Prozess-X --> Prozessx
8c9a351
@ruderphilipp ruderphilipp changed the title Refactoring: Verständlichkeit erhöht in GVTAN2Step und HBCIJobImpl Verständlichkeit erhöht in GVTAN2Step und HBCIJobImpl Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ruderphilipp