book icon indicating copy to clipboard operation
book copied to clipboard

Seconda revisione capitolo Design Patterns

Open Livio74 opened this issue 1 year ago • 20 comments

Livio74 avatar Feb 10 '24 16:02 Livio74

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 avatar Feb 15 '24 14:02 Livio74

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?

Cadienvan avatar Feb 17 '24 10:02 Cadienvan

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?

serenasensini avatar Feb 24 '24 17:02 serenasensini

Mi piace!

Cadienvan avatar Feb 24 '24 18:02 Cadienvan

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

Livio74 avatar Feb 25 '24 18:02 Livio74

Ciao @Livio74 , ho visto un bel po' di movimento sul capitolo in questi giorni, nice job!

Fammi sapere quando reputi sia pronto per una "seconda occhiata" così ci dedico il giusto tempo!

Cadienvan avatar Mar 17 '24 15:03 Cadienvan

Ciao @Cadienvan , ciao @serenasensini ho committato un po' di proposte di Serena , ci sono poi alcuni messaggi in risposta riferiti sempre ai dubbi di Serena Se volete , comunque sia , chiudo la pr e la rifaccio ma quelli che non ho committato ritornerebbero. Aspetto vostre News. Buona serata Ciao Livio

Livio74 avatar Mar 21 '24 16:03 Livio74

@serenasensini , @Cadienvan , è il caso secondo voi di chiudere la pull request e revisionarla in base anche alle vostre segnalazioni o continuiamo con l'interazione su questa pull?

Livio74 avatar Mar 26 '24 17:03 Livio74

Ciao @Livio74 , io continuerei qui sopra, non avrebbe senso ripartire, la maggior parte dei punti sono comunque già stati chiusi o in fase di chiusura!

Cadienvan avatar Mar 27 '24 20:03 Cadienvan

Ciao @serenasensini , ciao @Cadienvan, ho alcuni punti in sospeso in cui aspettavo vostri feedback, come ci muoviamo? magari se vi va potremmo guardarci insieme così poi ognuno va avanti per suo conto (io in particolare)

Grazie Livio

Livio74 avatar Apr 23 '24 14:04 Livio74

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

Cadienvan avatar Jun 25 '24 19:06 Cadienvan

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.

Livio74 avatar Jun 26 '24 16:06 Livio74

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!

Cadienvan avatar Jul 09 '24 11:07 Cadienvan

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 avatar Aug 25 '24 14:08 Livio74

@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 avatar Aug 25 '24 14:08 Livio74

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 avatar Aug 25 '24 14:08 Livio74

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

Livio74 avatar Aug 25 '24 14:08 Livio74

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?

Cadienvan avatar Sep 05 '24 19:09 Cadienvan

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.

Livio74 avatar Sep 06 '24 13:09 Livio74

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

Cadienvan avatar Sep 16 '24 19:09 Cadienvan