FEAT ITEM-308-back-ajuster-le-controles-sur-le-nombres-de-colonnes-pa…

…r-ligne-dans-un-fichier-de-correspondances : - ajout d'une constante dans Constant.java - ajout d'un contrôle de longueur de ligne dans la méthode checkBodyLine() dans FichierEnrichiSupp.java - modification d'un contrôle de longueur de ligne dans la méthode check3Cols() dans FichierEnrichiSupp.java
abes-esr · Oct 21, 2024 · 2fe2fe3 · 2fe2fe3
1 parent 2f8c24d
commit 2fe2fe3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/core/src/main/java/fr/abes/item/core/components/FichierEnrichiSupp.java b/core/src/main/java/fr/abes/item/core/components/FichierEnrichiSupp.java
@@ -65,7 +65,7 @@ public void checkFileContent(Demande d) throws FileCheckingException, IOExceptio
      * @throws FileCheckingException : erreur dans le format de la ligne
      */
     private void check3Cols(String ligne) throws FileCheckingException {
-        if (ligne.split(";").length < 3) {
+        if (ligne.split(";").length != 3) {
             throw new FileCheckingException(Constant.ERR_FILE_3COL_SUPP);
         }
         if (ligne.length() < 11) {
@@ -84,12 +84,17 @@ private void check3Cols(String ligne) throws FileCheckingException {
      */
     private void checkBodyLine(String ligne, DemandeSupp demandeSupp) throws FileCheckingException {
         try {
+            // contrôle de la longueur de la ligne
+            if (ligne.split(";").length > 3) {
+                throw new FileCheckingException(Constant.ERR_FILE_ERRLINE + " \"" + ligne + "\" " + Constant.ERR_FILE_3COL_SUPP_ANY_LINE);
+            }
             String[] tabligne = ligne.split(";");
+            // contrôle du ppn
             if (demandeSupp.getTypeSuppression().equals(TYPE_SUPPRESSION.EPN) && tabligne[0] != null) {
                 checkPpn(tabligne[0], ligneCourante);
             }
             checkRcr(tabligne[1], demandeSupp.getRcr(), ligneCourante);
-            //cas ou l'epn est renseigné
+            // contrôle de l'epn s'il est renseigné
             if (tabligne.length > 2)
                 checkEpn(tabligne[2], ligneCourante);
         } catch (IndexOutOfBoundsException e) {

diff --git a/core/src/main/java/fr/abes/item/core/constant/Constant.java b/core/src/main/java/fr/abes/item/core/constant/Constant.java
@@ -109,6 +109,7 @@ public class Constant implements Serializable {
 	public static final String ERR_FILE_TOOMUCH_SUPP = Constant.ERR_FILE_TOOMUCH_START + Constant.MAX_LIGNE_FICHIER_INIT_SUPP + Constant.ERR_FILE_TOOMUCH_END;
 	public static final String ERR_FILE_3COL_MODIF = "La première ligne du fichier doit contenir 4 colonnes (ppn;rcr;epn;zone).";
 	public static final String ERR_FILE_3COL_SUPP = "La première ligne du fichier doit contenir 3 colonnes (ppn;rcr;epn)";
+	public static final String ERR_FILE_3COL_SUPP_ANY_LINE = "La ligne ne doit contenir que trois colones";
 	public static final String ERR_FILE_ERRLINE = "Erreur ligne ";
 	public static final String ERR_FILE_ONLYONEPPN = "la ligne ne doit contenir qu'un ppn (sur 9 caractères).";
 	public static final String ERR_FILE_HEAD4TH = "La valeur en-tête de la quatrieme colonne n'est pas valide.";