diff --git a/README.md b/README.md index 4e6a86d..2da746e 100644 --- a/README.md +++ b/README.md @@ -5,3 +5,4 @@ * Création de la classe principale `CVDATPJUnit` et de sa classe de test `CVDATPJUnitTest` tous deux vides * Prototype de la méthode `swapTwoLast`, elle est statique car elle n’as pas besoin d’attributs de la classe. Ajout des test unitaires `swapTwoLastTest` et de la documentation dans la classe principale. * Création du corps de la méthode `swapTwoLast`, utilisation de `substring`, `charAt` et `length` pour permettre la permutation des deux dernier caractères. +* Refactoring de la méthode `swapTwoLast`, pour éviter de trop accéder a la méthode `length` la valeur est stocker dans une variable. diff --git a/src/cvdatpjunit/CVDATPJUnit.java b/src/cvdatpjunit/CVDATPJUnit.java index d8c0bac..69c15d6 100644 --- a/src/cvdatpjunit/CVDATPJUnit.java +++ b/src/cvdatpjunit/CVDATPJUnit.java @@ -16,7 +16,8 @@ public class CVDATPJUnit { */ public static String swapTwoLast(String in) { if (in != null && in.length() > 1) { - in = in.substring(0, in.length()-2) + in.charAt(in.length()-1) + in.charAt(in.length()-2); + int length = in.length(); + in = in.substring(0, length-2) + in.charAt(length-1) + in.charAt(length-2); } return in; }