より効率的にURLから画像をダウンロードする

Question

状況：ArrayList<String>には画像へのリンクが多数含まれています（http：/www.foo.com/bar/image1.jpg、http：/www.foo.com/bar/image2 .PNG、...など）

私はそれらを一つずつダウンロードするために、コードの作業部分を発見した：

public void run() { 
     try { 
      int counter = 1; 
      for (String image : imagesList) { 
       controller.setDownloadStatusTextArea("Downloading image " + counter + " of " + imagesList.size()); 
       URL u = new URL(image); 
       URLConnection uc = u.openConnection(); 
       String contentType = uc.getContentType(); 
       int contentLength = uc.getContentLength(); 
       InputStream raw = uc.getInputStream(); 
       InputStream in = new BufferedInputStream(raw); 
       byte[] data = new byte[contentLength]; 
       int bytesRead; 
       int offset = 0; 
       while (offset < contentLength) { 
        bytesRead = in.read(data, offset, data.length - offset); 
        if (bytesRead == -1) 
         break; 
        offset += bytesRead; 
       } 
       in.close(); 
       if (offset != contentLength) { 
        throw new IOException("Only read " + offset + " bytes; Expected " + contentLength + " bytes"); 
       } 
       String[] tmp = image.split("/"); 
       String filename = tmp[tmp.length - 1]; 
       FileOutputStream out = new FileOutputStream(filename); 
       out.write(data); 
       out.flush(); 
       out.close(); 
       counter++; 
      } 
      controller.setDownloadStatusTextArea("Download complete"); 
     } catch (Exception ex) { 
      controller.setDownloadStatusTextArea("Download failed"); 
     } 
    } 

これは私がJavaでこのような何かをやって初めてですそして、私はこのコードがforループの外でたくさんの変数を動かすことによってはるかに効率的であると感じています。しかし、機能性やパフォーマンスに悪影響を及ぼすことなく、どちらが安全に外に移動できるかはわかりません（どちらも否定的または肯定的な方法で）。このような状況の洞察は非常に高く評価されます。 また、ファイルのダウンロード先を指定することはできますか？今はプロジェクトフォルダに表示されているので、ユーザは自分のダウンロードフォルダを変更できるようにしたいと思う。

ありがとうございます。

Answer 1

このコードは、はるかに時間効率を上げることはできません。

このように考えてみましょう。最後の不要なオペコードをすべて研磨しても、JVMがこのコード部分を実行するのにかかる時間はまったく重要ではありません。実際の遅延は、データがネットワークを介して到着するのを待つことになります。

もっとスペース効率がよいかもしれませんが、それは必要ではないと思います。

編集：あなたはスレッドを使用して、複数の画像を同時にダウンロードされる何ができますか。しかし、上のコードが複雑に見える場合、私はそれに反対します：言語の周りに自分の道を学ぶためにもう少し時間をかけてください。

Answer 2

ユーザーがディレクトリを選択できるようにするSwingアプリケーションでは、setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY)でJFileChooserをインスタンス化します。

各繰り返しで正しく初期化されているかぎり、すべての変数宣言をループ外に移動できます。あなたは、ファイルをダウンロードして保存するのにかかる時間に比べて、多くの時間を節約しません。

Answer 3

イメージ全体にバイト配列を割り当てる必要はありません。小さいバッファのみが必要です。 8kB。 次に、接続から8 KBを読み取り、FileOutputStreamにループで書き込みます。

コード全体を簡単にする（ループを蹴る）には、たとえば次のようなコードを使用できます。 Commons-IO （全体のjavadocを表示するには、「フレーム」リンクをクリックしてください）。