feat: optimize the table node parse steps and add logs

lib/src/editor/block_component/table_block_component/table_block_component.dart

@@ -110,10 +110,66 @@
   }
 
   @override
-  bool validate(Node node) =>
-      node.attributes.isNotEmpty &&
-      node.attributes.containsKey(TableBlockKeys.colsLen) &&
-      node.attributes.containsKey(TableBlockKeys.rowsLen);
+  bool validate(Node node) {
+    // check the node is valid
+    if (node.attributes.isEmpty) {
+      Log.editor.debug('TableBlockComponentBuilder: node is empty');
+      return false;
+    }
+
+    // check the node has rowPosition and colPosition
+    if (!node.attributes.containsKey(TableBlockKeys.colsLen) ||
+        !node.attributes.containsKey(TableBlockKeys.rowsLen)) {
+      Log.editor.debug(
+        'TableBlockComponentBuilder: node has no colsLen or rowsLen',
+      );
+      return false;
+    }
+
+    final colsLen = node.attributes[TableBlockKeys.colsLen];
+    final rowsLen = node.attributes[TableBlockKeys.rowsLen];
+
+    // check its children
+    final children = node.children;
+    if (children.isEmpty) {
+      Log.editor.debug('TableBlockComponentBuilder: children is empty');
+      return false;
+    }
+
+    if (children.length != colsLen * rowsLen) {
+      Log.editor.debug(
+        'TableBlockComponentBuilder: children length(${children.length}) is not equal to colsLen * rowsLen($colsLen * $rowsLen)',
+      );
+      return false;
+    }
+
+    // all children should contain rowPosition and colPosition
+    for (var i = 0; i < colsLen; i++) {
+      for (var j = 0; j < rowsLen; j++) {
+        final child = children.where(
+          (n) =>
+              n.attributes[TableCellBlockKeys.colPosition] == i &&
+              n.attributes[TableCellBlockKeys.rowPosition] == j,
+        );
+        if (child.isEmpty) {
+          Log.editor.debug(
+            'TableBlockComponentBuilder: child($i, $j) is empty',
+          );
+          return false;
+        }
+
+        // should only contains one child
+        if (child.length != 1) {
+          Log.editor.debug(
+            'TableBlockComponentBuilder: child($i, $j) is not unique',
+          );
+          return false;
+        }
+      }
+    }
+
+    return true;
+  }
 }
 
 class TableBlockComponentWidget extends BlockComponentStatefulWidget {

lib/src/editor/block_component/table_block_component/table_cell_block_component.dart

@@ -27,6 +27,19 @@
   Node node,
 );
 
+Node tableCellNode(String text, int rowPosition, int colPosition) {
+  return Node(
+    type: TableCellBlockKeys.type,
+    attributes: {
+      TableCellBlockKeys.rowPosition: rowPosition,
+      TableCellBlockKeys.colPosition: colPosition,
+    },
+    children: [
+      paragraphNode(text: text),
+    ],
+  );
+}
+
 class TableCellBlockComponentBuilder extends BlockComponentBuilder {
   TableCellBlockComponentBuilder({
     super.configuration,

lib/src/editor/block_component/table_block_component/table_node.dart

@@ -12,44 +12,59 @@
   TableNode({
     required this.node,
   }) : _config = TableConfig.fromJson(node.attributes) {
-    assert(node.type == TableBlockKeys.type);
-    assert(node.attributes.containsKey(TableBlockKeys.colsLen));
-    assert(node.attributes[TableBlockKeys.colsLen] is int);
-    assert(node.attributes.containsKey(TableBlockKeys.rowsLen));
-    assert(node.attributes[TableBlockKeys.rowsLen] is int);
-
-    assert(node.attributes[TableBlockKeys.rowDefaultHeight] != null);
-    assert(node.attributes[TableBlockKeys.colMinimumWidth] != null);
-    assert(node.attributes[TableBlockKeys.colDefaultWidth] != null);
-
-    final int colsCount = node.attributes[TableBlockKeys.colsLen];
-    final int rowsCount = node.attributes[TableBlockKeys.rowsLen];
-    assert(node.children.length == colsCount * rowsCount);
-    assert(
-      node.children.every(
-        (n) =>
-            n.attributes.containsKey(TableCellBlockKeys.rowPosition) &&
-            n.attributes.containsKey(TableCellBlockKeys.colPosition),
-      ),
-    );
-    assert(
-      node.children.every(
-        (n) =>
-            n.attributes.containsKey(TableCellBlockKeys.rowPosition) &&
-            n.attributes.containsKey(TableCellBlockKeys.colPosition),
-      ),
-    );
+    if (node.type != TableBlockKeys.type) {
+      Log.editor.debug('TableNode: node is not a table');
+      return;
+    }
+
+    final attributes = node.attributes;
+    final colsLen = attributes[TableBlockKeys.colsLen];
+    final rowsLen = attributes[TableBlockKeys.rowsLen];
+
+    if (colsLen == null ||
+        rowsLen == null ||
+        colsLen is! int ||
+        rowsLen is! int) {
+      Log.editor.debug(
+        'TableNode: colsLen or rowsLen is not an integer or null',
+      );
+      return;
+    }
 
-    for (var i = 0; i < colsCount; i++) {
+    if (node.children.length != colsLen * rowsLen) {
+      Log.editor.debug(
+        'TableNode: the number of children is not equal to the number of cells',
+      );
+      return;
+    }
+
+    // every cell should has rowPosition and colPosition to indicate its position in the table
+    for (final child in node.children) {
+      if (!child.attributes.containsKey(TableCellBlockKeys.rowPosition) ||
+          !child.attributes.containsKey(TableCellBlockKeys.colPosition)) {
+        Log.editor.debug('TableNode: cell has no rowPosition or colPosition');
+        return;
+      }
+    }
+
+    for (var i = 0; i < colsLen; i++) {
       _cells.add([]);
-      for (var j = 0; j < rowsCount; j++) {
-        final cell = node.children.where(
-          (n) =>
-              n.attributes[TableCellBlockKeys.colPosition] == i &&
-              n.attributes[TableCellBlockKeys.rowPosition] == j,
-        );
-        assert(cell.length == 1);
-        _cells[i].add(newCellNode(node, cell.first));
+      for (var j = 0; j < rowsLen; j++) {
+        final cell = node.children
+            .where(
+              (n) =>
+                  n.attributes[TableCellBlockKeys.colPosition] == i &&
+                  n.attributes[TableCellBlockKeys.rowPosition] == j,
+            )
+            .firstOrNull;
+
+        if (cell == null) {
+          Log.editor.debug('TableNode: cell is empty');
+          _cells.clear();
+          return;
+        }
+
+        _cells[i].add(newCellNode(node, cell));
       }
     }
   }

test/new/block_component/table_block_component/table_node_test.dart

@@ -216,7 +216,8 @@ void main() {
         ],
       };
 
-      expect(() => TableNode.fromJson(jsonData), throwsAssertionError);
+      // it should not throw error
+      expect(() => TableNode.fromJson(jsonData), isNot(throwsFlutterError));
     });
 
     test('default constructor (from list of list of strings)', () {