はじめに
コーディング中に Android Studio 上で出てくる警告はちゃんと気にして潰しているのですが、
Lint チェックし忘れて
「コマンドラインからビルドすると警告出るんですけど…」
という問い合わせがあったりします。
本当にすみません。
そんな自戒の意味を込めて、今まで解決した項目の解決方法メモ。
やりたいこと
- ここでメモることで Lint の存在を思い出す
- 反省する
備忘録
ビルドするコマンド
cd ${プロジェクトのフォルダ}
./gradlew build
Lint だけするコマンド
cd ${プロジェクトのフォルダ}
./gradlew ${チェックするモジュール名}:lint
Lint の結果ファイルの場所
${プロジェクトのフォルダ}/${モジュール名}/build/outputs/lint-results.html
InflateParams: Layout Inflation without a Parent
フラグメントとかビューとかをインフレートすると出るアレ。
こういうのはコード的に問題ないけど、Lint 的に NG。
LayoutInflater inflater = LayoutInflater.from(getActivity());
View view = inflater.inflate(resource, null, false);
第三引数の attachToRoot が false の時は、
第二引数の root って実質不要なんだけど、こうすると怒られない。
LayoutInflater inflater = LayoutInflater.from(getActivity());
// ちゃんとルートビューグループを指定する
View view = inflater.inflate(resource, root, false);
GradleDependency: Obsolete Gradle Dependency
うっかり古いビルドツール指定すると出るアレ。
android {
compileSdkVersion 23
buildToolsVersion "23.0.1"
/// 〜〜〜省略〜〜〜
}
特に理由がなければ、最新のやつを指定しよう。
android {
compileSdkVersion 23
buildToolsVersion "23.0.2"
/// 〜〜〜省略〜〜〜
}
理由がある場合は、コメントしつつ無効化しよう。
android {
compileSdkVersion 23
buildToolsVersion "23.0.1"
/// 〜〜〜省略〜〜〜
lintOptions {
// どうしても23.0.1 を使いたいので、無効化
disable 'GradleDependency'
}
}
WrongConstant: Incorrect constant
なんかダメな指定をした時に出るアレ。
自分の場合は、ビューの表示/非表示切り替えで
非表示時に「View.GONE」にするか「View.INVISIBLE」にするか迷ったので、
あとで修正しやすいようにこう書いていた。
private static final int VISIBILITY_HIDE = View.GONE;
// 〜〜〜省略〜〜〜
view.setVisibility(VISIBILITY_HIDE);
そうすると、こんな感じで怒られる。
${ソースコードのファイル}:${問題箇所の行数}:
Must be one of: View.VISIBLE, View.INVISIBLE, View.GONE
ここでは記載してないけど、ちゃんと問題箇所のコードもちょっと表示してくれる。
要は、__指定の定数以外を設定できるからダメ！__って言ってる。
というわけで、ちゃんと指定の定数で書く。
view.setVisibility(View.GONE);
このチェックは、重要度が「エラー」なので、ひっかかるとコマンドからビルドできない。
Android Studio 上でビルドできるからといって、油断してはいけない。（戒め）
UnusedResources: Unused resources
不要な文字列リソースとかあると出てくるアレ。
ただの消し忘れです。本当にすみません…。
ここで出てきたリソースは、ちゃんと消しときましょう。
GoogleAppIndexingWarning: Missing support for Google App Indexing
いわゆる、ディープリンクの対応がされてないよって警告のアレ。
不要な時は、無効化しましょう。
android {
compileSdkVersion 23
buildToolsVersion "23.0.2"
/// 〜〜〜省略〜〜〜
lintOptions {
// ディープリンクは利用しないので、無効化
disable 'GoogleAppIndexingWarning'
}
}
ちなみに、無効化は複数指定できます。
android {
compileSdkVersion 23
buildToolsVersion "23.0.1"
/// 〜〜〜省略〜〜〜
lintOptions {
// どうしても23.0.1 を使いたいので、無効化
// ディープリンクは利用しないので、無効化
disable 'GradleDependency', 'GoogleAppIndexingWarning'
}
}
RtlHardcoded: Using left/right instead of start/end attributes
レイアウトファイルで古い属性の paddingLeft とか layout_marginRight とか使ってると出てくるアレ。
<View
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:id="@+id/view"
android:layout_alignRight="@+id/relative_parent"/>
コードの例だと、layout_alignRight を使っているけど、
要は「〜Left」や「〜Right」だとひっかかります。
で、単純に新しいのに置き換えると、こうなる。
<View
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:id="@+id/view"
android:layout_alignEnd="@+id/relative_parent"/>
「〜Start」や「〜End」にすればよい。
とはいえ、これらは API 17 から追加されたものなので、
それ以前をサポートするやつだと、落ちる。
じゃあ、どうすればいいのかというと、両方書く。
<View
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:id="@+id/view"
android:layout_alignRight="@+id/relative_parent"
android:layout_alignEnd="@+id/relative_parent"/>
なぜ、そうすべきかも lint−results.html に書いてある。
Using Gravity#LEFT and Gravity#RIGHT can lead to problems when a layout is rendered in locales where text
flows from right to left. Use Gravity#START and Gravity#END instead. Similarly, in XML gravity and
layout_gravity attributes, use start rather than left.
For XML attributes such as paddingLeft and layout_marginLeft, use paddingStart and layout_marginStart.
NOTE: If your minSdkVersion is less than 17, you should add both the older left/right attributes as well
as the new start/right attributes. On older platforms, where RTL is not supported and the start/right
attributes are unknown and therefore ignored, you need the older left/right attributes. There is a
separate lint check which catches that type of error.
(Note: For Gravity#LEFT and Gravity#START, you can use these constants even when targeting older
platforms, because the start bitmask is a superset of the left bitmask. Therefore, you can use
gravity="start" rather than gravity="left|start".)
なるほどなー。
AllowBackup: AllowBackup/FullBackupContent Problems
バックアップの設定をデフォルトのままにしとくと出るアレ。
<application
android:allowBackup="true"
...>
...
</application>
別にバックアップ使わないときは false を設定しておこう。
<application
android:allowBackup="false"
...>
...
</application>
この場合、もしライブラリ側が allowBackup を true にしているとビルドできないので、
そういうときは、tools:replace 属性も追加しよう。
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:tools="http://schemas.android.com/tools"
...>
<application
android:allowBackup="false"
tools:replace="android:allowBackup"
...>
...
</application>
</manifest>
UselessParent: Useless parent layout
親レイアウトが意味ない感じになっていると出てくるアレ。
こういうやつね。
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="match_parent">
<RelativeLayout
android:layout_width="match_parent"
android:layout_height="match_parent">
...
</RelativeLayout>
</RelativeLayout>
ちゃんと意味のある親にしよう。
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="match_parent">
...
</RelativeLayout>
InvalidPackage: Package not included in Android
主にライブラリのせいで出てくるアレ。
自分のときは、「picasso2-okhttp3-downloader」を
使ってたときに出たので、以下のように回避した。
dependencies {
compile fileTree(dir: 'libs', include: ['*.jar'])
testCompile 'junit:junit:4.12'
/// 〜〜〜省略〜〜〜
compile ('com.jakewharton.picasso:picasso2-okhttp3-downloader:1.0.2') {
exclude group: 'com.squareup.okio', module: 'okio'
}
}
ライブラリによって対応方法が違うので、
「どの jar がダメなのか」のエラー文でググるのがいいと思う。
すべて通る
と、こうなる。
おめでとう！ありがとう！
すべて無効化する
Lint チェック(ﾟ⊿ﾟ)ｲﾗﾈ って場合は、こう書くと Lint 自体を無効化できる。
android {
compileSdkVersion 23
buildToolsVersion "23.0.1"
/// 〜〜〜省略〜〜〜
lintOptions {
// エラーとか完全無視する
abortOnError false
}
}
とはいえ、何も考えずに Android Studio 使っていると見逃すエラーもあるので、
個人的にはやらないほうがいいと思う。
まとめ
- Lint ダイジ、ゼッタイ！
- Android Studio だけじゃ見つからないエラーも見つかるよ！
- 他に見つけたら、追記するよ！
参考
- ライブラリで指定された AndroidManifest の属性を置き換える
http://y-anz-m.blogspot.jp/2015/09/androidmanifest.html