RealtimeDatabaseのkeyを配列にしてランダムで取り出したい

RealtimeDatabaseに入っているUIDを配列にしてランダムで取り出したいです。

この、1人と書いてある下のUID(このUIDは適当に作ったダミーです)を配列に入れ、ランダムで取り出したいのですが、Firebaseの公式ドキュメントを読んでもいまいち理解出来ません。

今あるコードとしては、

Swift
1    var uids: Array? = []
2    
3    func matchDatabase(){
4        ref = Database.database().reference()
5        // データベースから誰か1人のデータを持ってくる
6        ref.child("field").child("1人").observe(DataEventType.childAdded, with: {(snapshot) -> Void in
7            
8            let snapshotkey = snapshot.key as! NSDictionary
9            let uid = snapshotkey as! String
10            // 配列にuidを格納
11            self.uids?.append(uid)
12            
13            // uidsからランダムにuidを取り出す
14            var index = arc4random(uids?.count)
15            
16        })       
17    }

となっており、
var index = arc4random(uids?count)
の部分でエラーが発生しています。
SwiftもFirebaseも初心者のため、完全に手詰まってしまいました...。

どなたか教えていただけると幸いです。
よろしくお願いします。

fuzzball

2018/08/30 23:53

エラーの内容を書いて下さい。あと print(self.uids)の出力を書いて下さい。

行動規範の内容に同意します

回答1件

ベストアンサー

RealtimeDatabaseはよくわからないので主にSwiftについてだけ。
それだけでもいくつか問題ありますね。

arc4random()は引数をとらない関数です。引数をとるのはarc4random_uniform()です。
arc4random_uniform()の引数の型はUInt32ですが、uids?countはInt?型です。

オプショナルなのもそうですが、Swiftは型が厳密なのでIntとUInt32も自動でキャストしません。

uidsはオプショナルにしたり、Arrayの型パラメータを省略する必要はありますか？

不要なオプショナルや曖昧な型は避けたほうがいいです。後若干好みが入りますがArrayは使わず[Type]を使うことをオススメします。

snapshot.keyはString型ですよね？NSDictionayとStringにキャストするのは間違いなく失敗しますし多分不要です。

以上の問題を解決するなら私ならこう書きます。

Swift
1var uids: [String] = []
2
3func matchDatabase(){
4// 略
5    let uid = snapshot.key
6    // 配列にuidを格納
7    self.uids.append(uid)
8
9    // uidsからランダムにuidを取り出す
10    var index = arc4random_uniform(UInt32(uids.count))
11// 略
12}