Androidソースコードレビューで指摘する事が多い項目まとめ2

去年Androidソースコードレビューで指摘する事が多い項目まとめという記事を書いた時はアプリ全体を一度に見るような機会が多かったため、内容も大きめのものばかり書いていましたが、最近はプルリクエスト単位でレビューする機会が増えたので細かい項目についてまとめてみようと思います。

ミリ秒で時間を指定する時に自前で計算している

1000L * 60L * 60L * 24Lのようなコード。

TimeUnitを使いましょう。

24時間の場合は以下のように書けます。

TimeUnit.DAYS.toMillis(1L)

URLUtil.isNetworkUrl()を使いましょう。

ただしequalsIgnoreCaseで判定してます。

StringBuilder#append("&key=").append(value);のようなコードをいっぱい書いているようなところ。

Uri.Builderを使って以下のように書けます。

Uri.Builder#appendQueryParameter("key", value);

?や&、=を意識しなくてよくなり見やすくなります。

LayoutInflater#inflate()でrootを指定しないと適当なLayoutParamsを用意されて見た目が変わる可能性があるので指定しましょう。

nullにすると警告が出るのですぐわかると思います。

Header/FooterはItemのViewと同じようにHeaderViewListAdapter#getViewで返されてListViewに追加されるのでListViewをrootとして渡せばOKです。

できるだけmarginやpaddingで調整しましょう。

どうしてもViewを置いて調整したいときはSpaceを使いましょう。このViewは描画処理で何もしないので背景色などが指定出来ない代わりに低負荷です。

大抵はListViewにpaddingTop/paddingBottomを設定してclipToPaddingをfalseにすると解決します。

文字列リソースにできるものは文字列リソースにしましょう。

ただし、以下に当てはまるものは文字列リソースに向かないのでベタ書きで良いと思っています。