Modifier "java.io.BufferedOutputStream" ?

Version imprimable

21/06/2012, 18h36
ZwQuery

Modifier "java.io.BufferedOutputStream" ?

Bonjour à tous,

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

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
21/06/2012, 19h02
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.
21/06/2012, 19h51
ZwQuery

Merci d'avoir répondu si vite :ccool:

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... :)
22/06/2012, 00h37
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!
22/06/2012, 07h24
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... :lol:

J'espère avoir été un peu plus clair.
22/06/2012, 07h32
thelvin

Citation:

Envoyé par ZwQuery

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.

Citation:

Envoyé par ZwQuery

Je vais suivre le conseil de thelvin...

Je vois mal comment.
22/06/2012, 08h53
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à.

Bonjour à tous,

Citation:

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:

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:

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

04/07/2012, 09h27
tchize_

Citation:

Envoyé par ZwQuery

(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.
05/07/2012, 19h17
ZwQuery

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

Citation:

Elle ne devrait pas

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

Citation:

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.
05/07/2012, 19h50
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:

protected void download() throws IOException

par

Code:

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

Citation:

Envoyé par ZwQuery

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:

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();