Android 工作室:线程应用程序未返回与非线程应用程序相同的值

Android studio: thread app not returning same value as non thread app

我正在开发一个遍历所有 phone 目录并收集所有歌曲的应用程序。 当我 运行 它通常工作正常,只需要大约 6 秒来完成所有事情,并导致应用程序跳过很多帧。 我改变了它,所以每次找到一个文件时,一个不同的线程读取元数据并保存它。 最后我也在等待所有这些,因为之后我会尝试使用该列表。 突然间,有几首歌是空的,即使它们在任何地方都没有被初始化。 是什么原因造成的?一个运行良好但不支持线程的应用程序..?

调用搜索的构造函数:

phoneSongsList = new Playlist();
findSongs(Environment.getExternalStorageDirectory().getAbsolutePath()); //.concat("/Music")
for (Thread thread : threads) {
    try {
        thread.join();
    } catch (InterruptedException e) {
        e.printStackTrace();
    }
}

递归查找歌曲的函数:

public void findSongs(String path) {
    File home = new File(path);
    for (final File file : home.listFiles()) {
        if (file.isDirectory())
            findSongs(path.concat("/" + file.getName()));
        else if (isAcceptableExtension(file.getName())) {
            Thread t = new Thread(new Runnable() {
                @Override
                public void run() {
                    phoneSongsList.add(fileToSong(file));
                }
            });
            t.start();
            threads.add(t);
        }
    }
}

将文件转换为歌曲对象的函数:

private Song fileToSong(File file) {
    final Album album = new Album();
    Song song = new Song();
    song.setName(file.getName().substring(0, (file.getName().length() - 4))); // remove suffix
    song.setPath(file.getPath());

    final MediaMetadataRetriever metaRetriever = new MediaMetadataRetriever();
    metaRetriever.setDataSource(file.getPath());

    song.setArtists(metaRetriever.extractMetadata(MediaMetadataRetriever.METADATA_KEY_ARTIST));
    album.setName(metaRetriever.extractMetadata(MediaMetadataRetriever.METADATA_KEY_ALBUM));
    album.setYear(metaRetriever.extractMetadata(MediaMetadataRetriever.METADATA_KEY_YEAR));
    album.setCover(metaRetriever.getEmbeddedPicture(), context);
    song.setAlbum(album);
    song.setDuration(Long.parseLong(metaRetriever.extractMetadata(MediaMetadataRetriever.METADATA_KEY_DURATION)));
    song.setGenre(metaRetriever.extractMetadata(MediaMetadataRetriever.METADATA_KEY_GENRE));

    metaRetriever.release();
    return song;
}

Playlist.add函数:

public void add(Song song) {
    add(list.size(), song);
}
public void add(int index, Song song) {
    if(song==null)
        return;
    if (index > list.size())
        index = list.size();
    if (list.contains(song))
        list.remove(song);
    list.add(index, song);
}

即使我明确指定不会将空对象添加到列表中,它在保存歌曲时 运行 没问题,但在尝试读取时会出现空错误。 每次i 运行 不同的歌曲和不同数量的歌曲被设置为null。

请帮忙

您正动态地尝试将新线程添加到其他线程的线程列表中,但在一个线程上读取该线程。这意味着您将在添加所有线程之前完成这些线程的某些子集上的循环。整个方法是一个大的竞争条件。

这不是线程会加速多少的东西,而且你在做线程时都是错误的。把它扔掉,只在一个后台线程上做,不要加入那个线程(或者你也可以按顺序做)——完成后让它post回到主线程。