From 09bae8f0a90e5ef31e71956b209a55b4750374b6 Mon Sep 17 00:00:00 2001
From: CHARLAIX FLORIAN p1905458 <flifloo@gmail.com>
Date: Fri, 24 Apr 2020 11:02:19 +0200
Subject: [PATCH] Refactor of swapTwoLast function

---
 README.md                        | 1 +
 src/cvdatpjunit/CVDATPJUnit.java | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)

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;
     }