fix(table-sheet): 明细表数据为空时错误的展示一行空数据 close #2255

packages/s2-core/__tests__/unit/data-set/pivot-data-set-spec.ts

@@ -158,6 +158,10 @@ describe('Pivot Dataset Test', () => {
         getDimensionsWithoutPathPre(sortedDimensionValues[EXTRA_FIELD]),
       ).toEqual(['number', 'number', 'number', 'number']);
     });
+
+    test('should get correctly empty dataset result', () => {
+      expect(dataSet.isEmpty()).toBeFalsy();
+    });
   });
 
   describe('test for query data', () => {

packages/s2-core/__tests__/unit/data-set/table-data-set-spec.ts

@@ -99,6 +99,10 @@ describe('Table Mode Dataset Test', () => {
     test('should get correct meta data', () => {
       expect(dataSet.meta).toEqual(expect.objectContaining([]));
     });
+
+    test('should get correctly empty dataset result', () => {
+      expect(dataSet.isEmpty()).toBeFalsy();
+    });
   });
 
   describe('test for query data', () => {

packages/s2-core/__tests__/unit/facet/table-facet-spec.ts

@@ -13,9 +13,11 @@
 import { getFrozenLeafNodesCount } from '@/facet/utils';
 import { SpreadSheet } from '@/sheet-type';
 import { getTheme } from '@/theme';
+
 const actualDataSet = jest.requireActual(
   '@/data-set/base-data-set',
 ).BaseDataSet;
+
 jest.mock('@/sheet-type', () => {
   const container = new Canvas({
     width: 100,
@@ -59,6 +61,9 @@
         interaction: {
           clearHoverTimer: jest.fn(),
         },
+        dataSet: {
+          isEmpty: jest.fn(),
+        },
       };
     }),
   };
@@ -77,6 +82,7 @@
         getCellData: () => 1,
         getFieldMeta: jest.fn(),
         getFieldFormatter: actualDataSet.prototype.getFieldFormatter,
+        isEmpty: jest.fn(),
       };
     }),
   };
@@ -764,7 +770,7 @@
   });
 
   test('should get correct col layout with frozen col', () => {
     const { width } = facet.spreadsheet.options;
     const { frozenColCount } = facet.cfg;
     const { colNodes } = facet.layoutResult;
     const topLevelNodes = colNodes.filter((node) => node.parent.id === 'root');

packages/s2-core/src/cell/table-col-cell.ts

@@ -77,6 +77,7 @@ export class TableColCell extends ColCell {
         y,
       };
     }
+
     return {
       x: position.x + x - scrollX,
       y: position.y + y,

packages/s2-core/src/data-set/base-data-set.ts

@@ -3,6 +3,7 @@ import {
   find,
   get,
   identity,
+  isEmpty,
   isNil,
   map,
   max,
@@ -117,6 +118,10 @@ export abstract class BaseDataSet {
     return this.displayData;
   }
 
+  public isEmpty() {
+    return isEmpty(this.getDisplayDataSet());
+  }
+
   public getValueRangeByField(field: string): ValueRange {
     const cacheRange = getValueRangeState(this.spreadsheet, field);
     if (cacheRange) {

packages/s2-core/src/facet/table-facet.ts

@@ -43,7 +43,7 @@ import {
   getFrozenRowsForGrid,
   getRowsForGrid,
 } from '../utils/grid';
-import type { PanelIndexes } from '../utils/indexes';
+import type { Indexes, PanelIndexes } from '../utils/indexes';
 import { getValidFrozenOptions } from '../utils/layout/frozen';
 import { BaseFacet } from './base-facet';
 import { CornerBBox } from './bbox/cornerBBox';
@@ -1129,14 +1129,17 @@ export class TableFacet extends BaseFacet {
       }
     }
 
-    const indexes = calculateInViewIndexes(
-      scrollX,
-      scrollY,
-      this.viewCellWidths,
-      this.viewCellHeights,
-      finalViewport,
-      this.getRealScrollX(this.cornerBBox.width),
-    );
+    // https://github.com/antvis/S2/issues/2255
+    const indexes = this.spreadsheet.dataSet.isEmpty()
+      ? ([] as unknown as Indexes)
+      : calculateInViewIndexes(
+          scrollX,
+          scrollY,
+          this.viewCellWidths,
+          this.viewCellHeights,
+          finalViewport,
+          this.getRealScrollX(this.cornerBBox.width),
+        );
 
     this.panelScrollGroupIndexes = indexes;
 

packages/s2-core/src/sheet-type/table-sheet.ts

@@ -123,6 +123,7 @@ export class TableSheet extends SpreadSheet {
       }
       return new TableDataCell(facet, this);
     };
+
     return {
       ...this.options,
       ...fields,

s2-site/docs/api/basic-class/base-data-set.zh.md

@@ -28,6 +28,7 @@ s2.dataSet.getFieldName('type')
 | getCellData         | 获取单个的单元格数据               | (params: [CellDataParams](#celldataparams)) => [DataType[]](#datatype) |                                             |
 | getMultiData        | 获取批量的单元格数据               | (query: [DataType](#datatype), isTotals?: boolean, isRow?: boolean, drillDownFields?: string[]) => [DataType[]](#datatype) |                                             |
 | moreThanOneValue    | 是否超过 1 个数值                  | () => [ViewMeta](#viewmeta)                                  |                                             |
+| isEmpty    | 是否为空数据集                 | () => `boolean`                                 |      `@antv/s2-v1.51.1`                                       |
 
 ### DataType