-
-
Notifications
You must be signed in to change notification settings - Fork 25
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?
Conversation
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.
Si tratta di un capitolo sicuramente tosto in termini di tematiche, dove l'approccio da seguire non è semplice; ci sono diverse parti da rivedere per aggiungere chiarezza e, inoltre, suggerirei di aggiungere anche degli esempi per rendere più evidente l'applicazione di tali pattern, rispetto a renderlo un elenco catalogato di pattern.
In ogni caso, è un lavorone e va premiato l'impegno messo per assemblare questo capitolo! 🙌
docs/it/design-pattern.md
Outdated
- Spingere di più sul concetto che i Design Pattern siano delle soluzioni _tipo_ a problemi _tipo_ ma che non sempre la situazione sarà così lineare da permetterci in primis di comprendere il problema che stiamo affrontando e inoltre non è detto che la soluzione sia ottimale in termini di, ma non solo: performance, tempi di sviluppo, armonia con altre scelte architetturali, esperienza del team. | ||
- In linea con quanto sopra: Non tutti i dev di un team potrebbero essere in grado di adottare i design pattern nella maniera corretta e, come ogni standard, se non rispettato può portare più problemi che benefici. | ||
- Importante delineare il fatto che spesso i Design Pattern, come altri concetti e best practise del software, siano astrazioni applicative e molti dei concetti rappresentati dai pattern possano essere ritrovati sia a livelli di astrazione più alti (Architettura e addirittura Infrastruttura) o più bassi (Singole righe di codice o singole funzioni). | ||
- (Già citato in una frase) Come linea generale eviteremo di listare i design pattern in maniera accademica in quanto il libro sarà un incipit a un futuro approfondimento, ma ci concentreremo sui perché della loro nascita e del loro utilizzo, le principali dinamiche e i principali concetti attorno ad essi, e i pro e i contro generali. |
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
Co-authored-by: Serena Sensini <[email protected]>
Grazie mille , sono molto d'accordo sul tuo feedback e spero che ci siano altre persone che partecipino (o partecipano). |
Co-authored-by: Serena Sensini <[email protected]>
Co-authored-by: Serena Sensini <[email protected]>
Co-authored-by: Serena Sensini <[email protected]>
Co-authored-by: Serena Sensini <[email protected]>
Co-authored-by: Serena Sensini <[email protected]>
Co-authored-by: Serena Sensini <[email protected]>
Co-authored-by: Serena Sensini <[email protected]>
eh, sugli esempi non so, effettivamente rischiamo di farli andare lunghi.. |
Direi che la cosa migliore sarebbe estrarre un paio di pattern per tipologia e riportare degli esempi che siano anche astrazioni di alto livello, ma rappresentative del caso d'uso. As d esempio, per la Factory, senza bisogno di codice, si può parlare delle interfacce come generalizzazione di un prodotto: senza forzare idee complesse, basta pensare all'interfaccia Poligono e alle classi che ne derivano. Che ne pensate? |
Mi piace! |
sono d'accordo , magari facciamo un po' di tentativi e vediamo come va? |
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.
Ho proposto una correzione!
…egati Co-authored-by: Serena Sensini <[email protected]>
Co-authored-by: Serena Sensini <[email protected]>
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.
Ciao Livio, ho cominciato a indicare alcuni suggerimenti, questo weekend è stato infernale, settimana prossima chiudo con i punti aperti!
Co-authored-by: Serena Sensini <[email protected]>
Co-authored-by: Serena Sensini <[email protected]>
@Livio74 @serenasensini ho applicato ciò su cui eravamo d'accordo tutti. Credo che gli esempi possano risultare troppo complessi da stendere. Cosa ne pensate se tagliassimo la testa al toro e linkassimo al sito refactoring guru che spiega molto bene con ottimi esempi battle-tested i pattern? Magari proporrei una intro in cui diciamo che suggeriamo di seguire questa guida affiancati da quel sito di modo da seguire lì sopra gli esempi pratici! |
Ciao @serenasensini e @Cadienvan, |
Ciao @Livio74 per il momento ho messo in draft, tempo permettendo togli pure gli elementi che abbiamo detto essere di troppo, aggiungi i dovuti link qua e là e poi quando ci sei mi tagghi e riapriamo la review! |
Ciao @serenasensini e @Cadienvan, |
@serenasensini il 24/02 diceva: |
in riferimento ai due commenti sopra pensavo: |
Ad ogni modo facciamo sempre in tempo a fare meglio magari vediamo cosa ne pensa anche la community |
Secondo me il capitolo ha preso la giusta forma, credo che l'introduzione e i link alle varie categorie di pattern siano più che sufficienti per far capire alle persone che non devono aspettarsi QUI degli esempi ma che possono trovarli sul sito di Guru. Che ne dite? |
Co-authored-by: Michael Di Prisco <[email protected]>
A me piace , poi c'è sempre la community che può migliorarlo (inteso anche noi). Sulle ali dell'entusiasmo ho committato uno delle due review ma c'era il riferimento anche a @serenasensini. Se volete poi lo ripristino. |
@serenasensini se sei d'accordo e non hai altri punti in sospeso io partirei con una doppia approvazione, mia e tua, e lo apriamo alle review degli altri ambassador! |
No description provided.