Conversion

1.  Conversion.java
• Certains commentaires nécessaires sont absents.
• Les noms tabMonnaies et tabValeurs sont mal choisis. En règle générale, les identificateurs sont insuffisement significatifs. De plus tabMonnaies et tabValeurs sont des constantes de classe et devraient donc être en majuscules séparées par des soulignés.
• La gestion de l'absence d'une devise est très dangereuse. D'ailleurs, elle n'est pas traitée dans la méthode main()
• Le programme est difficilement réutilisable. Il est nécessaire de modifier Conversion pour rajouter le traitement d'une devise supplémentaire.
• Se baser sur l'égalité d'indexation entre une devise et son taux de change est dangereux. De plus, cela rend difficile l'utilisation de plusieurs noms pour une même devise.
• La méthode conversion() n'est pas efficace. Son temps de calcul est proportionnel au nombre de devises manipulées.
• La variable i de conversion() n'a pas à être déclarée en dehors de la boucle.
• La classe Conversion est incomplète. Il n'est pas possible de savoir si une devise est traitée sans demander une conversion.
  De plus, il paraît souhaitable de disposer de la fonction de conversion de l'Euro vers une autre devise.
2. Conversion.java
• Cette version corrige certains défauts constatés dans la version 1.
  
• Il n'est toutefois toujours pas possible de rajouter facilement une devise.
  
• La complexité de la méthode conversion() reste à améliorer.
• Le return -1 n'est plus le résultat d'une méthode publique. Il est donc contrôlé en "interne". On peut éviter le return -1 de la méthode rechercheIndiceDevise(String devise), sans avoir de duplication de code, en remplaçant les méthodes rechercheIndiceDevise(String devise), indiceDevise (String devise) et boolean deviseTraitee (String devise) par le code suivant, utilisant une variable supplémentaire.

  private int indiceDevise;

  private static int indiceDevise (String devise) {
      if (! deviseTraitee(devise)) {
          throw new Error("Devise " + devise + " inconnue");
      }
      return indiceDevise;
  }

  public static boolean deviseTraitee(String devise) {
      for (indiceDevise = 0; indiceDevise < devises.length; indiceDevise++) {
          if (devises[indiceDevise].equals(devise)) {
              break;
          }
      }
      return indiceDevise < devises.length;
  }

• L'instruction throw new Error(<message>) a pour effet d'arrêter l'exécution du programme en affichant le message passé en paramètre. C'est un peu l'équivalent du assert() du langage C.
  En fait, assert existe en Java. L'expression assert b lève une AssertException si b vaut false. La différence avec l'utilisation directe d'une Error est qu'une instance d'Error n'est pas capturable et que pour que l'assertion soit prise en compte, il faut utiliser l'option d'exécution -ea de la machine virtuelle.
  Dans tous les cas, il serait préférable de lever une exception plutôt que de provoquer à coup sûr un arret.
3. Conversion.java, Devise.java
• Cette version représente un changement d'architecture avec l'ajout d'une classe Devise qui améliore grandement la modularité.
• On notera dans la classe Devise la redéfinition des méthodes equals(Object), toString() et hashCode(), héritées de java.lang.Object.
• La classe Conversion contient un bloc d'initialisation (bloc "static").
• La méthode nom() de la classe Devise est déclarée final pour qu'on ne puisse pas modifier un nom en utilisant l'héritage.
4. Conversion.java, Devise.java
• Seule la classe Conversion a été modifiée : cette version utilise une liste au lieu d'un tableau, ce qui permet d'augmenter facilement les devises traitées.
• On notera l'utilisation d'une liste générique, c'est à dire dont le contenu est typé. Ce type de liste apparaît dans la version 5.0. Voir http://docs.oracle.com/javase/tutorial/extra/generics/index.html, ainsi que le mécanisme de parcours qui remplace le traditionnel Iterator. Voir http://docs.oracle.com/javase/tutorial/java/nutsandbolts/for.html.
5. conversion.Conversion.java, conversion.Devise.java, conversion.DeviseExistanteException.java, conversion.DeviseInconnueException.java,  TestConversion.java
   
• Version avec paquetage.
  
• On encapsule la création des devises dans la classe Conversion, ce qui garantit que cette classe connaît toutes les conversions créées. Pour cela, le constructeur de Devise est accessible uniquement dans son paquetage.
• Deux classes d'exceptions sont crées pour gérer les cas critiques. On notera le traitement des exceptions dans le bloc static de la classe Conversion.
6. conversion.Conversion.java, conversion.Devise.java, conversion.DeviseAbstraite.java, conversion.DeviseConstante.java, conversion.DeviseVariable.java, conversion.DeviseExistanteException.java, conversion.DeviseInconnueException.java,  TestConversion.java
   
• Version avec devises à taux variable ou constant. Pour cela, on utilise deux classes implémentant l'interface Devise, ainsi qu'une classe abstraite pour factoriser le code.
• La méthode tauxDeConversion() est laissée abstraite dans DeviseAbstraite car la donnée correspondante ne peut être factorisée à ce niveau. En effet, dans le cas d'une devise à taux fixe, on doit s'assurer que le taux ne peut être modifié.
• Version 6.1 : l'interface Devise a été modifiée en rajoutant du code par défaut pour les méthodes conversionEnEuro(double somme) et conversionDepuisEuro(double somme). Le code de ces  méthodes est donc supprimé (car inutile) dans DeviseAbstraite.
  
7. conversion.Conversion.java, conversion.Devise.java, conversion.DeviseAbstraite.java, conversion.DeviseConstante.java, conversion.DeviseVariable.java, conversion.DeviseExistanteException.java, conversion.DeviseInconnueException.java,  TestConversion.java
   
• Cette version utilise une table de hachage pour stocker les devises, indexée sur leurs noms, de façon à améliorer la complexité de la récupération d'une devise existante à partir de son nom.