オフライン表示時のキャッシュ取得で、単純ミスしてた

オフライン表示時のキャッシュ取得で、hashUrlからurlを取得する処理しているんだけど、以下のような書き方になっていてバグってた。

fun getHashUrlToItem(hashUrl: String): String{
    hashUrlToUrl(hashUrl)?.let {
        it
    }
    Qlog.d("db not found hashUrl: $hashUrl")
    return ""
}

いや、普通に見れば問題あるよねってわかるんだけど、kotlin使っていると

return if(a=0){
    "a=0"
else{
    "a<>0"
}

みたいなのをよく使うせいで、itがreturnされると誤認して作ってたよ。
上記関数の上位で、ガードしてたせいで発見できず。で、最近の修正で上位の関数のガード処理を単純化したことで、今回の問題が発覚。

個人アプリとはいえ、そろそろ単体試験などもやってかないとかも。。

0 件のコメント:

コメントを投稿

オフライン表示時のキャッシュ取得で、単純ミスしてた

オフライン表示時のキャッシュ取得で、hashUrlからurlを取得する処理しているんだけど、以下のような書き方になっていてバグってた。 fun getHashUrlToItem(hashUrl: String): String{     hashUrlToUrl(hashUrl)...