Skip to content

Conversation

@lord-haffi
Copy link
Collaborator

No description provided.

@lord-haffi lord-haffi self-assigned this Sep 15, 2023
@lord-haffi lord-haffi added enhancement New feature or request good first issue Good for newcomers labels Sep 15, 2023
Copy link
Contributor

@hf-kklein hf-kklein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich glaube ich habe nur die Hälfte verstanden aber es wirkt überzeugend :)

@hf-krechan
Copy link

Wofür ist dieses Repo gut?
Mir ist der Generics-Ansatz auch noch nicht ganz klar, was hier die Benefits und was es besser macht als Protocols?

@hf-kklein
Copy link
Contributor

was hier die Benefits

Angenommen du hast ein Projekt X, das mit Types aus Projekt Y umgehen soll, wobei zwar Y das Projekt X kennt aber nicht anders rum. Im Code von X sollen trotzdem Informationen über die Typen aus Y verwendet werden können. Dann brauchst du dieses Repo. (Konkret für uns ist X=bomf).

besser als Protocol

es ist weder besser noch schlechter. einfach was anderes.

@lord-haffi
Copy link
Collaborator Author

Wofür ist dieses Repo gut? Mir ist der Generics-Ansatz auch noch nicht ganz klar, was hier die Benefits und was es besser macht als Protocols?

Im anderen PR hab ich ein gutes Beispiel in die README gepackt, schau dir das am besten mal an. Erinnerst du dich noch, wie du im Wim.Bee damals der CRUD-Base Klasse die Model-Klasse zusätzlich im Konstruktor übergeben musstest, weil du die nicht aus dem Generic rausbekommen hast? Mit diesem Repo kannst du es aus den Generics saugen :)

@lord-haffi lord-haffi merged commit 65495d3 into main Sep 16, 2023
@lord-haffi lord-haffi deleted the basic_implementation branch September 16, 2023 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants