now the logic of ignoring entities can be reused

Author: iimironchuk

lib/src/common/parameters/excluded_entities_list_parameter.dart

@@ -0,0 +1,50 @@
+import 'package:analyzer/dart/ast/ast.dart';
+
+/// A model representing "exclude_entity" parameters for linting, defining
+/// identifiers (classes, mixins, enums, extensions) to be ignored during
+/// analysis.
+class ExcludedEntitiesListParameters {
+  /// The parameter model
+  final List<String> excludedEntityNames;
+
+  /// A common parameter key for analysis_options.yaml
+  static const String excludeEntityKey = 'exclude_entity';
+
+  /// Constructor for [ExcludedEntitiesListParameters] class
+  ExcludedEntitiesListParameters({
+    required this.excludedEntityNames,
+  });
+
+  /// Method for creating from json data
+  factory ExcludedEntitiesListParameters.fromJson(Map<String, dynamic> json) {
+    final raw = json['exclude_entity'];
+    if (raw is List) {
+      return ExcludedEntitiesListParameters(
+        excludedEntityNames: List<String>.from(raw),
+      );
+    }
+    return ExcludedEntitiesListParameters(
+      excludedEntityNames: [],
+    );
+  }
+
+  /// Returns whether the target node should be ignored during analysis.
+  bool shouldIgnoreEntity(Declaration node) {
+    if (excludedEntityNames.isEmpty) return false;
+
+    if (node is ClassDeclaration && excludedEntityNames.contains('class')) {
+      return true;
+    } else if (node is MixinDeclaration &&
+        excludedEntityNames.contains('mixin')) {
+      return true;
+    } else if (node is EnumDeclaration &&
+        excludedEntityNames.contains('enum')) {
+      return true;
+    } else if (node is ExtensionDeclaration &&
+        excludedEntityNames.contains('extension')) {
+      return true;
+    }
+
+    return false;
+  }
+}

lib/src/lints/prefer_match_file_name/models/prefer_match_file_name_parameters.dart

@@ -1,10 +1,10 @@
-/// A data model class that represents the "prefer factmatch file name" input
+import 'package:solid_lints/src/common/parameters/excluded_entities_list_parameter.dart';
+
+/// A data model class that represents the "prefer match file name" input
 /// parameters
 class PreferMatchFileNameParameters {
-  /// A list of methods that should be excluded from the lint.
-  final Iterable<String> excludeEntity;
-
-  static const bool _defaultIgnoreExtensionsValue = false;
+  /// A list of entities that should be excluded from the lint.
+  final ExcludedEntitiesListParameters excludeEntity;
 
   /// Constructor for [PreferMatchFileNameParameters] model
   const PreferMatchFileNameParameters({
@@ -14,9 +14,6 @@ class PreferMatchFileNameParameters {
   /// Method for creating from json data
   factory PreferMatchFileNameParameters.fromJson(Map<String, Object?> json) =>
       PreferMatchFileNameParameters(
-        excludeEntity: (json['exclude_entity'] as Iterable?)
-                ?.map((e) => e.toString())
-                .toList() ??
-            [],
+        excludeEntity: ExcludedEntitiesListParameters.fromJson(json),
       );
 }

lib/src/lints/prefer_match_file_name/visitors/prefer_match_file_name_visitor.dart

@@ -1,5 +1,6 @@
 import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/ast/visitor.dart';
+import 'package:solid_lints/src/common/parameters/excluded_entities_list_parameter.dart';
 import 'package:solid_lints/src/common/parameters/excluded_identifiers_list_parameter.dart';
 import 'package:solid_lints/src/lints/prefer_match_file_name/models/declaration_token_info.dart';
 
@@ -10,7 +11,7 @@ class PreferMatchFileNameVisitor extends RecursiveAstVisitor<void> {
 
   /// Iterable that contains the name of entity (or entities) that should
   /// be ignored
-  final Iterable<String> excludedEntities;
+  final ExcludedEntitiesListParameters excludedEntities;
 
   /// Constructor of [PreferMatchFileNameVisitor] class
   PreferMatchFileNameVisitor({
@@ -26,28 +27,28 @@ class PreferMatchFileNameVisitor extends RecursiveAstVisitor<void> {
       (a, b) => _publicDeclarationsFirst(a, b) ?? _byDeclarationOrder(a, b),
     );
 
-  bool _shouldIgnore(Declaration node) {
-    if (excludedEntities.isEmpty) return false;
-
-    if (node is ClassDeclaration && excludedEntities.contains('class')) {
-      return true;
-    } else if (node is MixinDeclaration && excludedEntities.contains('mixin')) {
-      return true;
-    } else if (node is EnumDeclaration && excludedEntities.contains('enum')) {
-      return true;
-    } else if (node is ExtensionDeclaration &&
-        excludedEntities.contains('extension')) {
-      return true;
-    }
-
-    return false;
-  }
+  // bool _shouldIgnore(Declaration node) {
+  //   if (excludedEntities.isEmpty) return false;
+  //
+  //   if (node is ClassDeclaration && excludedEntities.contains('class')) {
+  //     return true;
+  //   } else if (node is MixinDeclaration && excludedEntities.contains('mixin')) {
+  //     return true;
+  //   } else if (node is EnumDeclaration && excludedEntities.contains('enum')) {
+  //     return true;
+  //   } else if (node is ExtensionDeclaration &&
+  //       excludedEntities.contains('extension')) {
+  //     return true;
+  //   }
+  //
+  //   return false;
+  // }
 
   @override
   void visitClassDeclaration(ClassDeclaration node) {
     super.visitClassDeclaration(node);
 
-    if (_shouldIgnore(node)) return;
+    if (excludedEntities.shouldIgnoreEntity(node)) return;
 
     _declarations.add((token: node.name, parent: node));
   }
@@ -56,7 +57,7 @@ class PreferMatchFileNameVisitor extends RecursiveAstVisitor<void> {
   void visitExtensionDeclaration(ExtensionDeclaration node) {
     super.visitExtensionDeclaration(node);
 
-    if (_shouldIgnore(node)) return;
+    if (excludedEntities.shouldIgnoreEntity(node)) return;
 
     final name = node.name;
     if (name != null) {
@@ -68,7 +69,7 @@ class PreferMatchFileNameVisitor extends RecursiveAstVisitor<void> {
   void visitMixinDeclaration(MixinDeclaration node) {
     super.visitMixinDeclaration(node);
 
-    if (_shouldIgnore(node)) return;
+    if (excludedEntities.shouldIgnoreEntity(node)) return;
 
     _declarations.add((token: node.name, parent: node));
   }
@@ -77,7 +78,7 @@ class PreferMatchFileNameVisitor extends RecursiveAstVisitor<void> {
   void visitEnumDeclaration(EnumDeclaration node) {
     super.visitEnumDeclaration(node);
 
-    if (_shouldIgnore(node)) return;
+    if (excludedEntities.shouldIgnoreEntity(node)) return;
 
     _declarations.add((token: node.name, parent: node));
   }