Skip to content
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

Draft
wants to merge 63 commits into
base: main
Choose a base branch
from

Conversation

Livio74
Copy link
Contributor

@Livio74 Livio74 commented Feb 10, 2024

No description provided.

Copy link
Member

@serenasensini serenasensini left a 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 Show resolved Hide resolved
docs/it/design-pattern.md Outdated Show resolved Hide resolved
docs/it/design-pattern.md Outdated Show resolved Hide resolved
docs/it/design-pattern.md Outdated Show resolved Hide resolved
docs/it/design-pattern.md Outdated Show resolved Hide resolved
docs/it/design-pattern.md Outdated Show resolved Hide resolved
docs/it/design-pattern.md Outdated Show resolved Hide resolved
docs/it/design-pattern.md Outdated Show resolved Hide resolved
- 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.
Copy link
Member

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

Copy link
Contributor Author

@Livio74 Livio74 Feb 15, 2024

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

Copy link
Contributor Author

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

docs/it/design-pattern.md Outdated Show resolved Hide resolved
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
@Livio74
Copy link
Contributor Author

Livio74 commented Feb 15, 2024

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! 🙌

Grazie mille , sono molto d'accordo sul tuo feedback e spero che ci siano altre persone che partecipino (o partecipano).
Anche sugli esempi sono d'accordo , anche a me piace molto questo approccio , vorrei capire meglio come farlo.
Il senso è che se faccio singoli esempi sul singolo design pattern e il rischio è un po' l'esplosione del capitolo. Mi piacerebbe chiacchierare con voi @Cadienvan , @serenasensini e chi vuole.

Livio74 and others added 7 commits February 15, 2024 17:18
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
@Cadienvan
Copy link
Member

eh, sugli esempi non so, effettivamente rischiamo di farli andare lunghi..
@serenasensini hai suggerimenti per non farlo diventare un capitolo composto al 99% da esempi di codice?

@serenasensini
Copy link
Member

eh, sugli esempi non so, effettivamente rischiamo di farli andare lunghi.. @serenasensini hai suggerimenti per non farlo diventare un capitolo composto al 99% da esempi di codice?

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?

@Cadienvan
Copy link
Member

Mi piace!

@Livio74
Copy link
Contributor Author

Livio74 commented Feb 25, 2024

sono d'accordo , magari facciamo un po' di tentativi e vediamo come va?

Copy link
Member

@Cadienvan Cadienvan left a 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!

docs/it/design-pattern.md Outdated Show resolved Hide resolved
Livio74 and others added 2 commits March 10, 2024 21:15
…egati

Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Copy link
Member

@Cadienvan Cadienvan left a 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!

Cadienvan and others added 2 commits June 25, 2024 21:30
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
Co-authored-by: Serena Sensini <serena.sensini@gmail.com>
@Cadienvan
Copy link
Member

@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!

@Livio74
Copy link
Contributor Author

Livio74 commented Jun 26, 2024

Ciao @serenasensini e @Cadienvan,
sono d'accordo sul fatto di linkare il sito di Refactoring Guru per gli esempi e per la introduzione, nulla ci vieta poi di introdurre alcuni in un secondo tempo.

@Cadienvan Cadienvan marked this pull request as draft July 9, 2024 11:54
@Cadienvan
Copy link
Member

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!

@Livio74
Copy link
Contributor Author

Livio74 commented Aug 25, 2024

Ciao @serenasensini e @Cadienvan,
questo fine settimana ho fatto qualche commit concordato con @Cadienvan sul mio repository.
Riporto un commento di @serenasensini per decidere insieme come comportarci

@Livio74
Copy link
Contributor Author

Livio74 commented Aug 25, 2024

@serenasensini il 24/02 diceva:
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?

@Livio74
Copy link
Contributor Author

Livio74 commented Aug 25, 2024

in riferimento ai due commenti sopra pensavo:
Sono d'accordo sul continuare l'idea di linkare il sito di Refactoring Guru.
L'ultimo commit è una impostazione iniziale di questa cosa.
Stavo pensando anche di aggiungere un link per ogni Design pattern con una descrizione dell'esempio illustrato.
Cosa ne pensate?

@Livio74
Copy link
Contributor Author

Livio74 commented Aug 25, 2024

Ad ogni modo facciamo sempre in tempo a fare meglio magari vediamo cosa ne pensa anche la community

docs/it/design-pattern.md Outdated Show resolved Hide resolved
docs/it/design-pattern.md Outdated Show resolved Hide resolved
@Cadienvan
Copy link
Member

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 <Cadienvan@users.noreply.github.com>
@Livio74
Copy link
Contributor Author

Livio74 commented Sep 6, 2024

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.

@Cadienvan Cadienvan changed the base branch from add/design-patterns to main September 7, 2024 16:01
@Cadienvan Cadienvan linked an issue Sep 7, 2024 that may be closed by this pull request
@Cadienvan
Copy link
Member

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[🆕]: Design Patterns
3 participants