4

我有一个适用于 Android 的应用程序,它可以从 Internet 下载数百个文件。有些文件在下载后变成 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

您应该在 FileOutputStream 上调用 flush() 以确保将所有数据写入文件。这应该使您的 0 字节文件问题发生的频率降低。

使用 File.length() 检查 0 字节文件应该可以正常工作。您能否在设备上打开一个外壳(adb shell)并运行 ls -l 以查看它显示的字节数为 0(可能您的文件管理器有一些奇怪的问题)。另外请调试(或放置一些日志语句) sFullPath 包含正确的文件路径。我看不到 sFullPath 在上面的代码中设置的位置,以及为什么您不只使用 outputFile 而是重新创建另一个 File 对象。

于 2013-07-08T17:49:49.847 回答
1

在我看来,您的问题是只有在Utils.downloadFile调用返回 true 时才检查“垃圾”文件。getInputStream如果调用或第一次下载失败read,您将创建一个长度为零的文件,该文件永远不会被删除。

于 2013-07-09T16:12:17.100 回答