mborne / cours-patron-conception Goto Github PK

Ceci évitera d'avoir à faire des vilains schémas au tableau.

La lecture de deux formats apporte peu dans le TP. Elle complexifie l'exécution du projet et la correction.

Il convient de conserver uniquement ShapeFile

• Fusionner GraphReader, ShpGraphReader et XmlGraphReader en GraphReader
• Faire sauter la question 0.6 en conservant dès le début la géométrie
• Revoir README.md et aide pour supprimer les références au XML (a,b)

Principes

• Ajouter un controlleur d'accès aux données du graphe (edges suffisant?)
• Ajouter un site statique exploitant l'API pour faire des calculs de plus court chemin

Remarques

• ATTENTION à supporter deux formats de réponse (avant et après ajout de Path)
• Pourrait aussi servir de base pour l'auto-correction

http://mborne.github.io/cours-patron-conception/#46

https://www.intertech.com/Downloads/eBook/ApplicationArchitectureGuide.pdf

Contexte

Deux dépôts permettant du gagner du temps sur la correction :

• tp-geometry-check-2023
• tp-graph-check-2023

...mais correction encore trop chronophage ;)

Améliorations possibles

• Faire un seul dépôt privé "tp-java-checker" et des variantes en fonction du TP

But : supprimer les doublons de code entre les deux dépôts + avoir un dépôt dédié aux résultats des corrections des sessions.

• Uniformiser les contrôles (PMD à ajouter pour tp-graph-check)

• Automatiser plus de contrôles

tp-graph : conteneuriser l'exécution avec mvn spring-boot:run et lancer des tests fonctionnels sur l'API (postman tentant mais mocha sera sûrement plus souple)
tp-geometry : batterie de test fonctionnel

• Permettre l'exécution autonome par les étudiants?

A faire en mode service pour pouvoir ajouter des batteries de tests unitaires et fonctionnels sur les geometry (sans que ça se traduise par fournir les tests unitaires et fonctionnels à pomper)

Pré-requis

• tp-geometry : imposer un nommage rigoureux (vérifier qu'il n'y a pas de question amenant un changement de package, nom de classe ou de méthode)
• tp-graph : idem à minima sur l'API (préciser le package pour PathNode et PathTree pour pouvoir tester en profondeur?)

c.f. correctif question "0.15 - Optimisation du temps de chargement du graphe"

A faire pour 2022 sur la branche upgrade en mettant à jour par la suite les branches du corrigé :

GraphBuilder graphBuilder = new GraphBuilder();
FeatureIterator<SimpleFeature> features = null;
try {
    features = collection.features();
    while (features.hasNext()) {
        SimpleFeature feature = features.next();
        //...
    }
}finally {
    if ( features != null ){
        features.close();
    }
    dataStore.dispose();
}
return graphBuilder.getGraph();

Contexte

Beaucoup de "on" dans les fiches...

Patrons de création

• src/annexe/design_pattern/creational/Singleton.md
• src/annexe/design_pattern/creational/Prototype.md
• src/annexe/design_pattern/creational/Factory.md
  • src/annexe/design_pattern/creational/FactoryMethod.md (intégré en tant que variante)
  • src/annexe/design_pattern/creational/AbstractFactory.md (intégré en tant que variante)
• src/annexe/design_pattern/creational/Builder.md

Les patrons de structure

• src/annexe/design_pattern/structural/Facade.md
• src/annexe/design_pattern/structural/Decorator.md
• src/annexe/design_pattern/structural/Composite.md
• src/annexe/design_pattern/structural/Adapter.md
• src/annexe/design_pattern/structural/Bridge.md

Patrons de comportement

• src/annexe/design_pattern/behavior/Observer.md
• src/annexe/design_pattern/behavior/Visitor.md
• src/annexe/design_pattern/behavior/Strategy.md
• src/annexe/design_pattern/behavior/NullObject.md
• src/annexe/design_pattern/behavior/Iterator.md

Les patrons d'architecturaux

• src/annexe/design_pattern/architectural/couches.md
• src/annexe/design_pattern/architectural/MVC.md
• src/annexe/design_pattern/architectural/MapReduce.md

• Ajouter un getCoordinate() sur Point à partir de la question 0.2
• Ajouter asText() sur AbstractGeometry à la question 0.11 et 0.12

• 0.14 - Supprimer l'implémentation de GeometryListener et faire invalider le cache sur translate dans la question précédente
• 0.16 - ajouter ReturningGeometryVisitor sans modifier le reste du code

(faire quelques contrôles de degré sortant/entrant?)