0

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

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

请帮忙。

4

1 回答 1

1

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

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

于 2021-05-27T23:15:59.267 回答