4

インターネットから何百ものファイルをダウンロードする Android 用のアプリがあります。一部のファイルは、ダウンロード後に 0 バイトになることが判明しました。アプリはそのようなケースを検出し、ダウンロード後にそのようなファイルを削除しようとしますが、失敗する場合があります。この問題は、Android 4.x デバイスでより頻繁に発生します。

ダウンロードを行う方法は次のとおりです。から実際に読み取ったバイト数を取得しますinputStream.read(buffer)。

public class Utils
{
public static class DownloadFileData
{
    int nTotalSize;
    int nDownloadedSize;
}
public interface ProgressCallback
{
    void onProgress(long nCurrent, long nMax);
}
public static boolean downloadFile(String sFileURL, File whereToSave, DownloadFileData fileData, ProgressCallback progressCallback)
{
    InputStream inputStream = null;
    FileOutputStream fileOutput = null;
    try
    {
        URL url = new URL(sFileURL);
        URLConnection connection = url.openConnection();

    //set up some things on the connection
        connection.setDoOutput(true);
        connection.connect();

        fileOutput = new FileOutputStream(whereToSave);
        inputStream = connection.getInputStream();
        fileData.nTotalSize = connection.getContentLength();
        fileData.nDownloadedSize = 0;

            byte[] buffer = new byte[1024];
            int bufferLength = 0; //used to store a temporary size of the buffer

        // now, read through the input buffer and write the contents to the file
        while ((bufferLength = inputStream.read(buffer)) > 0)
        {
            // if interrupted, don't download the file further and return
            // also restore the interrupted flag so that the caller stopped also
            if (Thread.interrupted())
            {
                Thread.currentThread().interrupt();
                return false;
            }

            // add the data in the buffer to the file in the file output stream
            fileOutput.write(buffer, 0, bufferLength);
            // add up the size so we know how much is downloaded
            fileData.nDownloadedSize += bufferLength;

            if (null != progressCallback && fileData.nTotalSize > 0)
            {
                progressCallback.onProgress(fileData.nDownloadedSize, fileData.nTotalSize);
            }
        }
  return true;
    }
    catch (FileNotFoundException e)
    {
        return false; // swallow a 404
    }
    catch (IOException e)
    {
        return false; // swallow a 404
    }
    catch (Throwable e)
    {
        return false;
    }
    finally
    {
        // in any case close input and output streams
        if (null != inputStream)
        {
            try
            {
                inputStream.close();
                inputStream = null;
            }
            catch (Exception e)
            {
            }
        }
        if (null != fileOutput)
        {
            try
            {
                fileOutput.close();
                fileOutput = null;
            }
            catch (Exception e)
            {
            }
        }
    }
}

ダウンロードを処理するコードは次のとおりです。読み取りバイト数が正しくない場合があるため (> 0 で、実際のファイルのサイズは 0 バイトです)、ダウンロードしたファイルのサイズを で確認しますoutputFile.length()。しかし、ファイルが実際には 0 バイトであっても、これは値 > 0 を返します。また、新しいファイルを作成して、そのサイズを で読み取ろうとしましたrecheckSizeFile.length()。実際には 0 バイトですが、それでもサイズは > 0 と判断されます。

Utils.DownloadFileData fileData = new Utils.DownloadFileData();
boolean bDownloadedSuccessully = Utils.downloadFile(app.sCurrenltyDownloadedFile, outputFile, fileData, new Utils.ProgressCallback()
                    {
... // progress bar is updated here
});

if (bDownloadedSuccessully)
{
    boolean bIsGarbage = false;
    File recheckSizeFile = new File(sFullPath);
    long nDownloadedFileSize = Math.min(recheckSizeFile.length(), Math.min(outputFile.length(), fileData.nDownloadedSize));
        // if the file is 0bytes, it's garbage
    if (0 == nDownloadedFileSize)
    {
        bIsGarbage = true;
    }
    // if this is a video and if of suspiciously small size, it's
    // garbage, too
    else if (Utils.isStringEndingWith(app.sCurrenltyDownloadedFile, App.VIDEO_FILE_EXTENSIONS) && nDownloadedFileSize < Constants.MIN_NON_GARBAGE_VIDEO_FILE_SIZE)
    {
        bIsGarbage = true;
    }
    if (bIsGarbage)
    {
        ++app.nFilesGarbage;
        app.updateLastMessageInDownloadLog("File is fake, deleting: " + app.sCurrenltyDownloadedFile);
        // delete the garbage file
        if (null != outputFile)
        {
            if (!outputFile.delete())
            {
                Log.e("MyService", "Failed to delete garbage file " + app.sCurrenltyDownloadedFile);
            }
        }
    }
    else
    {
        ... // process the normally downloaded file
    }

よくわかりませんが、Android にはファイル サイズの読み取りに関するバグがあると思います。誰かが同様の問題を見たことがありますか？それとも、ここで何か間違ったことをしているのですか？ありがとう！

編集：ファイルが0バイトであることをどのように判断するか：ダウンロードされるすべてのファイルは、説明されているルーチンを通過します。その後、ファイル ブラウザー (Ghost Commander) でダウンロード フォルダーを表示すると、一部のファイル (おそらく 10% など) が 0 バイトです。ビデオ プレーヤーで再生することはできません (「壊れたファイル」アイコンとして表示されます)。

4

2 に答える 2

1

あなたの問題は、Utils.downloadFile呼び出しがtrueを返した場合にのみ「ガベージ」ファイルをチェックすることです。getInputStream呼び出しまたは最初の でダウンロードが失敗した場合、read削除されない長さ 0 のファイルが作成されます。

于 2013-07-09T16:12:17.100 に答える