Android图库应用批量移图异常：AsyncTask循环仅执行一次

云上老周

2026-5-15

修复多选图片移动仅执行一次的问题

嘿，我一眼就揪出你代码里的问题了——你的AsyncTask的doInBackground()方法里，循环刚跑第一次就直接return了，这直接终止了整个方法，后面的图片根本没机会被处理！

核心问题分析

在LongOperation的doInBackground()中，你在try块里写了return destinationImage;，这意味着第一次循环处理完一张图片后，方法就直接结束了，循环自然只执行一次，剩下的选中图片完全没被遍历到。

修复步骤

移除循环内的return语句
把return destinationImage;从循环里彻底删掉，或者如果需要返回结果，就等所有图片处理完后再统一返回（其实移动操作不需要返回单个文件，把AsyncTask的返回类型改成Void更合理）。
修正MediaScanner的上下文
你的moveFile方法里，MediaScannerConnection.scanFile(this, ...)中的this指向的是AsyncTask实例，不是Activity上下文，这会导致媒体扫描失败，要改成ImageGallery.this。
确认图片路径获取逻辑
检查PhotosActivity里的getImagePaths方法，al_images.get(int_position)中的int_position是不是当前选中文件夹的正确索引，如果这个值不对，可能导致你传递的图片路径本身就有问题，后续移动自然也会出问题。

修改后的关键代码

修正后的LongOperation类

private class LongOperation extends AsyncTask<Void, Void, Void> { // 把返回类型改成Void更贴合需求
    private int targetFolderIndex;

    // 通过构造函数传入目标文件夹索引，避免依赖外部变量的不确定性
    public LongOperation(int folderIndex) {
        this.targetFolderIndex = folderIndex;
    }

    @Override
    protected Void doInBackground(Void... params) {
        for (String imagePath : selectedImages) {
            File sourceImage = new File(imagePath);
            File destinationImage = new File(
                al_images.get(targetFolderIndex).getDirectoryPath() 
                + File.separator 
                + sourceImage.getName()
            );
            try {
                moveFile(sourceImage, destinationImage, false);
                // 这里绝对不能写return！让循环跑完所有图片
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
        return null;
    }

    private void moveFile(File file_Source, File file_Destination, boolean isCopy) throws IOException {
        FileChannel source = null;
        FileChannel destination = null;
        if (!file_Destination.exists()) {
            file_Destination.createNewFile();
        }
        try {
            source = new FileInputStream(file_Source).getChannel();
            destination = new FileOutputStream(file_Destination).getChannel();
            long count = 0;
            long size = source.size();
            while ((count += destination.transferFrom(source, count, size - count)) < size) ;
            if (!isCopy) {
                file_Source.delete();
                // 修正上下文为ImageGallery.this，确保媒体扫描正常工作
                MediaScannerConnection.scanFile(ImageGallery.this, 
                    new String[] { file_Source.getAbsolutePath() }, 
                    null,
                    new MediaScannerConnection.OnScanCompletedListener() {
                        public void onScanCompleted(String path, Uri uri) {
                            Log.i("ExternalStorage", "Scanned " + path + ":");
                            Log.i("ExternalStorage", "-> uri=" + uri);
                        }
                    });
            }
        } finally {
            if (source != null) {
                source.close();
            }
            if (destination != null) {
                destination.close();
            }
        }
    }
}

调用AsyncTask的对应修改

// 传入选中的目标文件夹索引i
new LongOperation(i).execute();

额外建议

另外，AsyncTask在Android API 30之后已经被官方废弃了，如果你想让代码更稳定、适配性更好，可以考虑改用Kotlin协程或者Java ExecutorService来处理后台任务，避免后续版本的兼容性问题。

内容的提问来源于stack exchange，提问作者Amelia

Android图库应用批量移图异常：AsyncTask循环仅执行一次

核心问题分析 #

修复步骤 #

修改后的关键代码 #