blackbracken / neji-android

:gear: A client app to manage your items using QR code for Android

BlackBracken

• likes to write code in type-safe and expressive languages.

neji-android's Issues

swipe-to-delete時にアイテムの裏が赤くなる演出をつける

わかりづらいので

アイコン追加

カテゴリの追加に失敗する

アイテム一覧の画面追加

検索結果の画面を使いまわすのではなく, アイテム一覧を表示するための画面を追加する.
加えて, アイテムの追加はこの画面からのみ行えるようにして, ボックスとリージョンの指定は除去する.

FirebaseAppの内部で通信に失敗する

signInWithEmailAndPasswordを呼び出す時, 通信に失敗する場合がある.
タイムアウトしないので, 呼び出しからコールバックが呼ばれたり例外が吐かれない.
再現手段が不明なので要検証.

• エミュレータだと失敗する
• 実機でも時々失敗する
• IPv6だとコケる?(要検証)

itemTypeとitemKindで意味が異なる

分かりづらいしもっと明確に語彙を分けるべき

Please add documentation on how to login to app

Can you please add instructions on how to get these values of Firebase, email, password?

Please add some documentation on how to successfully login to this app. Thank you.

ボックスとリージョンの削除機能の追加

RecyclerView上で項目を左右にスワイプするとダイアログを出して削除できるように.

ボックス内のアイテム数をドキュメントに持たせる

バリデーションの欠落

重複チェックや空欄チェックが欠落している部分がある

ボックス登録時にQRコードの文字列の重複を確認する

まだしてない

画像のキャッシングに関する設定が甘い

「毎回ロード」か「メモリ+ストレージキャッシング」の二択しかない、SwipeRefreshLayout被せてリロード可能にし最適化する

EditItemViewでアイテムの属するボックスを変えたときボックス内の個数に変更が適用されない

FirebaseRepositoryのシグネチャが異様

モデルの中の一部の値のみ(e.g. id)を要求するべきなのにモデルそのものを要求している、など

テスト書く

俺があと2人欲しい

アイテムカテゴリの追加/削除機能の実装

検索結果をページングする

ref. https://medium.com/@ericampire/firestore-pagination-with-paging-3-762e371cc57f

kotlinx-coroutines-play-servicesに依存する

ref. https://github.com/Kotlin/kotlinx.coroutines/tree/develop/integration/kotlinx-coroutines-play-services

主にFirebaseRepositoryの実装内でのsuspendCoroutineの依存を剥がしたい

QrSearchの際赤点の座標がずれる

左右には問題ないが, 下に行けばいくほど誤差が大きくなっている. 要検証.

各追加画面の実装

• リージョンの追加
• ボックスの追加
• アイテムの追加

リモートに変更を送ってもローカルにそれが適用されていない

Firestoreへの移行

• モデルをドキュメント化
• ページングが必要になりそうな値の対応
  • 諸々の登録数の上限を聞く
  • Recyclerな画面にそれぞれ対応
• モックの用意と検証

リージョンやボックスが内容物の個数を持つようにする

N+1問題も解決する. 内容物の削除時にコレクション側に変更を加える必要があるので注意すること.

検索でアイテムを探した時に入ってる箱を検索することができない

存在していない、アイテムの画像を読み込もうとしている

E/StorageException: StorageException has occurred.
    Object does not exist at location.
     Code: -13010 HttpResult: 404
E/StorageException: Could not open resulting stream.
    java.io.IOException: Could not open resulting stream.
        at com.google.firebase.storage.StreamDownloadTask.createDownloadStream(StreamDownloadTask.java:145)
        at com.google.firebase.storage.StreamDownloadTask.access$000(StreamDownloadTask.java:36)
        at com.google.firebase.storage.StreamDownloadTask$1.call(StreamDownloadTask.java:167)
        at com.google.firebase.storage.StreamDownloadTask$1.call(StreamDownloadTask.java:164)
        at com.google.firebase.storage.StreamDownloadTask$StreamProgressWrapper.ensureStream(StreamDownloadTask.java:325)
        at com.google.firebase.storage.StreamDownloadTask$StreamProgressWrapper.access$100(StreamDownloadTask.java:262)
        at com.google.firebase.storage.StreamDownloadTask.run(StreamDownloadTask.java:175)
        at com.google.firebase.storage.StorageTask.lambda$getRunnable$7(StorageTask.java:1072)
        at com.google.firebase.storage.StorageTask$$Lambda$12.run(Unknown Source:2)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
        at java.lang.Thread.run(Thread.java:923)
    StorageException has occurred.
    Object does not exist at location.
     Code: -13010 HttpResult: 404
E/StorageException: Could not open resulting stream.
    java.io.IOException: Could not open resulting stream.
        at com.google.firebase.storage.StreamDownloadTask.createDownloadStream(StreamDownloadTask.java:145)
        at com.google.firebase.storage.StreamDownloadTask.access$000(StreamDownloadTask.java:36)
        at com.google.firebase.storage.StreamDownloadTask$1.call(StreamDownloadTask.java:167)
        at com.google.firebase.storage.StreamDownloadTask$1.call(StreamDownloadTask.java:164)
        at com.google.firebase.storage.StreamDownloadTask$StreamProgressWrapper.ensureStream(StreamDownloadTask.java:325)
        at com.google.firebase.storage.StreamDownloadTask$StreamProgressWrapper.access$100(StreamDownloadTask.java:262)
        at com.google.firebase.storage.StreamDownloadTask.run(StreamDownloadTask.java:175)
        at com.google.firebase.storage.StorageTask.lambda$getRunnable$7(StorageTask.java:1072)
        at com.google.firebase.storage.StorageTask$$Lambda$12.run(Unknown Source:2)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
        at java.lang.Thread.run(Thread.java:923)
E/StorageException: StorageException has occurred.
    Object does not exist at location.
     Code: -13010 HttpResult: 404
    Could not open resulting stream.
    java.io.IOException: Could not open resulting stream.
        at com.google.firebase.storage.StreamDownloadTask.createDownloadStream(StreamDownloadTask.java:145)
        at com.google.firebase.storage.StreamDownloadTask.access$000(StreamDownloadTask.java:36)
        at com.google.firebase.storage.StreamDownloadTask$1.call(StreamDownloadTask.java:167)
        at com.google.firebase.storage.StreamDownloadTask$1.call(StreamDownloadTask.java:164)
        at com.google.firebase.storage.StreamDownloadTask$StreamProgressWrapper.ensureStream(StreamDownloadTask.java:325)
        at com.google.firebase.storage.StreamDownloadTask$StreamProgressWrapper.access$100(StreamDownloadTask.java:262)
        at com.google.firebase.storage.StreamDownloadTask.run(StreamDownloadTask.java:175)
        at com.google.firebase.storage.StorageTask.lambda$getRunnable$7(StorageTask.java:1072)
        at com.google.firebase.storage.StorageTask$$Lambda$12.run(Unknown Source:2)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
        at java.lang.Thread.run(Thread.java:923)

アイテム編集画面の追加

Firebaseに癒着することを許容する

今はリポジトリにして切り分けているが, 分離するメリットよりデメリットが大きいように思う.
アプリの仕様レベルでFirebaseに依存しているので差し替えるタイミングが無く, テストにはFirebase Local Emulator Suiteを用いれば良い気がする(ユニットテストだと目標を達成できない可能性, 愚直にモックを作るのも吝かではない).

QRコードの画像を保存できるようにする

ボックスとリージョンの編集画面作成

画像のパスを持つ変数の名称変更

FireStore上でのパス変更 imageUrl -> imagePath

QRコードが存在しないボックスのツールバーからQRコード系のボタンを消す

QRコード対応

QRのテキストはunique-keyとは別枠で用意したほうが良いのでは

QRコード読み込んだ後即座に画面遷移させない

各アイテム、ボックス、リージョンの削除機能の実装

• アイテム
• ボックス
• リージョン

検索機能の実装

• 名前による検索
• 種別による検索
• リージョン/ボックスによるフィルタリング

アイテムの編集時の値のバリデーションをView上で行わない

ref. AddItemViewModel
ViewModel上でバリデーションを行うこと, ついでにテストも書きたい