issue-167. fixed after comments

Author: shaark

lib/src/lints/avoid_returning_widgets/models/avoid_returning_widgets_parameters.dart

@@ -1,10 +1,10 @@
-import 'package:solid_lints/src/models/exclude_params.dart';
+import 'package:solid_lints/src/models/excluded_identifiers_list_parameter.dart';
 
 /// A data model class that represents the "avoid returning widgets" input
 /// parameters.
 class AvoidReturningWidgetsParameters {
   /// A list of methods that should be excluded from the lint.
-  final ExcludeParameters exclude;
+  final ExcludedIdentifiersListParameter exclude;
 
   /// Constructor for [AvoidReturningWidgetsParameters] model
   AvoidReturningWidgetsParameters({
@@ -14,9 +14,9 @@ class AvoidReturningWidgetsParameters {
   /// Method for creating from json data
   factory AvoidReturningWidgetsParameters.fromJson(Map<String, dynamic> json) {
     return AvoidReturningWidgetsParameters(
-      exclude: ExcludeParameters.fromJson(
+      exclude: ExcludedIdentifiersListParameter.fromJson(
         excludeList:
-            json[ExcludeParameters.excludeParameterName] as Iterable? ?? [],
+            json[ExcludedIdentifiersListParameter.excludeParameterName] as Iterable? ?? [],
       ),
     );
   }

lib/src/models/exclude_params.dart

@@ -1,22 +1,22 @@
 /// Model class for ExcludeRule parameters
-class ExcludedIdentifiersParameter {
+class ExcludedIdentifierParameter {
   /// The name of the method that should be excluded from the lint.
   final String methodName;
 
   /// The name of the class that should be excluded from the lint.
   final String? className;
 
-  /// Constructor for [ExcludedIdentifiersParameter] model
-  const ExcludedIdentifiersParameter({
+  /// Constructor for [ExcludedIdentifierParameter] model
+  const ExcludedIdentifierParameter({
     required this.methodName,
     required this.className,
   });
 
   ///
-  factory ExcludedIdentifiersParameter.fromJson(
+  factory ExcludedIdentifierParameter.fromJson(
     Map<dynamic, dynamic> json,
   ) {
-    return ExcludedIdentifiersParameter(
+    return ExcludedIdentifierParameter(
       methodName: json['method_name'] as String,
       className: json['class_name'] as String?,
     );

…dels/excluded_identifiers_parameter.dart …odels/excluded_identifier_parameter.dartlib/src/models/excluded_identifiers_parameter.dart renamed to lib/src/models/excluded_identifier_parameter.dart lib/src/models/excluded_identifiers_parameter.dart renamed to lib/src/models/excluded_identifier_parameter.dart

@@ -0,0 +1,56 @@
+import 'package:analyzer/dart/ast/ast.dart';
+import 'package:collection/collection.dart';
+import 'package:solid_lints/src/models/excluded_identifier_parameter.dart';
+
+/// A data model class that represents the exclude input
+/// parameters.
+class ExcludedIdentifiersListParameter {
+  /// A list of identifiers (classes, methods, functions) that should be
+  /// excluded from the lint.
+  final List<ExcludedIdentifierParameter> exclude;
+
+  /// A common parameter key for analysis_options.yaml
+  static const String excludeParameterName = 'exclude';
+
+  /// Constructor for [ExcludedIdentifiersListParameter] model
+  ExcludedIdentifiersListParameter({
+    required this.exclude,
+  });
+
+  /// Method for creating from json data
+  factory ExcludedIdentifiersListParameter.fromJson({
+    required Iterable<dynamic> excludeList,
+  }) {
+    final exclude = <ExcludedIdentifierParameter>[];
+
+    for (final item in excludeList) {
+      if (item is Map) {
+        exclude.add(ExcludedIdentifierParameter.fromJson(item));
+      }
+    }
+    return ExcludedIdentifiersListParameter(
+      exclude: exclude,
+    );
+  }
+
+  /// Returns whether the target node should be ignored during analysis.
+  bool shouldIgnore(Declaration node) {
+    final methodName = node.declaredElement?.name;
+
+    final excludedItem =
+        exclude.firstWhereOrNull((e) => e.methodName == methodName);
+
+    if (excludedItem == null) return false;
+
+    final className = excludedItem.className;
+
+    if (className == null || node is! MethodDeclaration) {
+      return true;
+    } else {
+      final classDeclaration = node.thisOrAncestorOfType<ClassDeclaration>();
+
+      return classDeclaration != null &&
+          classDeclaration.name.toString() == className;
+    }
+  }
+}