web-dev-qa-db-ja.com

Androidバイナリファイルのダウンロードの問題

アプリからインターネットからバイナリファイル（ビデオ）をダウンロードするときに問題が発生します。 Quicktimeでは、直接ダウンロードすると正常に機能しますが、アプリを使用すると、テキストエディタでまったく同じように見えても、どういうわけか混乱します。以下に例を示します。

    URL u = new URL("http://www.path.to/a.mp4?video");
    HttpURLConnection c = (HttpURLConnection) u.openConnection();
    c.setRequestMethod("GET");
    c.setDoOutput(true);
    c.connect();
    FileOutputStream f = new FileOutputStream(new File(root,"Video.mp4"));


    InputStream in = c.getInputStream();

    byte[] buffer = new byte[1024];
    int len1 = 0;
    while ( (len1 = in.read(buffer)) > 0 ) {
         f.write(buffer);
    }
    f.close();
javaandroiddownloadhttpurlconnectionfileoutputstream
71
Isaac Waller

それが唯一の問題かどうかはわかりませんが、古典的なJavaグリッチがあります。read（）が常にリクエストより少ないバイト数を返すことができるため、読み取りは1024バイト未満になる可能性がありますが、書き込みは常に前のループ反復からのバイトを含めて常に1024バイトを書き出します。

修正：

 while ( (len1 = in.read(buffer)) > 0 ) {
         f.write(buffer,0, len1);
 }

Android=上の3Gのより高いレイテンシネットワーキングまたはより小さいパケットサイズが、効果を悪化させているのでしょうか？

93
Ry4an Brase
new DefaultHttpClient().execute(new HttpGet("http://www.path.to/a.mp4?video"))
        .getEntity().writeTo(
                new FileOutputStream(new File(root,"Video.mp4")));
28
njzk2

1つの問題は、バッファーの読み取りです。入力ストリームのすべての読み取りが1024の正確な倍数でない場合、不良データをコピーします。つかいます：

byte[] buffer = new byte[1024];
int len1 = 0;
while ( (len1 = in.read(buffer)) != -1 ) {
  f.write(buffer,0, len1);
}
16
Clint
 public class download extends Activity {

     private static String fileName = "file.3gp";
     private static final String MY_URL = "Your download url goes here";

     @Override
     public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        try {
            URL url = new URL(MY_URL);
            HttpURLConnection c = (HttpURLConnection) url.openConnection();
            c.setRequestMethod("GET");
            c.setDoOutput(true);
            c.connect();

            String PATH = Environment.getExternalStorageDirectory()
                + "/download/";
            Log.d("Abhan", "PATH: " + PATH);
            File file = new File(PATH);
            if(!file.exists()) {
               file.mkdirs();
            }
            File outputFile = new File(file, fileName);
            FileOutputStream fos = new FileOutputStream(outputFile);
            InputStream is = c.getInputStream();
            byte[] buffer = new byte[1024];
            int len1 = 0;
            while ((len1 = is.read(buffer)) != -1) {
                fos.write(buffer, 0, len1);
            }
            fos.flush();
            fos.close();
            is.close();
        } catch (IOException e) {
            Log.e("Abhan", "Error: " + e);
        }
        Log.i("Abhan", "Check Your File.");
    } 
}
14
user456118

このスレッドに関する以前のフィードバックに基づいてコードを修正しました。 Eclipseと複数の大きなファイルを使用してテストしました。正常に動作しています。これをコピーして環境に貼り付け、httpパスとファイルのダウンロード先を変更するだけです。

try {
    //this is the file you want to download from the remote server
    String path ="http://localhost:8080/somefile.Zip";
    //this is the name of the local file you will create
    String targetFileName
        boolean eof = false;
    URL u = new URL(path);
    HttpURLConnection c = (HttpURLConnection) u.openConnection();
    c.setRequestMethod("GET");
    c.setDoOutput(true);
    c.connect();
    FileOutputStream f = new FileOutputStream(new File("c:\\junk\\"+targetFileName));
        InputStream in = c.getInputStream();
        byte[] buffer = new byte[1024];
        int len1 = 0;
        while ( (len1 = in.read(buffer)) > 0 ) {
        f.write(buffer,0, len1);
                 }
    f.close();
    } catch (MalformedURLException e) {
    // TODO Auto-generated catch block
    e.printStackTrace();
    } catch (ProtocolException e) {
    // TODO Auto-generated catch block
    e.printStackTrace();
    } catch (FileNotFoundException e) {
    // TODO Auto-generated catch block
    e.printStackTrace();
    } catch (IOException e) {
    // TODO Auto-generated catch block
    e.printStackTrace();
}

がんばってアリレザ・アガモハマディ

3
grepit

Apacheのコピーメソッドを使用するだけです（ Apache Commons IO ）-Javaを使用する利点！

IOUtils.copy(is, os);

Finallyブロックでストリームを閉じることを忘れないでください：

try{
      ...
} finally {
  IOUtils.closeQuietly(is);
  IOUtils.closeQuietly(os);
}
2
AlikElzin-kilaka