kokoichi206 / android-engineer-codecheck Goto Github PK

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

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

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

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

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

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

• レイアウトエラー
• メモリリーク
• パースエラー
• 例外の処理漏れ
• クラッシュ

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

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

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

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

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

• 命名規約
• ネスト
• インデント
• スペースや改行
• コメントの適切性
• その他

参考：https://kotlinlang.org/docs/coding-conventions.html

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

• 非 null 表明演算子
• 強制ダウンキャスト
• 不必要な lateinit 修飾子
• 想定外の null の握り潰し

Repository 検索画面で言語が見つからなかった時『Written in null』と表示されてしまっている。
（下図 2 つ目）

期待結果

以下のように表現を変更したい（案）

Before: 『Written in null』
After: 『Unknown language』

また、言語がきちんと取得できたリポに対しては今まで通り表示されること。

直近の実行結果によると、18 の warnings が出てる。

初回でちゃんと確かめなかったのが良くないが、これを修正する。