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.
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.
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
Seconda revisione capitolo Design Patterns #213
base: main
Are you sure you want to change the base?
Seconda revisione capitolo Design Patterns #213
Changes from 52 commits
558c4e9
06756f0
ec08192
3a97905
3e0c694
55712b4
1018d73
0004e20
7514232
f4e1515
4df0fda
2ff61e6
15abf4b
03e4e46
d168957
4b70784
ce6ba33
ff72a87
54135b1
500655b
6288a7f
e0cc1b2
309181c
83649af
32a1056
3162d4a
9b685e4
cbf6bfa
9167432
e357818
3311775
e7d10a8
757b221
5066b74
e012542
2a4628a
a128f06
53f7283
03e6c62
144f366
4549c22
64c716c
e724574
d5c61a5
c3c1020
79157ec
bc98a90
51f06c7
fd7fda2
5734606
2c21e2d
7ee5a44
5640cff
15b0649
ae547b3
06d7b0a
d20784e
ec3e331
5ca6ae6
f10e0a6
48b128e
f54f474
f82c60c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Questa frase, seppur sensata, non mi convince. Non so se sia per il fatto che, a tutti gli effetti, è un riassunto di quanto sopra, ma non mi convince del tutto.. Cioè più che altro direi che come molti altri aspetti della programmazione, anche i design pattern sono una best practise, e non una imposizione..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Frase non chiara. Cosa intendevi?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
provo a spiegarlo meglio , magari puoi darmi anche qualche spunto.
la riga 40,41 indica che non è obbligatorio applicarli ma generalmente se vengono applicati la nostra applicazione aderirà meglio ai principi SOLID ad esempio al Principio di singola responsabilità e quindi una classe avrà una sola responsabilità sul cosa fa oppure al Principio aperto/chiuso e la nostra classe sarà aperta per essere estensa, ma chiusa alle modifiche proprio per modificare il comportamento del software.
La riga 43 indica possiamo suddividere la parte che ci interessa dove identifichiamo un pattern in client , cioè quella parte che interagisce con quella che ci interessa e il pattern vero e proprio che riusciamo ad identificare con le classi che lo compongono eccetto appunto il client che viene considerato ma è esterno. In generale si cerca di non modificarlo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Questa non l'ho capita nemmeno io, neanche dopo la spiegazione ahahahah intendi che c'è una separazione tra l'applicazione del pattern e il modo con cui vi si interagisce? Se sì, credo forse che sia una finezza non necessaria, seppur vera. Nel caso la si voglia mantenere, la frase va rivista ma non saprei come..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E' proprio una separazione logica strutturale , prendete ad esempio la spiegazione del pattern builder su Refactoring guru (https://refactoring.guru/design-patterns/builder) , andate sul capitolo structure la classe denominata client è la parte "esterna" che con le 4 righe sopra non dovrebbe subire cambiamenti. Il client è la parte che interagisce col nostro pattern. Spero di essere stato più chiaro
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Continua a sembrarmi una frase molto contorta.. Per spiegarne il significato sarebbe necessario andare troppo nel dettaglio, personalmente farei che toglierla.. @serenasensini ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concordo con Michael. Forse la toglierei, per non aggiungere complessità ad un argomento già abbastanza astratto!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Non ho capito benissimo la frase, puoi rispiegarla?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probabilmente l'idea era creare uno schema o prenderne uno già fatto e farci riferimento per spiegare meglio ma poi mi sono dimenticato , Non so se per ora meglio toglierlo e rimetterlo una volta aggiunto questo schema
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Frase non necessaria, in quanto già specificato
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Questi qui sotto sono appunti sul come fare il capitolo , una volta completato vanno rimossi.
Non so se c'è un modo migliore per associarli al file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in effetti la riga 125 l'avevo già esplicitata in una frase , potremmo toglierla