de download Android problemas de arquivo binário

Question

Estou tendo problemas para baixar um arquivo binário (vídeo) em meu aplicativo da internet. Em Quicktime, Se eu baixá-lo diretamente funciona bem, mas através do meu aplicativo de alguma forma obter do desarrumada (mesmo que eles parecem exatamente o mesmo em um editor de texto). Aqui está um exemplo:

    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();

Answer 1

Eu não sei se é o único problema, mas você tem uma falha de Java clássico lá: Você não está contando com o fato de que read () é sempre permissão para retornar menos bytes do que você pedir. Assim, sua leitura poderia receber menos do que 1.024 bytes, mas a sua escrita sempre escreve exatamente 1024 bytes possivelmente incluindo bytes de iteração do loop anterior.

Corrigir com:

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

Talvez a rede maior latência ou tamanhos de pacotes menores de 3G no Android estão a exacerbar o efeito?

Answer 2

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

Answer 3

Um problema é a sua leitura do buffer. Se cada leitura do fluxo de entrada não é um múltiplo exato de 1024 você vai copiar dados errados. Use:

byte[] buffer = new byte[1024];
int len1 = 0;
while ( (len1 = in.read(buffer)) != -1 ) {
  f.write(buffer,0, len1);
}

Answer 4

 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.");
    } 
}

Answer 5

I fixa o código com base em feedbacks anteriores sobre esta discussão. Eu testei usando o eclipse e vários arquivos grandes. Ele está funcionando bem. Só tem que copiar e colar este ao seu ambiente e alterar o caminho http ea localização que você gostaria que o arquivo a ser baixado para.

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();
}

Boa sorte Alireza Aghamohammadi

Answer 6

Apenas usar o Apache do método de cópia ( Apache Commons IO ) - a vantagem de usar Java !

IOUtils.copy(is, os);

Não se esqueça de fechar as correntes em um bloco finally:

try{
      ...
} finally {
  IOUtils.closeQuietly(is);
  IOUtils.closeQuietly(os);
}