在 Android 上下载文件时未检测到 0 字节文件

Posted 2023-02-25

【中文标题】在 Android 上下载文件时未检测到 0 字节文件

【英文标题】：0-byte files not detected when downloading files on Android

【发布时间】：2013-06-28 07:21:32

【问题描述】：

我有一个适用于 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 字节。它们无法通过视频播放器播放（显示为“损坏的文件”图标）。

【问题讨论】：

文件真的可以是0字节吗？它不会总是有一些元数据等，它本身就存在吗？也许您需要更多地寻找诸如实际内容大小之类的东西？

其实我也会检查小视频文件来删除它们。 MIN_NON_GARBAGE_VIDEO_FILE_SIZE 设置为 100kb。所以，如果一个视频文件有一些小的元数据块，它肯定会低于 100kb。而且我仍然得到未删除的 0 字节视频文件。单独的元数据可以超过 100kb 吗？

【参考方案1】：

您应该在 FileOutputStream 上调用 flush() 以确保将所有数据写入文件。这应该会减少 0 字节文件的问题。

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

【讨论】：

谢谢！是的，我会添加flush（），但afaik close 总是在关闭之前调用flush（可能是错误的）。 sFullPath 是写入文件的完整路径。它将在外部存储（SD 卡）上。我正在创建另一个文件只是为了三次检查大小。我的意思是，我已经通过 outputFile.length() 获得了大小，但我的猜测是，关闭文件可能会发生一些延迟（？），这就是为什么如果我试图在操作系统使用的任何缓冲区的内容之后立即检查大小尚未写入磁盘。但是是的，大小总是> 100kb，新打开的大小也是> 0。疯狂的东西

【参考方案2】：

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

【讨论】：

确实如此。我犯了一个多么愚蠢的错误。这是一个很好的例子，当一个人不使用干净代码编码原则时会发生什么。这些方法太长了，而且文件是在一种方法中创建并传递给另一种方法的，所以我只是没有注意到所有的可能性。