Android, Quellcode mit Checkstyle und FindBugs prüfen
In der Softwareentwicklung haben sich verschiedene Code Richtlinien gefestigt, seien es die „normalen“ Java Beans konventionen oder aber auch wie in diesem Tutorial die Checkstyle und FindBugs richtlinien, sicherlich gibt es hier auch viele viele andere Tools welche genannt werden könnten jedoch ist mir aufgefallen das die Check’s sich wiederholen und somit der Mehrwert nicht deutlich steigt.
Hier in diesem Tutorial möchte ich nun beschreiben wie ich Checkstyle und Findbugs in meinem aktuellen Android Projekt eingerichtet habe.
Inhaltsverzeichnis
Build Tool & IDE
Als Build Tool verwende ich Gradle welche in dem Android Studio enthalten ist und somit nicht extra konfiguriert werden muss.
Anlegen der Verzeichnisstruktur
Für die Ablage der Konfigurationsdateien erstellt man folgende Verzeichnisstruktur
(gelb markiert).
Erstellen des Gradle Scriptes
Aus einem anderen englisch sprachigen Tutorial habe ich mir ein passendes Gradle Script geladen und meinen Bedürfnissen wie folgt angepasst.
(Am Ende des Tutorials gibt es alle Dateien im praktischen Zip Download)
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 | apply plugin: 'checkstyle' apply plugin: 'findbugs' check.dependsOn 'checkstyle', 'findbugs', 'lint' task checkstyle(type: Checkstyle) { configFile file("${project.rootDir}/config/checkstyle/sun_checks.xml") configProperties.checkstyleSuppressionsPath = file("${project.rootDir}/config/checkstyle/suppressions.xml").absolutePath source 'src' include '**/*.java' exclude '**/gen/**' classpath = files() } task findbugs(type: FindBugs, dependsOn: assembleDebug) { ignoreFailures = false effort = "max" reportLevel = "high" excludeFilter = new File("${project.rootDir}/config/findbugs/findbugs-filter.xml") classes = files("${project.rootDir}/app/build/intermediates/classes") source 'src' include '**/*.java' exclude '**/gen/**' reports { xml.enabled = false html.enabled = true xml { destination "$project.buildDir/reports/findbugs/findbugs.xml" } html { destination "$project.buildDir/reports/findbugs/findbugs.html" } } classpath = files() } android { lintOptions { abortOnError true xmlReport false htmlReport true lintConfig file("${project.rootDir}/config/quality/lint/lint.xml") htmlOutput file("$project.buildDir/reports/lint/lint-result.html") xmlOutput file("$project.buildDir/reports/lint/lint-result.xml") } } |
Dieses Gradle Script muss nun noch in das bestehende Gradle Script integriert werden.
Dazu wird folgende Zeile der Datei „build.gradle (Module: app)“ hinzugefügt.
1 | apply from: '../config/quality.gradle' |
Ablage der Konfigurationsdateien für Checkstyle und FindBugs
Beispiel Konfigurationsdateien kann man auf der Herstellerseite herunterladen und nach seinen Bedürfnissen einfach anpassen.
Checkstyle
Ich habe mir die „sun_checks.xml“ geladen und einige für mich nicht benötigte Checks auskommentiert.
FindBugs
Eine Datei mit den FindBugs Filtern kann man unter http://findbugs.sourceforge.net/ finden.
Einbinden in den Buildprozess
Nachdem die Scripte erweitert bzw. angelegt wurden kann man die Task „checkstyle“ und „findbugs“ dem Buildprozess hinzufügen.
Dazu klickt man auf „Run“ -> „Edit Configurations“ dort wählt man die App aus (sollte vorselektiert sein.) und klickt im unteren Bereich aus das grüne + Zeichen dieses sollte nun folgendes Untermenü öffnen wo wir den Eintrag „Run Gradle task“ auswählen.
Es sollte sich nun folgende Maske öffnen in welchem wir das Projekt wählen und den Task eingeben welcher gestartet werden soll.
Dieses wiederholen wir nun auch für den Findbugs Task und das Ergebnis sollte danach wiefolgt aussehen.
Nachdem der Dialog mit der Schaltfläche „Apply“ und „OK“ bestätigt wurde wird nun bei jedem Build vorgang Checkstyle und Findbugs gestartet. Sollte hier also Checkstyle bzw. Findbugs fehler finden so wird der Build nicht fortgesetzt.
Der Download
Hier nun der Download mit dem Gradle Script und den Checkstyle sowie Findbugs Konfigurationsdateien.
