yasyun commented 3 years ago

issue: #6
レビューして欲しい点 SettingsPageViewController.swiftとSettingsPageCell.swiftを中心に見ていただきたいです。
その他・一度実装を終えていたのですが、#5でのレビューを受けて、enumを使用する形にコードを改善して見ました。より明示的にわかりやすくなったかと思います。至らない点があれば申し訳ありません。・gitの方でコンクリフトが起きてしまい少しファイル操作をしたので、大丈夫だとは思いますがもし何か抜けているファイル等ありましたら申し訳ないです。

yasyun commented 3 years ago

ログアウト時の処理 AccessTokenDelivery.swiftにあるdeleteAccessToken()を使用して、token = ""をしてトークンを初期化して再ログインしないと使えなくしました。

Kaito-a-bit commented 3 years ago

issue: #6

レビューして欲しい点 SettingsPageViewController.swiftとSettingsPageCell.swiftを中心に見ていただきたいです。

その他・一度実装を終えていたのですが、#5でのレビューを受けてコードを改善してみました。至らない点があれば申し訳ありません。

具体的にどのような改善をしたのかを書いてもらえると助かります。

Kaito-a-bit commented 3 years ago

・複数のシュミレータでレイアウトの崩れが起こっているので修正をお願いします。・これは一つの提案ですが、SettingPageのTableViewに関して、もしレイアウトを含めて躓いているのであればStatic Cellsでの実装を検討するのも良いと思います。

yasyun commented 3 years ago

issue: #6

レビューして欲しい点 SettingsPageViewController.swiftとSettingsPageCell.swiftを中心に見ていただきたいです。

その他・一度実装を終えていたのですが、#5でのレビューを受けてコードを改善してみました。至らない点があれば申し訳ありません。

具体的にどのような改善をしたのかを書いてもらえると助かります。

記述しました。

yasyun commented 3 years ago

・複数のシュミレータでレイアウトの崩れが起こっているので修正をお願いします。・これは一つの提案ですが、SettingPageのTableViewに関して、もしレイアウトを含めて躓いているのであればStatic Cellsでの実装を検討するのも良いと思います。

アドバイスで頂いたように、Static Cellsを使用して実装し直してみました。今までコードでcellの中身に情報を入れていたり、TableViewを2つ使っていたので、Static Cellsにしたことでコードの量も大幅に削減できました。ありがとうございます。

Kaito-a-bit commented 3 years ago