DialogContext -- Konstruktor statt statische Methode für neue Instanz #65
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ruderphilipp
changed the title
DialogContext): Nutze Konstruktor statt statischer …DialogContext
ruderphilipp
changed the title
DialogContext…Methode, die dann nur K'tor aufruft Laut Historie in Git kam die Klasse `DialogContext` mit Commit 500f075 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.
ruderphilipp
force-pushed
the
refactoring/4
branch
from
8fd3134 to
d8f87a6
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
DialogContextmit Commit 500f075 und hatte dort bereits die statische Methodecreatesowie einen privaten Konstruktur ohne Parameter, der dadurch gekapselt wird. Jedoch ruftcreateausschließ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.