168-7cm / yumemi-ios-codecheck Goto Github PK

概要

• コードレビューで指摘された箇所を修正する。

目的

• スキルアップのため。

課題

• 詳細画面がダークモード非対応です、ダイナミックカラーなど活用しましょう
• 文字数が多い時や画面が小さい時、詳細画面でも省略表示されてしまいます
• 画面回転の時などにレイアウトが崩れてしまいます
• サーチバーの初期表示はplaceholderを活用しましょう
• 検索にフロー制御がないため、APIを連続で呼出ししてしまいlimitがかかってしまいます、isLoadingRelayで制御やthrottleなどを活用しましょう
• isLoadingは.bind(to: activityIndicatorView.rx.isAnimating)で代用できます
• SwiftGenなどで自動生成されるソースは諸説ありますが、git管理に入れない方針が多いです
• ViewModelまでエラーハンドリングしてますが、ユーザーに何も知らされていないのが勿体無いと思いました
• RepositoryDetailViewController 画像をメインスレッドで取得しているようです。通信環境がよくないとフリーズしてしまいます。
• SearchRepositoryErrorでerrorMessageを定義しようとしたみたいですね...独自でも良いと思いますが、LocalizedError Protocolなどもありますのでご参考までに