Wansuko-cmd / Passon

3 stars 0 forks source link

Create機能を含めたEdit画面の作成 #23

Closed Wansuko-cmd closed 2 years ago

Wansuko-cmd commented 2 years ago

概要

PasswordGroupを作成する機能を追加しました(いったん追加してもindex画面のままな実装にしています) また、現段階ではPasswordGroupを作成しても何もできないので、Passwordを追加する機能、Remarkを記述する機能を追加しました。 PasswordGroupを作成したときはPasswordが何もないので、そのとき用の画面を用意しました

image

image

実装したところ

IndexFragmentおよびViewModelにCreate機能やDialog機能を追加 EditあたりにRemark機能、Passwordを増やす機能を追加 MessageRowという、代わりのメッセージを入れる部分を追加

懸念点

  1. PasswordがないときにMessageを出しているが、こんな感じに書いています

        if (list.isNotEmpty()) {
            list.forEach { password ->
                showPasswordRow {
                    TODO()
                }
            }
        } else {
            messageRow {
                id(UUID.randomUUID().toString())
                message(this@ShowEpoxyController.noPasswordMessage)
            }
        }

    ただ、もう少しスマートにかける気がします・・・

  2. Epoxyに着けるidなのですが、何でもいいときの最適解が分かりません(現状はUUIDを利用) 特別なものでもないときは、その旨が分かるようなものがいいとは思っています

  3. このボタンを押した際に空のパスワードを追加したいです

image image

ただ、どうやって画面を更新するのがベストなのかで悩んでいます

よろしくお願いします