Coupage string en plusieurs string

Bonjour,
J'ai une petite question, je cherche à améliorer mon code java,
J'ai une String qui est composée de sous chaines de 4caractères, qui sont séparées par un vide.
et je veux alimenter des string d'un objet :
Genre :

Code:

---

String s = YYAA AAAA BBBB CCCC DDDD EE

---

donc ce que je fais moi est :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31

String[] tabString = s.split(" ");   if( tabString == null ){ return; }   this.setIString2(tabString[1]); if( tabString.length >= 3 ){ this.setIString3(tabString[2]); } if( tabString.length >= 4 ){ this.setIString4(tabString[3]); } if( tabString.length >= 5 ){ this.setIString5(tabString[4]); } if( tabString.length >= 6 ){ this.setIString6(tabString[5]); } if( tabString.length >= 7 ){ this.setIString7(tabString[6]); } if( tabString.length >= 8 ){ this.setIString8(tabString[7]); } if( tabString.length >= 9 ){ this.setIString9(tabString[8]); }     }

---

Donc je me demandais, s'il n'y avait pas mieux que ce code ?
Merci

le test sur null est inutile. Split ne peux jamais retourner un null.

Test méthodes setIString2 3 4 5 tout ce que tu veux devraient avoir des noms explicites, genre

Code:

---

1 2	setStringDate(tabString[0]), setName(tabString[1]);

---

Paradoxalement, tu teste null mais tu as oublié de tester la taille de ton tableau (en tout cas pour l'index 1).

Code:

---

1 2	if (tabString.length!=EXPECTED_MIN_COLUMN_COUNT) throw new ApplicationException("Ligne incorrecte dans fichier à l'index ....)

---

Enfin, coder les index attendus dans des constantes serait plus lisible.

Code:

---

1 2 3 4

public static final int INDEX_DATE = 0; //.... if (tabString.length>INDEX_DATE) setStringDate(tabString[INDEX_DATE]);

---

Merci tout d'abord pour ta réponse
Donc, à part le test que tu me dis de rajouter pour le null, je ne peux pas faire mieux que mon code ??
Merci

Si si, tu peux donner de meilleurs noms que setIString1, etc.
À la limite s'il n'y a pas vraiment de meilleur nom que ça, alors iString1, iString2 etc devraient être remplacés par un tableau de String, avec un seul setter pour ce tableau. La seule vérification à faire serait alors sa taille.

Citation:

---

Envoyé par y_chafaqi

Donc, à part le test que tu me dis de rajouter pour le null, je ne peux pas faire mieux que mon code ??

---

Ca veut dire quoi "mieux que ce code" ? Si ce que tu veux, c'est factoriser, tu peux utiliser la reflexion pour appeler chaque méthode.
Plus simple, tant qu'à avoir des méthode au nom qui ne veut rien dire, au lieu de les appeler "void setIStringN(String s)", tu peux n'en utiliser qu'une : "void setIString(int num, String s)". Comme ca, c'est facilement parametrable avec une boucle.