swift TableViewにMPMediaQueryからデータを取得しすぎてしまう・・・

Question

###前提・実現したいこと
以下のコードで、MPMediaQueryからアーティストを取り出して、テーブルビューに入れているのですが、このように、同じアーティストを曲の数だけ繰り返してしまうのです・・・

コードは、

func getArtistInfo() -> Array<MPMediaItem> {
        var array = Array<MPMediaItem>()
        let artistQuery: MPMediaQuery = MPMediaQuery.artists()
        let artistCollections: [MPMediaItemCollection] = artistQuery.collections!
        for artist in artistCollections {
            let artistItems: [MPMediaItem] = artist.items
            for artists in artistItems {
                array.append(artists)
            }
        }
        return array
    }
//これで取得して、
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return data.count
    }

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = UITableViewCell(style: UITableViewCellStyle.default, reuseIdentifier: "myCell")
        cell.textLabel?.text = data[indexPath.row].artist
        return cell
    }
//これでテーブルに入れています。

###試したこと
クエリ周りの記述は本を見て丸写ししたものをちょっとアレンジをしただけなので、
あまり理解が進んでいません・・・
MediaQueryが曲ごとに各.artists()やら.albums()を参照しているからなのだろうとは思うのですが、OrderedSetなどで重複を削除しようにも、どうしたらいいのか・・・

###補足情報(言語/FW/ツール等のバージョンなど)
Xcode8 swift3

　
###追記　2017.12.6
階層の解釈として、

MPMediaQuery
↓
.artists() ← アーティストで分けますというメソッド
↓
.collections ← 各アーティストがどのような曲(items)を持っているのかという情報
↓
.items ← 各曲の詳細を内包した情報
↓
.artistなど ←各曲の情報

これであっているでしょうか？

つまり、上記コードでは
アーティストの数だけfor文が回されているだけでなく、曲の数だけfor文が回されるのがアーティストの数だけ繰り返されていた

ということですかね？

Comments

[fuzzball]

let artistItems: [MPMediaItemCollection] = artistQuery.collections! の artistItems.count はアーティスト数になってますか？

[mitci]

print(artistItems.count)を入れて実行したところ、アーティスト数になっていました・・・
ということは、それ以降のコードに不備があるということでしょうか？

あ、ちなみに、
for songs in artistItems {
array.append(songs)
}
となっていたために、
songsをartistsに直しました。もちろん、結果は変わらないですが・・・

[fuzzball]

こちらで試せないので想像で書きますが、artist.items でそのアーティストの曲が全て取り出されているのではないでしょうか？内側のartistItems（紛らわしいので名前変えた方がいいですよ）のcountを調べてみて下さい。そうであれば、結局すべての曲（MPMediaItem）からアーティストを抜き出していることになります。そうではなくて、1つのMPMediaItemCollectionから1つのアーティストを抜き出さないといけません。MPMediaItemCollectionについてもう少し調べてみて下さい。

[mitci]

すいません。本を見ながらやったのでcollectionとitemがごちゃごちゃになっていました・・・。内側のartistItemsですが、総曲数でもアーティスト数でもない中途半端な数値になっていました。MPMediaItemCollectionとMPMediaItemの違いというか、上下関係もよくわかっておらず（参考にした本はこのコードをとりあえず書いておけ、という説明でした・・・）とりあえず、5時間くらい考えて調べてもわからなかったのですが、ちょっと追記をしてみたので読んでいただけるとありがたいです。

[mitci]

なんとか解決にいたりました。ちょっと強引な気もしますが、どうでしょうか・・・

Answer 1

func getArtistInfo() -> Array<MPMediaItem> {
        var array = Array<MPMediaItem>()
        let artistQuery: MPMediaQuery = MPMediaQuery.artists()
        let artistCollections: [MPMediaItemCollection] = artistQuery.collections!
        for artist in artistCollections {
            let artistItems: [MPMediaItem] = artist.items
            for artists in artistItems {
                array.append(artists)
            }
        }
        return array
    }

では、artistCollectionsの数だけ、artistCollections内の曲の数だけappendを繰り返しているため、単純に中のforループを取り除き、かつ、一つ目の要素（そのartistCollectionsの持つ、一曲目に当たるitem）のみを取り出してappendし、tableに入れるときに、.artistを指定して、
各アーティストの一曲目につけられている``.artist`を取り出すことで、すべてのアーティストを羅列しています。

func getArtistInfo() -> Array<MPMediaItem> {
        var array = Array<MPMediaItem>()
        let artistQuery: MPMediaQuery = MPMediaQuery.artists()
        let artistCollections: [MPMediaItemCollection] = artistQuery.collections!
        for artist in artistCollections {
            let artistItems: [MPMediaItem] = artist.items
                array.append(artistItems[0])
        }
        return array
    }

と、思っていたのですが、
fuzzballさんのコメントにより、

func getArtistInfo() -> Array<String> {
        var array = Array<String>()
        let artistQuery: MPMediaQuery = MPMediaQuery.artists()
        let artistCollections: [MPMediaItemCollection] = artistQuery.collections!
        for artist in artistCollections {
                array.append((artist.representativeItem?.artist)!)
        }
        return array
    }

とrepresetiveItemを使うことで解決しました。

Comments

[fuzzball]

あるのか無いのかは分かりませんが、MPMediaItemCollectionからアーティスト名を取り出す方法が見つかるまでは、これで良いのではないかと思います。

ちなみに一曲目を取り出すところは、直接 array.append(artist.items.first) でいけると思います。

[mitci]

せっかくMediaQuely.artists()が用意されているのだし、ありそうな気がしますよね…
もちろん、これなら応用も効くのでしょうが……

とにかく、ありがとうございました！
職場でコメントを読み返していたら ああ、そうか！といきなり理解が進みました…笑

[fuzzball]

ドキュメントを眺めていて見つけました。
.representativeItem でコレクション内の共通な値を取得できるようです。

array.append(artist.representativeItem.artist) でいけると思います。
（適当にアンラップして下さい）

[mitci]

どこかで、見た記憶はあるのですが、完全にスルーしてしまっていました・・・

array.append((artist.representativeItem?.artist)!)

ちなみに、回答は新たに書いていただければそちらを回答とさせていただきますが、どうしましょう？
でいけました！