2021-02-26 Version 1.1.0: Updated rules

Author: fartem

src/main/java/com/smlnskgmail/jaman/checkstyle/checks/AndroidViewFieldNameCheck.java

@@ -49,6 +49,11 @@ public void visitToken(DetailAST ast) {
                         log(ast);
                     }
                     break;
+                case "FloatingActionButton":
+                    if (!ast.findFirstToken(TokenTypes.IDENT).getText().startsWith("fab")) {
+                        log(ast);
+                    }
+                    break;
             }
         }
     }

src/main/java/com/smlnskgmail/jaman/checkstyle/checks/MethodObjectReturnAnnotationCheck.java

@@ -9,6 +9,11 @@
 
 public class MethodObjectReturnAnnotationCheck extends AbstractCheck {
 
+    private static final List<String> REQUIRED_ANNOTATIONS = Arrays.asList(
+            "NonNull",
+            "Nullable"
+    );
+
     private static final String MESSAGE_KEY = "MethodObjectReturnAnnotationCheck";
 
     private static final List<Integer> EXCLUDED_TYPES = Arrays.asList(
@@ -27,10 +32,13 @@ public class MethodObjectReturnAnnotationCheck extends AbstractCheck {
     public void visitToken(DetailAST ast) {
         final DetailAST returnType = ast.findFirstToken(TokenTypes.TYPE);
         if (!EXCLUDED_TYPES.contains(returnType.getFirstChild().getType())) {
-            // TODO: add annotations check
             final DetailAST modifiers = ast.findFirstToken(TokenTypes.MODIFIERS);
             if (modifiers.getChildCount(TokenTypes.ANNOTATION) == 0) {
                 log(ast.getLineNo(), MESSAGE_KEY);
+            } else {
+                if (!REQUIRED_ANNOTATIONS.contains(modifiers.getFirstChild().findFirstToken(TokenTypes.IDENT).getText())) {
+                    log(ast.getLineNo(), MESSAGE_KEY);
+                }
             }
         }
     }

src/main/java/com/smlnskgmail/jaman/checkstyle/checks/MethodParametersAnnotationCheck.java

@@ -10,6 +10,11 @@
 
 public class MethodParametersAnnotationCheck extends AbstractCheck {
 
+    private static final List<String> REQUIRED_ANNOTATIONS = Arrays.asList(
+            "NonNull",
+            "Nullable"
+    );
+
     private static final List<Integer> EXCLUDED_TYPES = Arrays.asList(
             TokenTypes.LITERAL_BOOLEAN,
             TokenTypes.LITERAL_CHAR,
@@ -42,11 +47,13 @@ public void visitToken(DetailAST ast) {
     }
 
     private boolean isInvalidParameter(DetailAST parameterAST) {
+        if (EXCLUDED_TYPES.contains(parameterAST.findFirstToken(TokenTypes.TYPE).getFirstChild().getType())) {
+            return false;
+        }
         final DetailAST modifiers = parameterAST.findFirstToken(TokenTypes.MODIFIERS);
-        // TODO: add annotations check
-        return modifiers != null
-                && !EXCLUDED_TYPES.contains(parameterAST.findFirstToken(TokenTypes.TYPE).getFirstChild().getType())
-                && modifiers.findFirstToken(TokenTypes.ANNOTATION) == null;
+        DetailAST firstAnnotation = modifiers.findFirstToken(TokenTypes.ANNOTATION);
+        return firstAnnotation == null
+                || !REQUIRED_ANNOTATIONS.contains(firstAnnotation.findFirstToken(TokenTypes.IDENT).getText());
     }
 
     @Override