0

そこで入力されたUserIDのdocumentが存在すればエラーを返すというコードを書きました。
しかし、
➀のコードだとdbErrorの変数が更新されずにnullのまま返されてしまいます
➁のコードだとreturn文でエラーが出てしまいます

どこを改善すればエラーを返すことができるのでしょうか?


①のコード

private fun dbCheck(userid:String):String?{
    var dbError:String? = null
    val db = FirebaseFirestore.getInstance()
    val docRef = db.collection("Users").document(userid)
        docRef.get().addOnSuccessListener { document->
            dbError = if(document.exists()){
                "既に利用されているIDです"
            }else {
                null
            }
        }
     return dbError
}

②のコード

private fun dbCheck(userid:String){
    var dbError:String? = null
    val db = FirebaseFirestore.getInstance()
    val docRef = db.collection("Users").document(userid)
        docRef.get().addOnSuccessListener { document->
            dbError = if(document.exists()){
                "既に利用されているIDです"
            }else {
                null
            }
            return@addOnSuccessListener dbError //ここがエラーになる
        }
}
2
  • 1
    "2のコードだとエラーが出る" については、実際に表示されたエラーメッセージも質問に含めておくと回答のヒントになるかもしれません。
    – cubick
    Commented 2020年8月16日 16:39
  • エラーメッセージがないのでなんともいえませんが、Kotlinで関数からString? を返す場合には、返り値の型の指定が必要です
    – sei0o
    Commented 2020年8月17日 12:25

1 件の回答 1

0

docRef.get().addOnSuccessListenerでaddしているリスナー { document -> ...は非同期で実行されます。
ですので①のコードだと、docRef.get().addOnSuccessListenerを実行した後すぐにreturn dbErrorが実行されることになるのでnullが返ることになります。

対応策としては、

  1. doCheckメソッドがdocumentが正常に取得できた場合、エラーの場合のそれぞれのコールバック関数を引数として受け取るか
  2. あるいはcoroutineで非同期処理に対応した実装にする

などがあると思います。

1の場合のサンプルです。

private fun dbCheck(userid:String, onSuccess:()->Unit, onError:() -> Unit) {
    ...
    ...
    if(document.exists()){
        onError()
    }else {
        onSuccess()
    }
    ....

(この実装だとメソッド名と実行している内容が噛み合ってないですが・・)

2の非同期処理は少々長くなるので、次の記事等が参考になるかと思います。
https://qiita.com/amay077/items/54deb0de587baae47732

2
  • onError関数とonSuccess関数内はエラーメッセージを設定する処理でよいのでしょうか?
    – 柿ピー
    Commented 2020年8月18日 12:34
  • そこは実装次第ですね。ただそれについては本質問の主旨ではないので、これ以上ここでやり取りしないほうがいいと思いますm(_ _)m
    – itmammoth
    Commented 2020年8月19日 2:02

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。