Android Lintでコードの宣言順をチェックする / check-declaration-order-with-android-custom-lint

by Yusuke Suzuki

Embed

Start on current slide

Slide 1

Slide 1 text

Slide 2

Slide 2 text

Slide 3

Slide 3 text

Slide 4

Slide 4 text

© ZOZO, Inc. 4 課題  コードの宣言順を意識して実装・レビューをしなければならない  ● 宣言順が開発ガイドに定義されている  ○ コードの保守性向上のため  ○ 基本的にはKotlinのコーディング規約に準拠  ○ ZOZOTOWN固有のルールあり    ● 開発ガイドを確認する必要があり、開発・レビューの負担につながる    ● レビュー時に指摘漏れが発生する可能性がある      コードの宣言順についての開発ガイド参考: Kotlinのコーディング規約: https://kotlinlang.org/docs/coding-conventions.html

Slide 5

Slide 5 text

© ZOZO, Inc. 5 課題  コードの宣言順を意識して実装・レビューをしなければならない  ● 宣言順が開発ガイドに定義されている  ○ コードの保守性向上のため  ○ 基本的にはKotlinのコーディング規約に準拠  ○ ZOZOTOWN固有のコードあり    ● 開発ガイドを確認する必要があり、開発・レビューの負担につながる    ● レビュー時に指摘漏れが発生する可能性がある      コードの宣言順についての開発ガイド参考: Kotlinのコーディング規約: https://kotlinlang.org/docs/coding-conventions.html

Slide 6

Slide 6 text

Slide 7

Slide 7 text

Slide 8

Slide 8 text

Slide 9

Slide 9 text

Slide 10

Slide 10 text

Slide 11

Slide 11 text

Slide 12

Slide 12 text

Slide 13

Slide 13 text

© ZOZO, Inc. 13 PSI(Program Structure Interface)  IntelliJ製のAST  言語ごとに異なるPSIが存在する  今回はこちらを使用する    UAST(Universal AST)  UASTをサポートする言語で共通の表現を提供  Java/Kotlin両方で同じ表現を使用できる    Android Lintの仕組み  抽象構文木(AST; Abstract Syntax Tree)  通常の構文木から、意味的な情報のみを取り出したもの  PSIの例 UASTの例

Slide 14

Slide 14 text

Slide 15

Slide 15 text

© ZOZO, Inc. 15 Registry  Issueを登録  ServiceLoader経由でインスタンスを提供    Issue  Lintチェックで特定される問題  問題の種類、深刻度などの情報が含まれる    Detector  ASTの情報をもとにコードを解析  Issueをレポート  Android Lintの仕組み  Lintチェック  ASTの情報をもとに、コードの問題を検出する  依存関係

Slide 16

Slide 16 text

Slide 17

Slide 17 text

Slide 18

Slide 18 text

Slide 19

Slide 19 text

Slide 20

Slide 20 text

Slide 21

Slide 21 text

Slide 22

Slide 22 text

Slide 23

Slide 23 text

Slide 24

Slide 24 text

Slide 25

Slide 25 text

Slide 26

Slide 26 text

Slide 27

Slide 27 text

Slide 28

Slide 28 text

Slide 29

Slide 29 text

Slide 30

Slide 30 text

Slide 31

Slide 31 text

Slide 32

Slide 32 text

Slide 33

Slide 33 text

Slide 34

Slide 34 text

Slide 35

Slide 35 text

Slide 36

Slide 36 text

Slide 37

Slide 37 text

Slide 38

Slide 38 text

Slide 39

Slide 39 text

Slide 40

Slide 40 text

Slide 41

Slide 41 text

Slide 42

Slide 42 text

Slide 43

Slide 43 text

Slide 44

Slide 44 text

Slide 45

Slide 45 text

Slide 46

Slide 46 text

Slide 47

Slide 47 text

Slide 48

Slide 48 text

Slide 49

Slide 49 text

Slide 50

Slide 50 text

Slide 51

Slide 51 text

Slide 52

Slide 52 text

No content