Discussion :

[weill nathanal]

bonjour,
Je suis débutant en java. J'aimerai vous soumettre un problème que j'ai. Je ne pense pas qu'il s'agisse d'un problème algorithmique mais bien un problème inhérent au langage JAVA.
voici l'histoire.

j'ai réalisé un programme qui sera amené a fonctionné sur un grand nombre de données de la façon suivante:

1) lecture d'un fichier
2) analyse du fichier
3) traitement

Lors de ces trois phases, j'instancie un grand nombres d'objets.

l'exécution de ce programme prend environ 4.5 secondes lorsque je l'exécute sur un fichier.

J'ai ensuite essayé d'insérer une boucle for dans le main pour faire une moyenne du temps d'exécution :

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
public static void main(String[] args) {
		//un parseur de fichier
		CustomFileReader fic = null;
		//une structure de donnée pour stocker l'info
		HashMap<String, Double> infos=null;
 
		for (int i = 0 ; i < 4; i ++ ){
 
			long debut =System.nanoTime()/1000000000;
			long fin =0;
 
			System.out.println( "debut : "+debut ) ;
 
			fic = null;
			infos = null;
			1) lecture du fichier
		        	2) analyse du fichier
                       		3) traitement
 
 
 
 
			System.gc();
			fin = System.nanoTime()/1000000000;
			System.out.println( "fin : "+fin  ) ;
			System.out.println( "diff : "+(fin-debut)  ) ;
		}
		System.exit(0);
 
	}

voici ce que le programme genere comme sortie :

Code X :

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
debut : 1162548161

fin : 1162548166
diff : 5
debut : 1162548166

fin : 1162548183
diff : 17
debut : 1162548183

fin : 1162548219
diff : 36
debut : 1162548219

fin : 1162548282
diff : 63

diff correspond au temps d'exécution d'une boucle. On voit bien que le temps d'exécution crois très vite alors qu'il s'agit du même traitement exécuté 4 fois sur le même fichier

D'autre part, j'ai essayé par un script perl de lancer le programme sans la boucle for :

Code :

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

1
2
3
4
5
#!/usr/bin/perl
 
for($i = 0 ; $i <4 ; $i++){
       System("time java - jar test.jar");
}

je constate alors que le temps d'exécution reste constant...

Comment cela ce fait il que le temps d'exécution augmente dans un cas et pas dans l'autre?????
Quelqu'un aurai-t-il une idée
Je vais être amené a traiter un grand nombre d'information et je ne souhaiterai pas avoir a utiliser un artifice tel que "lancer le programme avec un script perl"
Si quelqu'un pourrai me donner une piste cela m'aiderai beaucoup.

[Faiche]

tu évites au maximum les déclarations et allocations à l intérieur de ta boucle, tu réutilises au maximum tes variables, et tu sors le plus de traitement possible de ta boucle.

[weill nathanal]

oui c'est ce que je fais:

tous mes objet sont déclaré en dehors des boucles et sont instancier a l'interieur (pas le choix)
en gros avant chaque boucle, je fais:
object toto = null;
dans la boucle:
toto = new object();
fain de la boucle

[Marnor]

A noter que tu utilises system.gc.... ce n'est vraiment pas recommandé...

[weill nathanal]

j'ai essaye avec sans system.gc() rien ne change....

[Marnor]

D'ou l'inutilité de l'appeler... le garbage collector est géré par java il n'est pas conseillé de l'appeler a tout va..... c'était juste une remarque...
Sinon remet tu bien toutes tes variables instanciés à null à la fin de ton execution (afin justement qu'elles soient nettoyées)...??

[weill nathanal]

oui a chaque debut de la boucle for, tous mes objets sont réinitialisé a null

[Marnor]

ce qui fait que ton prog met a null un objet qu'il connait deja... moi je demande en fin d'execution... jsuis pas si sur que ca que ca change beaucoup de chose... il faudrait que tu nous fasses voir plus en détail le code.....

[weill nathanal]

je viens d'essayé de mettre a nulle en fin de boucle et non en début: cela ne chage rien
je ve bien donner plus de détails sur mon code, mais il est déja assé complex et je pense pas que cela vous aiderai.
Vu que le temps double a chaque iteration...
Je pencherai plutot pour une solution inverse: declarer tous mes objets dans les boucles pour etre sur que la MV java les désalous bien en fin de parcourt...

[plegat]

Salut,

En testant ton code, en remplaçant ton traitement de fichier par une boucle "for" vide (j'ai aussi remplacé le "CustomFileReader"...), on tombe sur des durées similaires... donc ça doit venir de ton traitement de fichier...

Essaye cette manipulation de ton côté (tu enlèves les points 1-2-3 du code que tu nous a donné, tu remplaces par une boucle "for" vide suffisamment longue pour avoir des durées), dis-nous ce que tu obtiens.
Si tu as des écarts similaires, il faudra sans doute se pencher sur ton traitement du fichier... il doit y avoir un truc litigieux qui traine...

[weill nathanal]

merci je vais tester ca de suite

[weill nathanal]

Salut,

En testant ton code, en remplaçant ton traitement de fichier par une boucle "for" vide (j'ai aussi remplacé le "CustomFileReader"...), on tombe sur des durées similaires... donc ça doit venir de ton traitement de fichier...

Essaye cette manipulation de ton côté (tu enlèves les points 1-2-3 du code que tu nous a donné, tu remplaces par une boucle "for" vide suffisamment longue pour avoir des durées), dis-nous ce que tu obtiens.
Si tu as des écarts similaires, il faudra sans doute se pencher sur ton traitement du fichier... il doit y avoir un truc litigieux qui traine...

bon me revoila
alors g essayé de relancer le teste en mettant
létape 1
puis les étapes 1 et 2
puis 1, 2 et 3.
visiblemant, c qd je rajoute l'étape 3 que le temps augmente.
je vais creuser de ce coté mais je ne comprend toujours pas pourquoi lorsque je met a null tous mes objet cela ne suffit pas.

[weill nathanal]

voila l'histoire
c une grosse boulette de ma pare:
j'ai crée une liste statique membre de classe. A chaque création d'un objet de cette classe, j'ajoute des données dans cette liste. Du coup les données restaient stockées en membre de classe et quand je parcoure ces données, la taille augmentais.

voila
moralité bien faire gaffe aux objets statiques....
heureusement que j'ai commencé par dire que j'étais un débutant
merci a tous

[®om]

oui a chaque debut de la boucle for, tous mes objets sont réinitialisé a null

Ca sert à rien de mettre les variables à null, que ça soit au début ou à la fin de la boucle...
Au début, car de toute façon tes variables vont référencer un nouvel objet par la suite (sans doute par un new), donc l'ancien n'est plus lié, pas besoin de passer par un état intermédiaire "null".
À la fin, car de toute façon tu sors d'un bloc ( { } ) (en l'occurence, for), et donc tes variables n'existent plus après chaque tour de boucle...

Enfin, c'est juste parce que ça fait je ne sais pas combien de fois que je lis qu'il faut remettre à null des variables avant de quitter une fonction ou une boucle, alors que c'est inutile... (fin de bloc => variables locales disparues)

[weill nathanal]

oui c'est bien ce qui me semblais avoir lu dans la doc java
mais bon au début on commence toujours par remettre en cause la jvm et finalement il s'agit que d'une boulette de la part de programmeur...

[Marnor]

Ca lui aurait quand meme éviter pour sa variable static... d'avoir ce souci... mieux vaut prévenir que guérir....m'enfin c'est vrai que pour des variables locales c'est pas utile...