Run Dart Analyzer

- Run Dart Analyzer fixes.
- Add flutter_lints.

Author: rayliverified

analysis_options.yaml

@@ -0,0 +1,29 @@
+# This file configures the analyzer, which statically analyzes Dart code to
+# check for errors, warnings, and lints.
+#
+# The issues identified by the analyzer are surfaced in the UI of Dart-enabled
+# IDEs (https://dart.dev/tools#ides-and-editors). The analyzer can also be
+# invoked from the command line by running `flutter analyze`.
+
+# The following line activates a set of recommended lints for Flutter apps,
+# packages, and plugins designed to encourage good coding practices.
+include: package:flutter_lints/flutter.yaml
+
+linter:
+  # The lint rules applied to this project can be customized in the
+  # section below to disable rules from the `package:flutter_lints/flutter.yaml`
+  # included above or to enable additional rules. A list of all available lints
+  # and their documentation is published at
+  # https://dart-lang.github.io/linter/lints/index.html.
+  #
+  # Instead of disabling a lint rule for the entire project in the
+  # section below, it can also be suppressed for a single line of code
+  # or a specific dart file by using the `// ignore: name_of_lint` and
+  # `// ignore_for_file: name_of_lint` syntax on the line or in the file
+  # producing the lint.
+  rules:
+    # avoid_print: false  # Uncomment to disable the `avoid_print` rule
+    # prefer_single_quotes: true  # Uncomment to enable the `prefer_single_quotes` rule
+
+# Additional information about this file can be found at
+# https://dart.dev/guides/language/analysis-options

lib/responsive_grid.dart

@@ -39,7 +39,7 @@ class ResponsiveGridView extends StatelessWidget {
   final Clip clipBehavior;
   final String? restorationId;
 
-  ResponsiveGridView.builder({
+  const ResponsiveGridView.builder({
     Key? key,
     this.scrollDirection = Axis.vertical,
     this.reverse = false,
@@ -62,7 +62,7 @@ class ResponsiveGridView extends StatelessWidget {
     this.keyboardDismissBehavior = ScrollViewKeyboardDismissBehavior.manual,
     this.clipBehavior = Clip.hardEdge,
     this.restorationId,
-  });
+  }) : super(key: key);
 
   @override
   Widget build(BuildContext context) {
@@ -128,7 +128,7 @@ class ResponsiveGridView extends StatelessWidget {
           alignment == Alignment.topLeft ||
           alignment == Alignment.bottomLeft) {
         // Align left, no padding.
-        alignmentPadding = EdgeInsets.only(left: 0);
+        alignmentPadding = const EdgeInsets.only(left: 0);
       } else if (alignment == Alignment.center ||
           alignment == Alignment.topCenter ||
           alignment == Alignment.bottomCenter) {
@@ -137,7 +137,7 @@ class ResponsiveGridView extends StatelessWidget {
         double paddingCalc = constraints.maxWidth - crossAxisWidth;
         if (paddingCalc <= 0) {
           // There is no additional space. No padding.
-          alignmentPadding = EdgeInsets.only(left: 0);
+          alignmentPadding = const EdgeInsets.only(left: 0);
         } else if (paddingCalc > gridDelegate.crossAxisSpacing) {
           // There is enough room to center items correctly.
           // Add padding equivalent to the last item to the first item.
@@ -200,7 +200,7 @@ class _ResponsiveGridViewLayout extends BoxScrollView {
   final SliverChildDelegate childrenDelegate;
   final int? itemCount;
 
-  _ResponsiveGridViewLayout({
+  const _ResponsiveGridViewLayout({
     Key? key,
     Axis scrollDirection = Axis.vertical,
     bool reverse = false,
@@ -302,15 +302,15 @@ class ResponsiveGridDelegate extends SliverGridDelegate {
     // Item width.
     double? childCrossAxisExtent;
     // Switch between item sizing behaviors.
-    if (this.crossAxisExtent != null) {
+    if (crossAxisExtent != null) {
       crossAxisCount =
           (constraints.crossAxisExtent / (crossAxisExtent! + crossAxisSpacing))
               .floor();
       childCrossAxisExtent = crossAxisExtent;
       childMainAxisExtent = childCrossAxisExtent! / childAspectRatio;
       mainAxisStride = childMainAxisExtent + mainAxisSpacing;
       crossAxisStride = childCrossAxisExtent + crossAxisSpacing;
-    } else if (this.maxCrossAxisExtent != null) {
+    } else if (maxCrossAxisExtent != null) {
       crossAxisCount = (constraints.crossAxisExtent /
               (maxCrossAxisExtent! + crossAxisSpacing))
           .ceil();

lib/responsive_row_column.dart

@@ -1,3 +1,5 @@
+// ignore_for_file: constant_identifier_names
+
 import 'package:flutter/cupertino.dart';
 import 'package:flutter/widgets.dart';
 
@@ -67,7 +69,7 @@ class ResponsiveRowColumn extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    if (layout == ResponsiveRowColumnType.ROW)
+    if (layout == ResponsiveRowColumnType.ROW) {
       return Padding(
         padding: rowPadding,
         child: Row(
@@ -82,6 +84,7 @@ class ResponsiveRowColumn extends StatelessWidget {
           ],
         ),
       );
+    }
 
     return Padding(
       padding: columnPadding,
@@ -115,11 +118,12 @@ class ResponsiveRowColumn extends StatelessWidget {
     List<Widget> widgetList = [];
     for (int i = 0; i < childrenHolder.length; i++) {
       widgetList.add(childrenHolder[i].copyWith(rowColumn: rowColumn));
-      if (spacing != null && i != childrenHolder.length - 1)
+      if (spacing != null && i != childrenHolder.length - 1) {
         widgetList.add(Padding(
             padding: rowColumn
                 ? EdgeInsets.only(right: spacing)
                 : EdgeInsets.only(bottom: spacing)));
+      }
     }
     return widgetList;
   }
@@ -187,7 +191,7 @@ class ResponsiveRowColumnItem extends StatelessWidget {
         rowColumn: rowColumn ?? this.rowColumn,
         rowFlex: rowFlex ?? this.rowFlex,
         columnFlex: columnFlex ?? this.columnFlex,
-        rowFit: rowFlexFit ?? this.rowFit,
-        columnFit: columnFlexFit ?? this.columnFit,
+        rowFit: rowFlexFit ?? rowFit,
+        columnFit: columnFlexFit ?? columnFit,
       );
 }

lib/responsive_value.dart

@@ -1,3 +1,5 @@
+// ignore_for_file: constant_identifier_names
+
 import 'package:collection/collection.dart' show IterableExtension;
 import 'package:flutter/widgets.dart';
 
@@ -66,8 +68,9 @@ class ResponsiveValue<T> {
     if (activeCondition == null) return null;
     // Return landscape value if orientation is landscape and landscape override value is provided.
     if (ResponsiveWrapper.of(context).orientation == Orientation.landscape &&
-        activeCondition.landscapeValue != null)
+        activeCondition.landscapeValue != null) {
       return activeCondition.landscapeValue;
+    }
     // Return active condition value or default value if null.
     return activeCondition.value;
   }
@@ -159,28 +162,17 @@ class Condition<T> {
       : assert(breakpoint != null || name != null),
         assert((condition == Conditional.EQUALS) ? name != null : true);
 
-  const Condition.equals({required String name, T? value, T? landscapeValue})
-      : this.breakpoint = null,
-        this.name = name,
-        this.condition = Conditional.EQUALS,
-        this.value = value,
-        this.landscapeValue = landscapeValue;
+  const Condition.equals({required this.name, this.value, this.landscapeValue})
+      : breakpoint = null,
+        condition = Conditional.EQUALS;
 
   const Condition.largerThan(
-      {int? breakpoint, String? name, T? value, T? landscapeValue})
-      : this.breakpoint = breakpoint,
-        this.name = name,
-        this.condition = Conditional.LARGER_THAN,
-        this.value = value,
-        this.landscapeValue = landscapeValue;
+      {this.breakpoint, this.name, this.value, this.landscapeValue})
+      : condition = Conditional.LARGER_THAN;
 
   const Condition.smallerThan(
-      {int? breakpoint, String? name, T? value, T? landscapeValue})
-      : this.breakpoint = breakpoint,
-        this.name = name,
-        this.condition = Conditional.SMALLER_THAN,
-        this.value = value,
-        this.landscapeValue = landscapeValue;
+      {this.breakpoint, this.name, this.value, this.landscapeValue})
+      : condition = Conditional.SMALLER_THAN;
 
   Condition copyWith({
     int? breakpoint,
@@ -199,8 +191,7 @@ class Condition<T> {
 
   @override
   String toString() =>
-      'Condition(' +
-      'breakpoint: ' +
+      'Condition(' 'breakpoint: ' +
       breakpoint.toString() +
       ', name: ' +
       name.toString() +