hbci4java icon indicating copy to clipboard operation
hbci4java copied to clipboard

DialogContext -- Konstruktor statt statische Methode für neue Instanz

Open ruderphilipp opened this issue 4 years ago • 0 comments

Im Zuge meiner Einarbeitung in den Code sind mir einige Dinge aufgefallen. Dieser Pullrequest konzentriert sich auf die Methode DialogContext.create.

Laut Historie in Git kam die Klasse DialogContext mit Commit 500f0753218a136bec10ddee9ba47c2e52473a3d und hatte dort bereits die statische Methode create sowie einen privaten Konstruktur ohne Parameter, der dadurch gekapselt wird. Jedoch ruft create ausschließlich der Konstruktur auf und setzt die übergebenen Parameter in der neu erzeugten Objektinstanz. Es existiert keine zusätzliche Logik (z.B. Singleton o.ä.), wodurch diese Indirektion notwendig wäre.

Aus diesem Grund schlage ich hiermit vor, stattdessen einen regulären Konstruktur zu verwenden.

ruderphilipp avatar Jan 22 '22 17:01 ruderphilipp