Discussion :

[FlorentF]

Bonsoir,

J'ai fait un petit code pour pouvoir récupérer une page (style developpez.net) que je veux et la sauvegarder où je veux. Sauf qu'il y a un petit problème
Le programme s'execute correctement (c'est à dire que je peux me connecter et j'enregistre bien la page) En revanche, le programme ne s’arrête jamais. Comme s'il était pris dans une boucle infinie !

Quelqu'un aurait une petite idée du pourquoi du comment ?
Lien vers le code.

Merci d'avance !
Bonne soirée.

Edit:
De plus, il y a des nombres qui s'incrustent dans le fichier crée.. "1000" puis "0"

[thelvin]

'Faudrait que tu nous donnes l'URL en question, mais ça c'est le signe typique de la méthode "ma réponse ne se termine jamais, comme ça quand j'ai du nouveau à dire, j'ai qu'à le pousser dans la socket qui est encore ouverte."

Les chiffres au milieu, signes de chunked-encoding, en sont une preuve de plus.

Dans ce cas-là, c'est à toi de choisir quand tu estimes que tu as fini de recevoir la réponse, et fermer la socket de force, à ce moment-là.

Edit : Ah bah oui, tiens. Le keep-alive fonctionne comme ça. Je savais pas. Merci adiGuba.

[adiGuba]

Salut,



Déjà désolé mais le code n'est pas très propre

• Un bloc catch vide !!!!
• Plein de traitement dans un constructeur d'une classe qui ne comporte rien. Ce ne serait pas mieux de mettre tout cela dans une méthode static ?
• Il manque aussi le try/finally pour la fermeture des ressources...

Quand à ton problème, il vient du fait que tu ne respectes pas le protocole HTTP/1.1 contrairement à ce que tu annonces au serveur...
En HTTP/1.1 le fichier est tronqué en morceaux, avec une ligne indiquant la taille du morceau qu'il faut lire. Une taille de zéro indique la fin de la réponse mais la connexion reste ouverte pour une nouvelle demande (keep-alive).


Comme tu ne respectes pas cela tu te retrouves bloqué : le serveur attend une nouvelle demande de ta part, alors que toi tu attends le reste de la réponse (qui est déjà fini).


De deux choses l'une :

• soit tu prévoit d'implémenter toi-même le protocole HTTP/1.1, et dans ce cas il va falloir te plonger dans sa RFC pour être bien conforme.
• soit tu veux simplement télécharger le fichier et tu utilises la classe URL qui fait cela très bien :

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
	public static void download(URL url, File destination) throws IOException {
		InputStream input = url.openStream();
		try {
			OutputStream output = new FileOutputStream(destination);
			try {
				byte[] buffer = new byte[8192];
				int read;
 
				while ((read=input.read(buffer)) > 0) {
					output.write(buffer, 0, read);
				}
				output.flush();
			} finally {
				output.close();
			}
		} finally {
			input.close();
		}
	}

a++

[FlorentF]

Merci beaucoup pour votre aide !
Finalement, j'utilise la classe URL comme l'a conseillé adiGuba.