mtattin / ios-engineer-codecheck Goto Github PK

本プロジェクトは ViewController が必要以上の責務を抱えており、いわゆる Fat VC 状態です。最低限の責務の切り出しをしてあげましょう。

※本課題は #6 の下位互換です。

本アプリの機能は極めて単純です。使いやすさが損なわれない前提で、様々な新機能を追加してあげてください。

最近巷では MVC や MVVM などの GUI アーキテクチャから、Redux や VIPER などのアプリアーキテクチャまで様々なアーキテクチャが話題になっています。どれでもいいので本プロジェクトを自分の慣れたアーキテクチャに修正してください。

※本課題は #4 の上位互換です。

本プロジェクトに Storyboard 含め、至る所にバグが潜んであります。下記のリストを参考にバグを修正してください。

• レイアウトエラー
• メモリリーク
• パースエラー

本プロジェクトは安全性の低いコードたくさんあります。下記のリストを参考に、安全性の低いコードを撲滅し、安全性を高めましょう。

• 強制アンラップ
• 強制ダウンキャスト
• 不必要なIUO
• 想定外の nil の握り潰し

本プロジェクトのソースコードはとても読みにくいです。下記のリストを参考に読みやすいように修正してください。

• 命名規約（参考：Swift API Design Guidelines）
• ネスト
• インデント
• コメントの適切性
• スペースや改行
• その他

本アプリの UI は極めてシンプルです。これでも特に問題はありませんが、やはりいまいち映えないです。使いやすさが損なわれない前提で、UI をブラッシュアップしてください。方向性（エレガント、ゴージャス、モダン等）は問いません。また UIKit だけでなく、SwiftUI の導入も OK です。

現在のプログラムではテストがありません。UnitTest や UITest を追加してください。

本プロジェクトは様々な原則に違反しています。下記のリストを参考にプログラムをリファクタリングしましょう：

• DRY 原則
• CQS 原則
• 単一責任原則
• インターフェイス分離の原則
• 驚き最小の原則