Android Studio：线程应用程序返回的值与非线程应用程序不同

问题描述

我正在开发一个应用程序，它可以浏览所有电话目录并收集所有歌曲。 当我正常运行它时，它工作正常，只需大约 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);
}

即使我明确指定不会将空对象添加到列表中，它在保存歌曲时也能正常运行，但在尝试读取时会出现空错误。 每次我运行不同的歌曲并且不同数量的歌曲被设置为空。

请帮忙。

解决方法

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

这不是线程会加速太多的东西，无论如何你做的线程都是错误的。把它扔掉，只在一个后台线程上做，不要在那个线程上加入（或者你也可以按顺序做） - 完成后让它回发到主线程。