From d1bea1c69e96e1fa41308aadf5d35ca21f9b3b26 Mon Sep 17 00:00:00 2001
From: "Henry.Wang" <Henry.Wang@fanruan.com>
Date: Thu, 4 Nov 2021 21:05:08 +0800
Subject: [PATCH] =?UTF-8?q?REPORT-61410=20=E6=95=B0=E6=8D=AE=E9=9B=86?=
 =?UTF-8?q?=E9=A2=84=E8=A7=88=E6=97=B6=E5=8F=AF=E5=A4=8D=E5=88=B6-?=
 =?UTF-8?q?=E8=A1=A8=E5=A4=B4=E5=A4=8D=E5=88=B6=E9=97=AE=E9=A2=98=20&&=20R?=
 =?UTF-8?q?EPORT-61409=20=E6=95=B0=E6=8D=AE=E9=9B=86=E9=A2=84=E8=A7=88?=
 =?UTF-8?q?=E6=97=B6=E5=8F=AF=E5=A4=8D=E5=88=B6-=E8=B5=8B=E5=80=BC?=
 =?UTF-8?q?=E7=A9=BA=E5=80=BC=E6=97=B6=E6=98=BE=E7=A4=BA=E6=88=90Null?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../data/datapane/preview/CopyableJTable.java | 51 ++++++++++++-------
 1 file changed, 32 insertions(+), 19 deletions(-)

diff --git a/designer-base/src/main/java/com/fr/design/data/datapane/preview/CopyableJTable.java b/designer-base/src/main/java/com/fr/design/data/datapane/preview/CopyableJTable.java
index eb8d188a1..a452a4d37 100644
--- a/designer-base/src/main/java/com/fr/design/data/datapane/preview/CopyableJTable.java
+++ b/designer-base/src/main/java/com/fr/design/data/datapane/preview/CopyableJTable.java
@@ -106,6 +106,7 @@ public class CopyableJTable extends SortableJTable {
                     self.updateEndPoint(-1, column);
                     self.refreshTable();
                 }
+                self.requestFocusInWindow();
             }
 
             private int getColumn(MouseEvent e) {
@@ -249,26 +250,9 @@ public class CopyableJTable extends SortableJTable {
         FineLoggerFactory.getLogger().info("copy cell value");
         java.util.List<java.util.List<Object>> table = new ArrayList<>();
         if ((startRow != endRow || startCol != endCol) && Math.min(startCol, endCol) > -1) {
-            for (int i = Math.min(startRow, endRow); i <= Math.max(startRow, endRow); i++) {
-                table.add(new ArrayList<>());
-                for (int j = Math.min(startCol, endCol); j <= Math.max(startCol, endCol); j++) {
-                    Object text = this.getTableValue(i, j);
-                    table.get(table.size() - 1).add(text);
-                }
-            }
+            copyAreaData(table);
         } else if (pointList.size() > 0) {
-            Collections.sort(pointList, Comparator.comparing(Point::getX).thenComparing(Point::getY));
-            int startRow = pointList.get(0).x;
-            int currentRow = startRow;
-            table.add(new ArrayList<>());
-            for (Point point : pointList) {
-                while (currentRow < point.x) {
-                    table.add(new ArrayList<>());
-                    currentRow++;
-                }
-                Object text = this.getTableValue(point.x, point.y);
-                table.get(table.size() - 1).add(text);
-            }
+            copyPointsData(table);
         }
 
         Clipboard clip = Toolkit.getDefaultToolkit().getSystemClipboard();
@@ -276,6 +260,35 @@ public class CopyableJTable extends SortableJTable {
         clip.setContents(tText, null);
     }
 
+    private void copyAreaData(java.util.List<java.util.List<Object>> table) {
+        for (int i = Math.min(startRow, endRow); i <= Math.max(startRow, endRow); i++) {
+            table.add(new ArrayList<>());
+            for (int j = Math.min(startCol, endCol); j <= Math.max(startCol, endCol); j++) {
+                Object text = this.getTableValue(i, j);
+                if (text != null) {
+                    table.get(table.size() - 1).add(text);
+                }
+            }
+        }
+    }
+
+    private void copyPointsData(java.util.List<java.util.List<Object>> table) {
+        Collections.sort(pointList, Comparator.comparing(Point::getX).thenComparing(Point::getY));
+        int startRow = pointList.get(0).x;
+        int currentRow = startRow;
+        table.add(new ArrayList<>());
+        for (Point point : pointList) {
+            while (currentRow < point.x) {
+                table.add(new ArrayList<>());
+                currentRow++;
+            }
+            Object text = this.getTableValue(point.x, point.y);
+            if (text != null) {
+                table.get(table.size() - 1).add(text);
+            }
+        }
+    }
+
     private Object getTableValue(int row, int col) {
         Object value = null;
         if (col > -1) {