diff --git a/designer-form/src/main/java/com/fr/design/designer/creator/XChartEditor.java b/designer-form/src/main/java/com/fr/design/designer/creator/XChartEditor.java
index b808a935d1..b8914f1b81 100644
--- a/designer-form/src/main/java/com/fr/design/designer/creator/XChartEditor.java
+++ b/designer-form/src/main/java/com/fr/design/designer/creator/XChartEditor.java
@@ -376,4 +376,5 @@ public class XChartEditor extends XBorderStyleWidgetCreator {
     public boolean isComponent() {
         return true;
     }
+
 }
diff --git a/designer-realize/src/main/java/com/fr/design/share/effect/EffectItemGroup.java b/designer-realize/src/main/java/com/fr/design/share/effect/EffectItemGroup.java
index c04932d0d7..30456128f4 100644
--- a/designer-realize/src/main/java/com/fr/design/share/effect/EffectItemGroup.java
+++ b/designer-realize/src/main/java/com/fr/design/share/effect/EffectItemGroup.java
@@ -109,34 +109,30 @@ public class EffectItemGroup {
             Object value = cellElement.getValue();
             if (value instanceof ChartCollection) {
                 ChartCollection chartCollection = (ChartCollection) value;
-                for (int index = 0; index < chartCollection.getChartCount(); index++) {
-                    Chart chart= (Chart) chartCollection.getChart(index, ChartProvider.class);
-                    Plot plot = chart.getPlot();
-                    if (plot == null) {
-                        continue;
-                    }
-                    SourceNode chartSourceNode = SourceNodeUtils.createSourceNode(plot, chartCollection.getChartName(index), cellSourceNode);
-                    initChartPlot(plot, chartSourceNode, false);
-                }
-
+                initChartEditorByChartCollection(chartCollection, cellSourceNode);
             }
         }
-
     }
 
     private void initChartEditor() {
         ChartEditor editor = (ChartEditor) this.widget;
         ChartCollection chartCollection = (ChartCollection) editor.getChartCollection();
+        initChartEditorByChartCollection(chartCollection, null);
+    }
+
+    private void initChartEditorByChartCollection(ChartCollection chartCollection, SourceNode sourceNode) {
         for (int index = 0; index < chartCollection.getChartCount(); index++) {
-            Chart chart= (Chart) chartCollection.getChart(index, ChartProvider.class);
-            Plot plot = chart.getPlot();
-            if (plot == null) {
-                continue;
+            ChartProvider chartProvider = chartCollection.getChart(index, ChartProvider.class);
+            if (chartProvider instanceof Chart) {
+                Chart chart= (Chart) chartCollection.getChart(index, ChartProvider.class);
+                Plot plot = chart.getPlot();
+                if (plot == null) {
+                    continue;
+                }
+                SourceNode plotSourceNode = SourceNodeUtils.createSourceNode(plot, chartCollection.getChartName(index), sourceNode);
+                initChartPlot(plot, plotSourceNode, false);
             }
-            SourceNode plotSourceNode = SourceNodeUtils.createSourceNode(plot, chartCollection.getChartName(index), null);
-            initChartPlot(plot, plotSourceNode, false);
         }
-
     }
 
     private void initChartPlot(Plot plot, SourceNode sourceNode, boolean isSubChart) {