Tableau de caractère variable

Version imprimable

Bien le bonjour,
voilà, j'ai écris ce code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
		String queryx[][] = new String[5][1];
		int i = 0;
		queryx[i][0] = "test0";
		if(!Utils.isEmpty(filterEvtStatus)) {
			queryx[++i][0] = "test1";		
		}
		if(!Utils.isEmpty(filterLocSitID)) {
			queryx[++i][0] = "test2";		
		}
		if(!Utils.isEmpty(filterLocSuiviParActID)) {
			queryx[++i][0] = "test3";		
		}
		if(!Utils.isEmpty(filterPl_ID)) {
			queryx[++i][0] = "test4";		
		}
		String query[][] = new String[++i][1];
		i = 0;
		for(String[] temp:queryx){
			if(temp[0] != null){
				query[i] = temp;
				++i;
			}
		}

Qui comme vous pouvez le voir est vraiment moche.
Comment pourrais je faire pour n'avoir que les données conditionnées dans mon tableau de départ sans passer par un petit for pour éliminer les valeurs null?
Merci.

04/08/2011, 15h02
Invité

Pour info je passe la variable query dans une fonction qui attend du String[][].
04/08/2011, 15h26
Ivelios
Désolé mais je n'ai pas compris ce que tu souhaite au final, peux tu donner un peu exemple STP.

Code:

String queryx[][] = new String[5][1];

=
Code:

1 2 //Plus joli String queryx[] = new String[5];
Code:

++i

N'aide pas du tout à la compréhension du code :/

Code:

!Utils.isEmpty(...)

C'est quoi ça ?

A moins d'utiliser une liste tu sera toujours obligé de faire 2 parcours :
Le premier récupère le nombre de valeur (16 par exemple)
Le deuxième les stock dans un tableau de 16

Tu peux aussi crée une liste que tu remplie :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
List<String> mylist = new ArrayList<String>();
if(!Utils.isEmpty(filterEvtStatus)) {
	mylist.add("test1");		
}
if(!Utils.isEmpty(filterLocSitID)) {
	mylist.add("test2");		
}
if(!Utils.isEmpty(filterLocSuiviParActID)) {
	mylist.add("test3");	
}
if(!Utils.isEmpty(filterPl_ID)) {
	mylist.add("test4");		
}
String queryx[][] = new String[5][1];
queryx[0]=mylist.toArray();

Note : c'est une idée, je ne garantie pas la compilation de ce code.

Cordialement,
Patrick Kolodziejczyk.

En fait, j'ai légèrement modifié mon code depuis (problème de logique fonctionnelle) :
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 String queryx[][] = new String[1][5]; int i = 0; queryx[0][i] = "PvtEventID = 'truc1'" ; if(filterEvtStatus.equals("youpi") { queryx[0][++i] = "PvtStatus = 'truc2'"); } if(filterLocSitID.equals("youhou") { queryx[0][++i] = "PvtPersonID:PerLocID:LocSitID_ = 'truc3'"; } if(filterLocSuiviParActID.equals("ici") { queryx[0][++i] = "PvtPersonID:PerLocID:LocSuiviParActID_ = 'truc4'"); } if(filterPl_ID.equals("i'll be back") { queryx[0][++i] = "PvtPl_ID_ = 'truc5'"); } String query[][] = new String[1][++i]; i = 0; for(String temp:queryx[0]){ if(temp != null){ query[0][i] = temp; ++i; } }
En gros, dans la variable queryx, je récupère toujours un tableau avec un tableau de 16 chaines de caractère.
Mais certaine de ces chaines sont en fait nulle, donc je créer un autre tableau avec un tableau de i+1 String.
Je ne connais la taille de mon tableau qu'après les condition.
Je passerais bien par des listes, mais je suis limité par la signature de la méthode que j'utilise par la suite. A moins qu'il n'existe un moyen de transformer une liste de liste en tableau de tableau de chaine de caractère ?

Citation:

Code:

++i

N'aide pas du tout à la compréhension du code :/

Faux, des if sur une ligne c'est incompréhensible par exemple, pas des pré incrémentation.

Citation:
Envoyé par kolodz
Tu peux aussi crée une liste que tu remplie :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 List<String> mylist = new ArrayList<String>(); if(!Utils.isEmpty(filterEvtStatus)) { mylist.add("test1"); } if(!Utils.isEmpty(filterLocSitID)) { mylist.add("test2"); } if(!Utils.isEmpty(filterLocSuiviParActID)) { mylist.add("test3"); } if(!Utils.isEmpty(filterPl_ID)) { mylist.add("test4"); } String queryx[][] = new String[5][1]; queryx[0]=mylist.toArray();
Note : c'est une idée, je ne garantie pas la compilation de ce code.

Cordialement,
Patrick Kolodziejczyk.
Intéressant, mais le ArrayList<String>.toArray() me renvoi un tableau d'objet donc cast impossible Object[]->String[].

04/08/2011, 16h37
Ivelios

Citation:

Faux, des if sur une ligne c'est incompréhensible par exemple, pas des pré incrémentation.

Les placer directement dans un tableau ça rebute un peu. Mais chacun ses gout et ses couleurs, je respecte :P

Citation:

Pour info je passe la variable query dans une fonction qui attend du String[][].

N'y a t'il pas moyen de modifier cette fonction mystère à notre avantage? Ou est ce une contrainte fixe?
04/08/2011, 16h42
Ivelios

Citation:

mais le ArrayList<String>.toArray()

Au final, toArray ré-alloue un new Tableau. ça implique forcement (normalement) un parcours et donc une boucle.

Citation:

(In other words, this method must allocate a new array even if this collection is backed by an array)
04/08/2011, 16h43
Invité

Citation:

Envoyé par Ivelios

Les placer directement dans un tableau ça rebute un peu. Mais chacun ses gout et ses couleurs, je respecte :P

pas faux ;-)

Citation:

N'y a t'il pas moyen de modifier cette fonction mystère à notre avantage? Ou est ce une contrainte fixe?

Le problème c'est que je suis dans un progiciel dont je n'ai pas les sources.
Et la méthode utilisée est dans un class bien entendu.

Alors utilise l'autre fonction toArray :D

Voici un exemple simple :
Code:

1 2 3 4 5 6 List<String> test =new ArrayList<String>(); test.add("toto"); String table[] =(String[]) test.toArray(new String[test.size()]) ; for (int i = 0; i < table.length; i++) { System.out.println(table[i]); }
Ce code compile !

Cordialement,
Patrick Kolodziejczyk.

04/08/2011, 17h19
Invité

Parfait, avec la forme

Code:

String query[][] = {(String[]) test.toArray(new String[test.size()])};

ça marche nickel.

Pour info,J'ai fait des tests de perf entre les deux formes sur une boucle de 1000000, nous sommes à 1.4s pour la premiere et 1s pour la deuxième pour le meme environnement.
Donc c'est plus joli, mais pas tellement plus performant.
:lol:
04/08/2011, 17h29
kolodz

Tu prend 29% de temps en moins, je trouve le résultat déjà pas mal pour la réécriture d'un morceau de code de 10 lignes.

Bonne continuation.