Ta23ka98 / life_snap

1 stars 1 forks source link

Feature/#26 firstpage login #32

Closed KonSan0722 closed 2 years ago

KonSan0722 commented 2 years ago

ログイン機能の実装をしました。よろしくお願いします。

hirom0922 commented 2 years ago

@KonSan0722 お疲れ様です。 他の方と競合することがありませんので、ReviewはOKとしますが、 次のチャレンジとしまして、下記も意識いただければと思います。

1.ファイル分割 1ファイルがだいぶ長くなってきましたし、 画面ごとにファイル分割されても良いかなと思います。 (共用Widgetを作って部品化なども)

2.グローバルの変数の改善 あと、これらがグローバルの変数になっていますが、 Stateのインスタンス変数でも大丈夫なはずです。

String loginEMail = ''; String loginPassWord = '';

KonSan0722 commented 1 year ago

お忙しいところ、レビューありがとうございます! ファイル分け、自分も気になっていたところなので挑戦してみます! あと、グローバル変数なんですが、stateのインスタンス変数で定義してしまうと、setstateするごとにclassが再描画されてしまってtextformfield内の内容がログイン時に反映されず消えてしまいます。。 ログインボタン押下時、printでpasswordなど表してもやはり変数が初期化されているようでした。 ヒロムさんはいつもどこに変数を定義していますでしょうか?(riverpodだとまた違いますよね、、)

hirom0922 commented 1 year ago

通常こちらに定義するのですが、 画面の引き継ぎ時とかに消えるのかな?

image
KonSan0722 commented 1 year ago
スクリーンショット 2022-11-16 5 50 45

なるほど、、overrideの下に変数定義していたから初期化されていたんですね、、助かりました! すみません、修正しました。

KonSan0722 commented 1 year ago

ファイル分けはひとまず次のissueが終わってから取り掛からせていただきます!