HTTP Problem beim Download von einer Datei

[punkt21]

Hallo Leutz.

hab folgendes Problem:
ich will eine Datei deren URL ich habe downloaden. Die Datei wird erstellt auch mit richtiger Endung und der Name stimmt ebenfalls. Aber die Datein sind permanent 0kb groß und leer. Ich weiß aber nicht warum.

Hier mein Code:

public static void downloadFile(URL url, FileOutputStream os)
			throws IOException {
		if (url == null)
			return;
		HttpURLConnection conn = (HttpURLConnection)url.openConnection();
		conn.setRequestMethod("GET");
		conn.connect();
		int responseCode = conn.getResponseCode();
		
		if (responseCode == HttpURLConnection.HTTP_OK) {
			byte tmp_buffer[] = new byte[4096];
			InputStream is = conn.getInputStream();
			int len;
			while ( (len = is.read(tmp_buffer)) != -1) {
				os.write(tmp_buffer, len, 0);
			}
			os.flush();
		} else {
			throw new IllegalStateException("HTTP Response: " + responseCode);
		}
	}

danke schonmal

 

[Kaffeebohn]

Hast du die Streams am ende auch geschlossen? Ansonsten wird die Datei nicht abgeschlossen und deswegen nicht richtig erkannt und mit 0kb dargestellt.

Also sowas in den finally block packen wie

			if ( is != null )         try { is.close(); } catch ( IOException e ) { } 
			if ( os != null )         try { os.close(); } catch ( IOException e ) { }

 

[Triebi]

	while ( (len = is.read(tmp_buffer)) != -1) {
		os.write(tmp_buffer, len, 0);
	}

Alle Streams zu schließen ist schon mal eine gute Idee.
Aber kann es sein, dass das byte[] tmp_buffer mit dem Offset len und der Länge 0 geschrieben wird?
FileOutputStream (Java Platform SE 6)

 

[Kaffeebohn]

Ich kann Triebi rechtgeben, habe es mit einer eigenen Schleife von mir verglichen. Genauer hinschauen lohnt sich offensichtlich

Hier meine schleife

byte[] buffer = new byte[ 0xFFFF ];
			for ( int len; (len = is.read(buffer)) != -1; )
			{
				os.write( buffer, 0, len );

			}

 

[punkt21]

danke! habs jetz am rennen^^

kann geclosed werden