Renamed square commands. Modified mirror command. TODO: Fix tests.

Author: karpev98

lib/src/coloring/advanced_coloring.dart

@@ -196,7 +196,7 @@ class Coloring extends BasicColoring {
   /// Color with a square shape from the current position.
   ///
   /// Requires a list of [colors].
-  bool squareBottomLeft(List<int> colors, [int? n]) {
+  bool squareRightUpLeft(List<int> colors, [int? n]) {
     final int originalRow = move.row;
     final int originalColumn = move.column;
     if (move.right() && move.up() && move.left()) {
@@ -219,7 +219,7 @@ class Coloring extends BasicColoring {
   /// Color with a square shape from the current position.
   ///
   /// Requires a list of [colors].
-  bool squareBottomLeftReverse(List<int> colors, [int? n]) {
+  bool squareUpRightDown(List<int> colors, [int? n]) {
     final int originalRow = move.row;
     final int originalColumn = move.column;
     if (move.up() && move.right() && move.down()) {
@@ -242,7 +242,7 @@ class Coloring extends BasicColoring {
   /// Color with a square shape from the current position.
   ///
   /// Requires a list of [colors].
-  bool squareBottomRight(List<int> colors, [int? n]) {
+  bool squareUpLeftDown(List<int> colors, [int? n]) {
     final int originalRow = move.row;
     final int originalColumn = move.column;
     if (move.up() && move.left() && move.down()) {
@@ -265,7 +265,7 @@ class Coloring extends BasicColoring {
   /// Color with a square shape from the current position.
   ///
   /// Requires a list of [colors].
-  bool squareBottomRightReverse(List<int> colors, [int? n]) {
+  bool squareLeftUpRight(List<int> colors, [int? n]) {
     final int originalRow = move.row;
     final int originalColumn = move.column;
     if (move.left() && move.up() && move.right()) {
@@ -288,7 +288,7 @@ class Coloring extends BasicColoring {
   /// Color with a square shape from the current position.
   ///
   /// Requires a list of [colors].
-  bool squareTopLeft(List<int> colors, [int? n]) {
+  bool squareDownRightUp(List<int> colors, [int? n]) {
     final int originalRow = move.row;
     final int originalColumn = move.column;
     if (move.down() && move.right() && move.up()) {
@@ -311,7 +311,7 @@ class Coloring extends BasicColoring {
   /// Color with a square shape from the current position.
   ///
   /// Requires a list of [colors].
-  bool squareTopLeftReverse(List<int> colors, [int? n]) {
+  bool squareRightDownLeft(List<int> colors, [int? n]) {
     final int originalRow = move.row;
     final int originalColumn = move.column;
     if (move.right() && move.down() && move.left()) {
@@ -334,7 +334,7 @@ class Coloring extends BasicColoring {
   /// Color with a square shape from the current position.
   ///
   /// Requires a list of [colors].
-  bool squareTopRight(List<int> colors, [int? n]) {
+  bool squareLeftDownRight(List<int> colors, [int? n]) {
     final int originalRow = move.row;
     final int originalColumn = move.column;
     if (move.left() && move.down() && move.right()) {
@@ -357,7 +357,7 @@ class Coloring extends BasicColoring {
   /// Color with a square shape from the current position.
   ///
   /// Requires a list of [colors].
-  bool squareTopRightReverse(List<int> colors, [int? n]) {
+  bool squareDownLeftUp(List<int> colors, [int? n]) {
     final int originalRow = move.row;
     final int originalColumn = move.column;
     if (move.down() && move.left() && move.up()) {
@@ -713,9 +713,10 @@ class Coloring extends BasicColoring {
   /// Returns:
   ///   A boolean value.
   bool mirrorHorizontal() {
-    if (move.row <= 2) {
-      return mirrorHorizontalUpDown();
-    }
+    // if (move.row <= 2) {
+    //   return mirrorHorizontalUpDown();
+    // }
+    mirrorHorizontalUpDown();
 
     return mirrorHorizontalDownUp();
   }
@@ -727,9 +728,10 @@ class Coloring extends BasicColoring {
   /// Returns:
   ///   A boolean value.
   bool mirrorVertical() {
-    if (move.column <= 2) {
-      return mirrorVerticalLeftRight();
-    }
+    // if (move.column <= 2) {
+    //   return
+    // }
+    mirrorVerticalLeftRight();
 
     return mirrorVerticalRightLeft();
   }

lib/src/command_caller.dart

@@ -27,22 +27,26 @@ class CommandCaller {
 
   /// It's creating a map of functions that can be called by name.
   late final Map<String, Function> _coloring = <String, Function>{
+    // Directions
     "up": board.up,
     "down": board.down,
     "left": board.left,
     "right": board.right,
-    "squareBottomLeft": board.squareBottomLeft,
-    "squareBottomRight": board.squareBottomRight,
-    "squareTopLeft": board.squareTopLeft,
-    "squareTopRight": board.squareTopRight,
-    "squareBottomLeftReverse": board.squareBottomLeftReverse,
-    "squareBottomRightReverse": board.squareBottomRightReverse,
-    "squareTopLeftReverse": board.squareTopLeftReverse,
-    "squareTopRightReverse": board.squareTopRightReverse,
+    // Square
+    "squareRightUpLeft": board.squareRightUpLeft,
+    "squareUpLeftDown": board.squareUpLeftDown,
+    "squareDownRightUp": board.squareDownRightUp,
+    "squareLeftDownRight": board.squareLeftDownRight,
+    "squareUpRightDown": board.squareUpRightDown,
+    "squareLeftUpRight": board.squareLeftUpRight,
+    "squareRightDownLeft": board.squareRightDownLeft,
+    "squareDownLeftUp": board.squareDownLeftUp,
+    // Diagonal
     "diagonalUpLeft": board.diagonalUpLeft,
     "diagonalUpRight": board.diagonalUpRight,
     "diagonalDownLeft": board.diagonalDownLeft,
     "diagonalDownRight": board.diagonalDownRight,
+    // L
     "lUpLeft": board.lUpLeft,
     "lUpRight": board.lUpRight,
     "lDownLeft": board.lDownLeft,
@@ -51,6 +55,7 @@ class CommandCaller {
     "lLeftDown": board.lLeftDown,
     "lRightUp": board.lRightUp,
     "lRightDown": board.lRightDown,
+    // Zig-zag
     "zigzagLeftUpDown": board.zigzagLeftUpDown,
     "zigzagLeftDownUp": board.zigzagLeftDownUp,
     "zigzagRightUpDown": board.zigzagRightUpDown,
@@ -59,10 +64,12 @@ class CommandCaller {
     "zigzagUpRightLeft": board.zigzagUpRightLeft,
     "zigzagDownLeftRight": board.zigzagDownLeftRight,
     "zigzagDownRightLeft": board.zigzagDownRightLeft,
+    // Mirror
     "mirrorVertical": board.mirrorVertical,
     "mirrorHorizontal": board.mirrorHorizontal,
     "mirrorCellHorizontal": board.mirrorCellHorizontal,
     "mirrorCellVertical": board.mirrorCellVertical,
+    // Color
     "fillEmpty": board.fillEmpty,
     "color": board.color,
   };

pubspec.yaml

@@ -10,9 +10,9 @@ environment:
 dependencies:
   colorize: ^3.0.0
   dartx: ^1.1.0
-  path: ^1.8.2
+  path: ^1.8.3
 
 dev_dependencies:
-  dart_code_metrics: ^5.6.0
+  dart_code_metrics: ^5.7.2
   lints: ^2.0.1
   test: ^1.21.1