Discussion :

[liverbird]

Bonjour,

J'ai plusieurs tableaux de string de la forme [N/A, 29, 58, 84, 97, 88, 81, 77, 64, 29, N/A, N/A] par exmple.
Les tableaux sont tous différents donc les "N/A" peuvent se trouver n'importe où dans le tableau.
J'aimerais traiter ces données en remplaçant les cases "N/A"par la case voisine (si celle ci est différente de "N/A" bien sur).
Je pensais faire une boucle "for" pour parcourir le tableau et puis une "while" à l'intérieur du "for" pour voir si la case voisine et différente de "N/A".
Je ne suis pas très convaincu de mon approche car en plus si la fin du tableau contient des "N/A" comme l'exemple au dessus il faudra les remplacer par une case précédente et non suivante.
Qu'elle est ma meilleure approche dans ce cas là.
Merci

[liverbird]

J'ai testé le "while" dans le "for" et ça a marché. Je ne sais pas si ça reste la meilleure solution mais le résultat est satisfaisant.
Au début je fais ma boucle "for" sur les 12 cases du tableau et à l'intérieur j'appelle une méthode qui fait le "while"

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
for (int k=508; k<520; k++){
					if (lineArray[k].equals("N/A")){
						String value = getNearestValidValue(lineArray, k, 514);
						wdBNIList.add(value);
					}
					else
						wdBNIList.add(lineArray[k]);
				}

La méthode qui fait le "while" est là:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
public static String getNearestValidValue(String[] lineArray, int index, int middleIndex){
			int j = 1;
			if (index <middleIndex){
				while(lineArray[index+j].equals("N/A")){
					j++;
				}
				return lineArray[index+j];
			}
			else {
				while(lineArray[index-j].equals("N/A")){
					j++;
				}
				return lineArray[index-j];
			}
	}

A +

[thelvin]

Hello,

Il y a moyen d'optimiser, notamment si tes listes font des kilomètres et ne contiennent presque que des N/A et que ton programme ne fait que ça toute la journée.

Mais optimisation mise à part, c'est une manière assez simple et facile à comprendre de résoudre le problème. Donc bon c'est pas forcément à changer, quand on y pense.

[joel.drigo]

Salut,

Justement, le problème de tes while, c'est qu'ils présentent le défaut que tu évoquais : s'il n'y a que des N/A "à la suite" de la valeur trouvé ("à la suite" dans le sens de variation de j, je veux dire), la condition du while n'est jamais fausse et tu obtiendras une ArrayIndexOutOfBoundException. Il faut que la condition du while limite l'incrémentation (ou la décrémentation) du j, pour que ça ne sorte pas des index possibles du tableau (dans un sens comme dans l'autre, avec un for plutôt qu'un while, tant qu'à faire). Dans le cas extrême, il est même possible qu'aucune valeur ne soit trouvée (que des N/A dans le tableau).

[liverbird]

Merci pour vos réponses. J'ai en effet plusieurs milliers de tableaux mais ils ressemblent tous plus au moins au tableau que j'ai mis dans le premier commentaire. Du coup mon code marche. Mais c'est vrai que ce code mérite une optimisation comme l'a signalé joel.drigo. Je posterai une amélioration alors dès que j'en aurai une. Merci