leoando / android-engineer-codecheck

Hi there 👋

gist

Github Discussions

hashnode

zenn

android-engineer-codecheck's Issues

プログラム構造をリファクタリング

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

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

レビュー依頼する前にプロジェクト全体の不要コードとファイルを削除する

UI をブラッシュアップ

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

ソースコードの可読性の向上

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

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

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

アーキテクチャを適用

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

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

ソースコードの安全性の向上

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

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

軽微なレイアウト調整

https://qiita.com/blendthink/items/aa70b8b3106fb4e3555f#%E7%94%BB%E9%9D%A2%E5%9B%9E%E8%BB%A2%E3%82%84%E7%AB%AF%E6%9C%AB%E3%81%AE%E7%94%BB%E9%9D%A2%E3%82%B5%E3%82%A4%E3%82%BA%E3%83%95%E3%82%A9%E3%83%B3%E3%83%88%E3%82%B5%E3%82%A4%E3%82%BA%E3%83%80%E3%83%BC%E3%82%AF%E3%83%A2%E3%83%BC%E3%83%89%E3%81%AE%E3%81%93%E3%81%A8%E3%82%92%E8%80%83%E3%81%88%E3%81%A6%E3%81%84%E3%82%8B%E3%81%8B
の通りコーディングチェックでは、　画面向きや端末解像度によってのレイアウト崩れのチェックが行われる。

現状、以下の問題があるのでまとめてこのイシューにて対応する。

• 画面横向きの時に詳細画面のレイアウトが見切れる。

レイアウトのプレビュー対応を行う

https://qiita.com/blendthink/items/aa70b8b3106fb4e3555f#%E3%83%97%E3%83%AC%E3%83%93%E3%83%A5%E3%83%BC%E3%81%A7%E7%A2%BA%E8%AA%8D%E3%81%A7%E3%81%8D%E3%82%8B%E3%82%88%E3%81%86%E3%81%97%E3%81%A6%E3%81%84%E3%82%8B%E3%81%8B

バグを修正

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

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

READMEの更新

開発環境などの情報を書きたい。
アーキテクチャなどの変更が入りやすい内容に関しては、Wikiにまとめてリンクを入れる。

コードの気になるところ洗い出し

課題提出後なのでPRは出さないが、コード上の気になるところをこのイシューに洗い出しておく。

エラー発生時に復旧手段を作る

エラー発生時
この要件を満たしたい。
現状、トーストを出してるだけなので、ユーザ目線だと不親切になっている。

Themed Iconを導入する(OS13 Beta機能)

以下を参考にする
https://www.youtube.com/watch?v=5fDc2XMf10A&t=3s

R8 暗号化対応

テストを追加

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

ライブラリやtargetSDKのバージョンアップ

ライブラリやtargetSDKをできるだけ高いバージョンのStableにしたい・
また必要のないライブラリが依存関係として追加されているかも？整理したい。

新機能を追加

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

Fat Fragment の回避

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

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