HTTP Download einer Datei mit "Statistiken"

[AdamMyth]

Hallo,

ich möchte eine Datei von einem HTTP-Server downloaden. Den nötigen Source-Code habe ich mir bereits zusammengebastelt. Siehe hier:

		URL url = new URL("http://downloads.sourceforge.net/project/virtualboximage/OpenSolaris/2008.05/opensolaris-2008.05-x86.7z?use_mirror=freefr");
		
		URLConnection uc = url.openConnection();
		uc.connect();
		
		long totalread = 0, contentLength = uc.getContentLength();
		FileOutputStream out = new FileOutputStream(new File("C:/out.dat"));
		InputStream in = uc.getInputStream();
		int BUF_SIZE = 1 << 8, bytesRead = -1;
		byte[] buffer = new byte[BUF_SIZE];
		
		System.out.println("Laenge: " + contentLength);
		
		long last = -1;
		long down = -1;
		
		while ((bytesRead = in.read(buffer)) > -1) {
			out.write(buffer, 0, bytesRead);
			totalread += bytesRead;
			
			
			if (last < 0 && down < 0) {
				last = System.currentTimeMillis();
				down += bytesRead;
			} else {
				long diff = System.currentTimeMillis() - last;
				if (diff >= 1000) {
					last = -1;
					
					System.out.println(down + " in " + diff);
					System.out.println((totalread/contentLength)*100 + "%");
					System.out.println(down/diff);
					System.out.println("----------");
					down = -1;
				}
			}
			
		}
		in.close();
		out.close();

Nun, das mit dem runterladen klappt bereits problemlos. Doch ich habe Probleme mit den Statistiken zu dem Download. Denn ich will später eine Aussage machen, wie "129 kb/s", doch irgendwie geht das nicht ....

Könnt Ihr mir sagen, was ich falsch mache und wie ich das richtig mache??? Bittte ...

Gruß,
Adam

 

[SlaterB]

ein Problem ist, dass du in Zeile 24 nur maximal eine Byte-Ladung addierst,
danach wird immer der else-Fall durchlaufen, bis 1000ms passiert gar nix und dann hast du immer ~255/1000 was in Java bei Ganzzahlrechnung immer 0 ergibt,
deshalb geht auch die Prozent-Anzeige nicht

hier ein schöneres Beispiel, gleich mit Dummy-Datenquelle:

public class Test
{
    public static void main(String[] args) throws Exception {
        final long contentLength = 200000;
        long totalread = 0;
        InputStream in = new InputStream()
            {
                int c = 0;
                
                @Override
                public int read() throws IOException {
                    c++;
                    if (c >= contentLength)
                    {
                        return -1;
                    }
                    if (Math.random() < 0.002)
                    {
                        try
                        {
                            Thread.sleep(20);
                        }
                        catch (InterruptedException e)
                        {
                            e.printStackTrace();
                        }
                    }
                    return 0;
                }
            };
        int BUF_SIZE = 1 << 8, bytesRead = -1;
        byte[] buffer = new byte[BUF_SIZE];

        System.out.println("Laenge: " + contentLength);

        long last = System.currentTimeMillis();
        long down = -1;

        while ((bytesRead = in.read(buffer)) > -1)
        {
            totalread += bytesRead;
            down += bytesRead;
            long diff = System.currentTimeMillis() - last;
            if (diff >= 1000)
            {
                System.out.println(down + " in " + diff + ", " + totalread);
                System.out.println((totalread * 100.0 / contentLength) + "%"); // DecimalFormat kann schön formatieren
                System.out.println(down / diff + " kb/s"); // wenn down < diff sein kann, dann besser auch * 1.0
                System.out.println("----------");
                down = -1;
                last = System.currentTimeMillis();
            }
        }
        in.close();
        System.out.println("end " + (System.currentTimeMillis() - last));
    }
}

 

[Michael...]

was geht nicht?
Daraus

(totalread/contentLength)*100

würde ich

totalread*100/contentLength

machen, sonst hast Du immer 0%. Denn int/int = int

Dein down wird nie grösser als 256 (bytesRead) da Du es nur im ersten if Zweig erhöhst. Sollte es nicht unabhängig von der Bedingung erhöht werden und nur nach der Ausgabe wieder zurückgesetzt werden?

Und ungefähr jede Sekunde eine Ausgabe zu machen... Bin mir nicht sicher, ob das nicht die Aussage verfälscht, da die Ausgaben evtl. länger dauern als den Stream zu lesen und zu schreiben.