From bc576f1babe99fd7cdc3fa1ead32802899f2add7 Mon Sep 17 00:00:00 2001
From: Max <shadow.iqka@qq.com>
Date: Sat, 29 Mar 2025 17:19:58 +0800
Subject: [PATCH] refactor: Simplify data handling in processWriteAll function

- Replace individual row handling with a unified approach to convert input values into a two-dimensional slice of interfaces.
- Streamline the writing process by directly passing the constructed slice to the WriteAll method, enhancing code clarity and maintainability.
---
 excel/process.go | 48 ++++++++++--------------------------------------
 1 file changed, 10 insertions(+), 38 deletions(-)

diff --git a/excel/process.go b/excel/process.go
index 5a56495d..1dd6ca3b 100644
--- a/excel/process.go
+++ b/excel/process.go
@@ -250,51 +250,23 @@ func processWriteAll(process *process.Process) interface{} {
 		exception.New("excel.write.all %s error: %s", 500, handle, err.Error()).Throw()
 	}
 
-	// 处理二维切片值
+	// Convert data to [][]interface{}
+	var sheetData [][]interface{}
 	if arr, ok := values.([]interface{}); ok {
 		for _, row := range arr {
 			if rowArr, ok := row.([]interface{}); ok {
-				// 对于每行，使用 SetSheetRow
-				err = xls.SetSheetRow(sheet, cell, &rowArr)
-				if err != nil {
-					exception.New("excel.write.all %s:%s:%s error: %s", 500, handle, sheet, cell, err.Error()).Throw()
-				}
-
-				// 移动到下一行
-				colIndex, rowIndex, err := excelize.CellNameToCoordinates(cell)
-				if err != nil {
-					exception.New("excel.write.all %s:%s:%s error: %s", 500, handle, sheet, cell, err.Error()).Throw()
-				}
-				cell, err = excelize.CoordinatesToCellName(colIndex, rowIndex+1)
-				if err != nil {
-					exception.New("excel.write.all %s:%s:%s error: %s", 500, handle, sheet, cell, err.Error()).Throw()
-				}
+				sheetData = append(sheetData, rowArr)
 			} else {
-				// 单元素行
-				singleRow := []interface{}{row}
-				err = xls.SetSheetRow(sheet, cell, &singleRow)
-				if err != nil {
-					exception.New("excel.write.all %s:%s:%s error: %s", 500, handle, sheet, cell, err.Error()).Throw()
-				}
-
-				// 移动到下一行
-				colIndex, rowIndex, err := excelize.CellNameToCoordinates(cell)
-				if err != nil {
-					exception.New("excel.write.all %s:%s:%s error: %s", 500, handle, sheet, cell, err.Error()).Throw()
-				}
-				cell, err = excelize.CoordinatesToCellName(colIndex, rowIndex+1)
-				if err != nil {
-					exception.New("excel.write.all %s:%s:%s error: %s", 500, handle, sheet, cell, err.Error()).Throw()
-				}
+				sheetData = append(sheetData, []interface{}{row})
 			}
 		}
 	} else {
-		// 单元素
-		singleElement := []interface{}{values}
-		err = xls.SetSheetRow(sheet, cell, &singleElement)
-		if err != nil {
-			exception.New("excel.write.all %s:%s:%s error: %s", 500, handle, sheet, cell, err.Error()).Throw()
-		}
+		sheetData = [][]interface{}{{values}}
+	}
+
+	err = xls.WriteAll(sheet, cell, sheetData)
+	if err != nil {
+		exception.New("excel.write.all %s:%s error: %s", 500, handle, sheet, err.Error()).Throw()
 	}
 	return nil
 }