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

オフライン表示時のキャッシュ取得で、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されると誤認して作ってたよ。
上記関数の上位で、ガードしてたせいで発見できず。で、最近の修正で上位の関数のガード処理を単純化したことで、今回の問題が発覚。

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

内部通信・DB処理の近代化とリファクタリング

今回のアップデートでは、アプリの心臓部といえる通信・DB周りのリファクタリングを敢行しました。また、一部機能で先祖返りが発生していた箇所の再修正も行っています。 1. 記事一覧のスクロール位置保持 以前修正したはずの「記事から一覧に戻ると一番上に戻ってしまう」問題が、一部の処理変...