Discussion :

[judas54]

Bonjour,

Suite a ce 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
       JFileChooser jfc = new JFileChooser();
        jfc.showOpenDialog(null);
        File f=jfc.getSelectedFile();
        if(f==null) System.exit(0);
 
        BufferedInputStream b=new BufferedInputStream(new FileInputStream(f));
        byte tab[] =new byte[1024];
 
        BufferedOutputStream bs = new BufferedOutputStream(new FileOutputStream(new File(f.getName())));
 
        while((b.read(tab,0,tab.length))>tab.length){
 
            bs.write(tab,0,tab.length);
 
        }

J'ai un fichier qui fait 0 octet, pouvez vous me dire qu'est ce qui ne va pas dans mon code ?

Merci d'avance

[Pill_S]

Il manque les flush et les fermeture des flux...

[judas54]

Oui effectivement, j'avais oublié les flush et close, cependant voici le nouveau code, et j'ai toujours le meme resultat, un fichier de 0ko

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
        JFileChooser jfc = new JFileChooser();
        jfc.showOpenDialog(null);
        File f=jfc.getSelectedFile();
        if(f==null) System.exit(0);
 
        BufferedInputStream b=new BufferedInputStream(new FileInputStream(f));
        byte tab[] =new byte[1024];
        int emplacement = 0;
 
        BufferedOutputStream bs = new BufferedOutputStream(new FileOutputStream(new
File(f.getName())));
 
        while((b.read(tab,emplacement,tab.length))>tab.length){
 
            bs.write(tab,emplacement,tab.length);
            bs.flush();
        }
        b.close();
        bs.close();

[judas54]

Ok je répond a ma betise: le test sur la condition était faux: c'est pas

Code :

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

> tab.length

mais

Code :

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

>= tab.length

Merci en tout cas pour le close et flush, il est vrai que j'ai pas trop l'habitude de m'anipuler les flux

[Pill_S]

ici:

Code :

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

while((b.read(tab,emplacement,tab.length))>tab.length){

je remplacerais par

Code :

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

while((b.read(tab,emplacement,tab.length))>0)

PS: ce qui se traduit par "tout pendant qu'il y a des bytes à lire"

Mais ton code à d'autres problèmes... Il manque une variable pour stocker le nombre de bytes qui viennent d'être lus

[Napalm51]

Et de plus si le fichier sélectionné est dans le répertoire de ton application, alors tu vas l'écraser en essayant de le copier, car tu utilises son nom (f.getName()) ...

[judas54]

Merci de vos conseils, j'ai vu effectivement dans l'api que read renvoit -1 à la fin du fichier.