Refactor of swapTwoLast function
This commit is contained in:
parent
6ddbc8d246
commit
09bae8f0a9
2 changed files with 3 additions and 1 deletions
|
@ -5,3 +5,4 @@
|
||||||
* Création de la classe principale `CVDATPJUnit` et de sa classe de test `CVDATPJUnitTest` tous deux vides
|
* 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.
|
* 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.
|
* 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.
|
||||||
|
|
|
@ -16,7 +16,8 @@ public class CVDATPJUnit {
|
||||||
*/
|
*/
|
||||||
public static String swapTwoLast(String in) {
|
public static String swapTwoLast(String in) {
|
||||||
if (in != null && in.length() > 1) {
|
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;
|
return in;
|
||||||
}
|
}
|
||||||
|
|
Reference in a new issue