Discussion :

[Baboulinet_]

Bonjour chers collègues !

Je développe actuellement une application autonome en java.
Cette application va parcourir l'arborescence d'un dossier pour analyser des fichiers. Dans chaque fichiers il y a environ 30 000 lignes d'informations.
Parmi celles ci, il y a des lignes intéressantes.

Sur toutes les lignes, les différents paramètres dont j'ai besoin sont séparés par des "|" et je récupère seulement les lignes dans lesquelles il y a des requêtes sql qui m'intéresse.

Maintenant que le sujet est bien présenté, voici le gros du problème.

Pour sauvegarder ces paramètres j'ai donc créé une liste array pour me simplifier la vie au niveau de la taille de la liste. Dans mon "main" je crée 2 listes identiques qui possèdent toutes les données nécessaires à mon travail. Le problème se passe lors du parcours :
J'aimerai pouvoir remplir les données manquantes dans certains objets de ma liste, mais mon programme ne rentre pas dans mon "if" qui va dire au programme que le paramètre "untel" de l'objet de la liste 1 est vide et qu'il faut le remplir par rapport au paramètre 'untel' de l'objet de la liste 2 qui lui est plein, tout ça si les codes_msgid et code_user des objets sont identiques.
J'ai cru déceler le début de mon problème une fois que le problème passe dans le 1er "if" !

Voici un bout de code.

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
public static void main (String[] args) {
 
		File rep = new File("C:\\Temp");
		List<MSG_ID> list_msgid1 = new ArrayList<MSG_ID>();
		List<MSG_ID> list_msgid2 = new ArrayList<MSG_ID>();
 
		//Collections.sort(list_msgid1);
		list_msgid1 = anaFic(listFicLog(rep));
		list_msgid2 = list_msgid1;
 
 
 
		for(int i = 0; i < list_msgid1.size(); i++){ // tant que j'ai un element non parcouru dans la liste principale
 
			MSG_ID m1 = list_msgid1.get(i);
 
			for(int j = 0; j < list_msgid2.size(); j++){ //on parcourt la même liste pour comparer et associer les différents paramètres
 
				MSG_ID m2 = list_msgid2.get(j);
 
 
				if(m1.get_code_msgid() ==  m2.get_code_msgid() && m1.get_code_user() == m2.get_code_user()){ //on retrouve l'utilisateur avec son pda
 
 
 
					if(m1.get_requete_select() == "" && m2.get_requete_select() != ""){
						//cas où l'objet qui contient la requête select ne contient pas de données
						//on remplit donc les paramètres qui sont associés à la requête select dans l'objet MSG_ID
						System.out.println("Remplissage Select");
						m1.set_heure_select(m2.get_heure_select());
						m1.set_requete_select(m2.get_requete_select());
						list_msgid1.remove(j);//supprime l'objet devenu obsolète
 
 
					}
					else if(m1.get_requete_insert() == "" && m2.get_requete_insert() != ""){
						//cas où l'objet qui contient la requête insert ne contient pas de données
						//on remplit donc les paramètres qui sont associés insert à la requête dans l'objet MSG_ID
						System.out.println("Remplissage insert");
						m1.set_heure_insert(m2.get_heure_insert());
						m1.set_requete_insert(m2.get_requete_insert());
						list_msgid1.remove(j);//supprime l'objet devenu obsolète
					}
 
				}
 
			}
 
		}
 
 
		System.out.println("Analyse des fichiers terminée !");
 
 
 
	}

un exemple parlant de mon problème dans 2 objets différents :

Objet 1 :

Code msgid : 479599919822326460
Code user : 000198
Code zetes : 0
Heure requête insert : 20:47:26
Requête insert : <Insert into blabla>
Heure requête select :
Requête select :

Objet 2 :

Code msgid : 479599919822326460
Code user : 000198
Code zetes : 0
Heure requête insert :
Requête insert :
Heure requête select : 20:47:36
Requête select : <select blabla>

Vous voyez bien que malgré que le paramètre select du 1er objet est vide, il devrait se remplir par rapport à l'objet 2, mais ne le fait pas puisqu'il ne peut aller dans le "if" créé à ce sujet.

Je suis disponible à tout moment pour toute aide à la compréhension de ce problème, je checkerai assez souvent l'aide proposée tout en continuant de m'arracher les cheveux sur mon erreur qui est surement aussi grosse que l'écran .

Merci d'avance

[Baboulinet_]

Bon j'ai trouvé ...
Une méthode débile est à appliquer dans mon if, je laisse mon problème pour connaître vos avis (si je pouvais améliorer mon code par exemple).

[bpy1401]

bonjour Baboulinet_

Quand je teste une chaine de caractères , je fait

Code :

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

maChaine.equals("")

en non pas

Code :

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

1
2
 
maChaine == ""

pas sur que cela resolve tout.

[thelvin]

Disons qu'il faut faire equals() et non pas ==. C'est une certitude.

Enfin, dans le cas présent, on pourrait aussi vérifier si la longueur de la chaîne est zéro :

Code :

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

maChaine.length() == 0

Mais bon, on ne connaît pas trop la logique de ce programme, et si ça se trouve, tout est à revoir et ces comparaisons de chaînes n'ont pas lieu d'être. C'est possible, c'est dur à dire.

[Baboulinet_]

Vivi merci beaucoup, je l'avais déjà trouvé comme dit sur la réponse ci dessus.
C'était bien la fonction equals (mon indice dans la réponse ci dessus vous a mis sur la voie hihi)
J'ai jamais eu l'utilité de cette fonction car normalement les chaines peuvent sans dire doivent !!!!! être comparées avec ==

Enfin, je ne suis pas non plus un mega pro du java, mais se prendre la tête 3h sur une méthode bizarroide ...

[bpy1401]

Quand tu ecrit ceci

Code :

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

m1.get_requete_select() == ""

tu fait une comparaison entre objets,

Avec le .equels tu fais un e compraison du contenu

ce qui n'est pas forcement pareil

[Baboulinet_]

pourtant l'intérieur du paramètre get_requete_select me ramène un string
et " " est considérée comme une string vide
Voilà pourquoi je ne trouve pas trop logique cette fonction puisqu'avec les outils qu'on dispose, on aurait pu le faire rien qu'avec les ==
Mais maintenant je saurai que m1.machin == truc va comparer l'objet m1 avec l'objet truc
Donc si je comprends bien, il faut que j'assimile le "" comme un objet lorsque je le compare avec m1.get_requete_select ?

[atha2]

"" est bien un objet. Tu peux d'ailleurs écrire "".equals(m1.get_requete_select). Ainsi tu évites d'avoir une NullPointerException dans le cas ou m1.get_requete_select retourne null

[tchize_]

pour être plus précis, quand dans ton code tu écrit un chaine entre guillement, par exemple

String a = "bonjour";

le compilateur, 'en gros' convertis ceci en

String a = "bonjour".intern();

interne étant une méthode centralisant les string de ton application pour éviter les duplication. Pas contre, la majorité des string récupérées autrement, par exemple en parsant un fichier, ne sont pas passées explicitement dans cette méthode par le programmeur, pour des raisons de performance. Résultat, on a ce comportement:

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
 
String a = "bonjour";
String b = "bonjour le monde";
String c = b.subString(0,7); // "bonjour"
 
if (a == a) 
   System.out.println("fonctionne");
if (a == c);
   System.out.println("ne fonctionne pas");
if (a == "bonjour");
   System.out.println("fonctionne");
if (a == c.intern())
   System.out.println("fonctionne");
if (c.equals("bonjour"))
   System.out.println("fonctionne");

[Baboulinet_]

Merci beaucoup pour vos réponses

Passez une bonne journée,

Cordialement, Baboulinet !