search

trm11tkr / github-repo-search

株式会社ゆめみ様のFlutterエンジニアコードチェック課題。
0 stars 0 forks source link

SearchFormの責務を分割したい #17

Closed trm11tkr closed 1 year ago

trm11tkr commented 1 year ago

概要

SearchFormの責務を分割したい

解決案

  1. onFieldSubmittedで前回値と比較する
onFieldSubmitted: (String value) {
            if (formKey.currentState!.validate()) {
              final nowState = ref.watch(searchQueryProvider);
              if (value != nowState) {
                ref.watch(searchQueryProvider.state).update((state) => value);
              }

            }
          },
  1. StateNotifierProviderで管理するStateNotifierクラスの中で全各地と比較する 
    void change(String text) {
if (state != text) {
  state = text;
}
}

解決案2にすれば、TextControllerやGlobalKeyのインスタンスを保持することができ、 バリデーションを追加しても可読性が下がりにくいと考える。

よって解決案2を採用

タスク