Discussion :

[FFF]

Bonjour,

Je cherche un moyen de télécharger un fichier se trouvant sur un serveur web.
J'ai essayé ceci, mais le fichier que j'obtiens en local prends bien plus de place que la source, je comprends pas trop... :

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
    public boolean download(String lien,String fichier) {
        try{
        URL web = new URL(lien);
        BufferedInputStream in=new BufferedInputStream(web.openStream());
        FileOutputStream fos=new FileOutputStream(dossier+fichier,true);
        byte buff[]=new byte[10000];
        while (( in.read(buff)) != -1)
            fos.write(buff);
 
        fos.close();
        }
        catch (IOException io) {System.out.print(io);return false;}
        System.out.print("Téléchargement réussi du fichier "+lien+"\n");
        return true;
    }

merci d'avance !

[bouye]

Regarde plutot du cote de la classe java.net.HttpURLConnection.

[FFF]

Je viens de jeter un coup d'oeil, mais je ne vois pas ce qui me permet de récupérer un fichier en binaire avec cette classe.[/quote]

[FreshVic]

Je viens de jeter un coup d'oeil, mais je ne vois pas ce qui me permet de récupérer un fichier en binaire avec cette classe.

[/quote]
Bonjour,
Pourtant HttpUrlCOnnection permet d'ouvrir une connexion Http sur une url et d'ouvrir un flux entrant et un flux sortant !

DOnc en recuperant le contenue du flux entrant tu devrais avoir ton fichier !

[FFF]

Ben dis moi comment faire alors stp, je vois pas comment.
Je viens de faire ça mais j'ai toujours un fichier trop gros, mais je ne vois pas où est l'erreur :

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
    public boolean download(String lien) {
        try{
            URL web = new URL(lien);
            URLConnection connexion=web.openConnection();
            InputStream in=new BufferedInputStream(connexion.getInputStream());
            String FileName = web.getFile( ); 
            int FileLenght = uc.getContentLength( );
            FileName = FileName.substring(FileName.lastIndexOf('/') + 1);         
            FileOutputStream fos=new FileOutputStream(dossier+"\\tempfile\\"+FileName,true);
            byte buff[]=new byte[10000];
            while (( in.read(buff)) != -1)
                fos.write(buff);
 
            in.close();
            fos.flush();
            fos.close();
        }
        catch (IOException io) {System.out.print(io);return false;}
        System.out.print("Téléchargement réussi du fichier "+lien+"\n");
        return true;
    }

[adiGuba]

Salut,


Avant tout quelques remarques sur le code :

• Tu ne fermes pas tous tes fluxs, et il est préférable de le faire dans un bloc finally...
• L'utilisation de BufferedInputStream est inutile puisque tu utilises ton propre buffer.
• Il faut que tu récupère le code de retour du read() pour le passer au write(), sinon lors du dernier passage tu écris des caractères en plus : si tu lis seulement 10 bytes, write() écrira quand même tous le tableau... --> 9990 bytes de trop (ton problème vient surement de là)

Ensuite il n'est pas obligatoire d'utiliser directement HttpUrlCOnnection puisque c'est ce qui est utilisée (indirectement) par la méthode openStream()...


Cela peut donner :

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
public boolean download(String lien,String fichier) {
	InputStream webStream = null;
	FileOutputStream fos = null;
	try{
		URL web = new URL(lien);
		webStream = web.openStream();
		fos=new FileOutputStream(dossier+fichier,true);
 
		byte buff[]=new byte[10000];
		int readCount = webStream.read(buff);
		while (readCount != -1) {
			fos.write(buff, 0, readCount);
			readCount = webStream.read(buff);
		}
	}
	catch (IOException io) {
		System.out.print(io);
		return false;
	}
	finally {
		tryToClose(webStream);
		tryToClose(fos);
	}
	System.out.print("Téléchargement réussi du fichier "+lien+"\n");
	return true;
}

Avec tryToClose() comme cela :

Code :

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

1
2
3
4
5
6
7
8
9
public void tryToClose(Closeable closeable) {
	if (is!=null) {
		try {
			closeable.close();
		} catch (IOException lException) {
			// ignore
		}
	}
}

L'interface Closeable n'est apparut qu'avec Java 5.0, donc si tu ne l'utilises pas il faudra la coder deux fois (avec un InputStream et un OutputStream).


a++

[FFF]

Merci adiGuba !!!!
Réponse concise efficace, on ne peut plus complète !
Effectivement je pensais à tors que seuls les octets non nuls étaient écris dans le flux de sortie (ce qui marchait dans mon cas avec des fichiers, ne marche pas forcément pour un autre flux, du coup je vais aussi corriger cette erreur pour mes FileInpuStream/OutputStream sauf si qqun me dis que ce n'est pas nécessaire).

Ton code marche impec et j'ai pu corrigé mon erreur !
Utilisant Tiger c'est super !

Merci encore.