Discussion :

[ZwQuery]

Bonjour à tous,

J'ai codouillé une application qui utilise les threads, mais vu mon niveau...


En gros, soit je rencontre des problèmes de deadlocks soit je perds l'avantage des threads à moins... qu'il ne soit possible de coder en dur dans "java.io.BufferedOutputStream". Il faudrait modifier son code mais je ne sais pas si c'est possible... ni si ma question est claire... ^^


merci à vous

[thelvin]

Tu auras plus vite fait d'écrire un autre BufferedOutputStream que d'essayer de modifier son code (ce qui n'est pas impossible puisque ni la classe, ni ses méthodes, ne sont final.)

Par contre, je ne vois pas le rapport avec les threads.

[ZwQuery]

Merci d'avoir répondu si vite

Il faut bosser sur le bytecode si je veux modifier BufferedOutputStream ?


J'utilise File.renameTo() avec des threads qui accèdent aux mêmes variables et celui-ci est hors de la boucle utilisée par BufferedOutputStream.

Si nécessaire, je posterai du code...

[tchize_]

faudrait déjà préciser c'est quoi ton problème. Parce que jusque là tes messages n'ont ni queue ni tête!

[ZwQuery]

oui... en effet... désolé...

J'utilise BufferedOutputStream pour télécharger des fichiers.
J'ai donc créé une méthode qui utilise deux variables (name / url) avec BufferedOutputStream.

Lors de téléchargements simultanés, tout fonctionne parfaitement tant que je ne change pas le dossier de destination en cours de route. Les variables "name" et "url" ne sont plus utilisées sur les bons fichiers. J'aimerais donc "forcer" un chemin d'accès directement dans "mon" BufferedOutputStream.

Je vais suivre le conseil de thelvin...

J'espère avoir été un peu plus clair.

[thelvin]

J'espère avoir été un peu plus clair.

Pas beaucoup. Une fois qu'un BufferedOutputStream a été créé, tu peux modifier toutes les variables que tu veux, en tout cas lui il écrira toujours au même endroit et n'en aura rien à cirer de tes variables.

De toute façon tu n'as pas dit, concrètement, qu'est-ce qui se passe et pourquoi ce n'est pas bien.

Je vais suivre le conseil de thelvin...

Je vois mal comment.

[deathness]

Je pense que tu as plus un problème de compréhension de ce que tu veux faire ou du fonctionnement de tout ça qu'autre chose...
Il vaudrait mieux que tu revienne à la base de ce que tu veux faire et reparte de là.

[ZwQuery]

Bonjour à tous,

Il vaudrait mieux que tu revienne à la base de ce que tu veux faire et reparte de là.

Oui, de fait...
Ce sera plus clair avec du code (j'espère :p)


L'idée du programme est de télécharger simultanément d'autres programmes et de les copier dans un sous-dossier "Outils". Pour ce faire, l'utilisateur clique sur des boutons qui appellent une méthode "download".


Un thread est créé pour chaque "download". Les variables "name" (nom du fichier à télécharger) et "url" (adresse de l'outil) sont évidemment différentes pour chaque téléchargement.

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
 
...
protected void download() throws IOException {
 
...
 
java.io.BufferedInputStream in = null;
 
try {
in = new java.io.BufferedInputStream(new java.net.URL(url).openStream());
...
 
java.io.BufferedOutputStream bout = new BufferedOutputStream(fos,1024);
		   				byte[] data = new byte[1024];
		   				int x=0;
		   				try {
								while((x=in.read(data,0,1024))>=0)
								{
 
 
								bout.write(data,0,x);
 
								}
							} catch (IOException e1) {
								// TODO Auto-generated catch block
								isok.setText("Téléchargement de " +name+ " manqué !");
								e1.printStackTrace();
							}
		   				try {
								bout.close();
							} catch (IOException e1) {
								isok.setText("Téléchargement de " +name+ " manqué !");
								// TODO Auto-generated catch block
								e1.printStackTrace();
							}
		   				try {
								in.close();
							} catch (IOException e) {
								isok.setText("Téléchargement de " +name+ " manqué !");
								// TODO Auto-generated catch block
								e.printStackTrace();
 
							}		
 
 
 
								final File fichier = new File (name);
						...
 
	                          else
							  {
                                  isok.setText("Téléchargement de " +name+ " terminé !");
							  }
 
						     fichier.renameTo(new File("Outils/" +name));

Tout fonctionne à la perfection (le .JAR est conséquent) sauf que si je télécharge plusieurs fichiers en même temps (plusieurs appels à la méthode "download" avec des variables différentes), le programme ne copie pas les fichiers téléchargés dans le dossier "Outils" (parce que ma variable "name" a changé entretemps).

Par contre, si je retire

Code :

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

fichier.renameTo(new File("Outils/" +name));

alors tous les outils sont bien téléchargés dans le dossier courant.

J'aurais donc voulu forcer mon outil à télécharger directement les outils au bon endroit...

merci

[tchize_]

(parce que ma variable "name" a changé entretemps).

Elle ne devrait pas. Alors plutot que d'essayer de bidouiller pour amener les données directement dans le bon dossier, contente toi de t'assurer que toutes les données qui sont propre à chaque téléchargement sont dans une variable spécifique à ce téléchargement (typiquement ton Thread ou ton Runnable) plutot que d'être générale.

[ZwQuery]

Bonsoir et merci à tchize_ d'avoir pris le temps de répondre malgré l'extrême chaos de mes premiers messages.

Elle ne devrait pas

Je me doutais qu'il y avait une erreur de conception.

contente toi de t'assurer que toutes les données qui sont propre à chaque téléchargement sont dans une variable spécifique à ce téléchargement (typiquement ton Thread ou ton Runnable) plutot que d'être générale

Là, je comprends l'idée mais ne vois pas comment procéder.
Il me faudrait des noms de variables différents pour chaque téléchargement (name1, url1, name2, url2, ...). Dans ce cas, comment les récupérer pour ne pas devoir écrire 200 fois la méthode "download" ?

J'avais essayé d'utiliser "Synchronized", mais cela bloque vite le programme si je lance plus de 3 téléchargements simultanément.


A mon avis, un seul mot clef suffira à orienter mes recherches...
Est-ce que "wait()" et "notify()" seraient indiqués dans mon cas ?
isAlive() ne semblait pas renvoyer la bonne valeur.

Merci de m'indiquer une piste vers la solution qui serait la plus indiquée face à ce souci.

[wax78]

Avant de jouer avec des Thread ou quoi que ce soit de compliqués, réécrit ta fonction pour prendre en paramètre l'url et le name ca ira surement mieux (Je n'ai pas eu l'impression de voir le code qui appele download donc je présume la) comme l'a conseillé Tchize.


Remplace

Code :

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

protected void download() throws IOException

par

Code :

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

protected void download(URL url, String name) throws IOException

[tchize_]

Il me faudrait des noms de variables différents pour chaque téléchargement

Non, la même variable, mais stockée dans des instance de Thread ou Runnable ou Task ou que sais-je différente:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class MonThread extends Thread {
  private String unTruc;
  public MonThread(String unTruc){this.unTruc=unTruc;}
  public void run(){
 ////
  }
}
 
new MonThread("1").start();
new MonThread("2").start();
new MonThread("3").start();
new MonThread("4-0").start();