整理代码

designer_chart/src/com/fr/plugin/chart/column/VanChartCustomStackAndAxisConditionPane.java

@@ -57,8 +57,7 @@ public class VanChartCustomStackAndAxisConditionPane extends BasicBeanPane<Condi
         liteConditionPane.setPreferredSize(new Dimension(300, 300));
     }
 
-    private JPanel createDeployPane()
-    {
+    private JPanel createDeployPane() {
         isStacked = new UIButtonGroup<Integer>(new String[]{Inter.getLocText("Plugin-ChartF_YES"), Inter.getLocText("Plugin-ChartF_NO")});
         isPercentStacked = new UIButtonGroup<Integer>(new String[]{Inter.getLocText("Plugin-ChartF_YES"), Inter.getLocText("Plugin-ChartF_NO")});
         double p = TableLayout.PREFERRED;
@@ -114,18 +113,6 @@ public class VanChartCustomStackAndAxisConditionPane extends BasicBeanPane<Condi
         checkBox();
     }
 
-    public void updateBean(ConditionAttr conditionAttr){
-        AttrSeriesStackAndAxis seriesStackAndAxis = (AttrSeriesStackAndAxis)conditionAttr.getExisted(AttrSeriesStackAndAxis.class);
-
-        seriesStackAndAxis.setXAxisIndex(XAxis.getSelectedIndex());
-        seriesStackAndAxis.setYAxisIndex(YAxis.getSelectedIndex());
-
-        updateStackAndPercent(seriesStackAndAxis);
-
-        AbstractCondition con = (AbstractCondition) this.liteConditionPane.updateBean();
-        conditionAttr.setCondition(con);
-    }
-
     protected void updateStackAndPercent(AttrSeriesStackAndAxis seriesStackAndAxis) {
         seriesStackAndAxis.setStacked(isStacked.getSelectedIndex() == 0);
         if (seriesStackAndAxis.isStacked()) {
@@ -137,8 +124,15 @@ public class VanChartCustomStackAndAxisConditionPane extends BasicBeanPane<Condi
 
     public ConditionAttr updateBean() {
         ConditionAttr conditionAttr = new ConditionAttr();
-        conditionAttr.addDataSeriesCondition(new AttrSeriesStackAndAxis());
-        updateBean(conditionAttr);
+        AttrSeriesStackAndAxis seriesStackAndAxis = new AttrSeriesStackAndAxis();
+        seriesStackAndAxis.setXAxisIndex(XAxis.getSelectedIndex());
+        seriesStackAndAxis.setYAxisIndex(YAxis.getSelectedIndex());
+
+        updateStackAndPercent(seriesStackAndAxis);
+        conditionAttr.addDataSeriesCondition(seriesStackAndAxis);
+
+        AbstractCondition con = (AbstractCondition) this.liteConditionPane.updateBean();
+        conditionAttr.setCondition(con);
         return conditionAttr;
     }
 

designer_chart/src/com/fr/plugin/chart/designer/component/marker/VanChartImageMarkerPane.java

@@ -24,13 +24,9 @@ public class VanChartImageMarkerPane extends BasicBeanPane<VanChartAttrMarker> {
     private UISpinner width;
     private UISpinner height;
 
-    protected ImageBackgroundQuickPane getImageBackgroundPane() {
-        return imageBackgroundPane;
-    }
-
     public VanChartImageMarkerPane() {
         imageBackgroundPane = new ImageBackgroundQuickPane(false);
-        setImageBackgroundPaneBorder();
+        imageBackgroundPane.setPreferredSize(getImageBackgroundPreferredSize(imageBackgroundPane.getPreferredSize()));
         width = new UISpinner(0, 100, 0.5, 30);
         height = new UISpinner(0, 100, 0.5, 30);
 
@@ -52,8 +48,8 @@ public class VanChartImageMarkerPane extends BasicBeanPane<VanChartAttrMarker> {
         this.add(panel);
     }
 
-    protected void setImageBackgroundPaneBorder() {
-        imageBackgroundPane.setPreferredSize(new Dimension((int)TableLayout4VanChartHelper.EDIT_AREA_WIDTH, (int)imageBackgroundPane.getPreferredSize().getHeight()));
+    protected Dimension getImageBackgroundPreferredSize(Dimension dimension) {
+        return new Dimension((int) TableLayout4VanChartHelper.EDIT_AREA_WIDTH, (int) dimension.getHeight());
     }
 
     protected JPanel createContentPane(ImageBackgroundQuickPane imageBackgroundPane, JPanel sizePanel) {

designer_chart/src/com/fr/plugin/chart/map/line/VanChartLineMapEffectPane.java

@@ -49,6 +49,7 @@ public class VanChartLineMapEffectPane extends VanChartEffectPane{
 
         return panel;
     }
+
     protected void setContentPaneBorder() {
         return;
     }
@@ -64,8 +65,8 @@ public class VanChartLineMapEffectPane extends VanChartEffectPane{
 
     private void initTypeContentPane() {
         customContentPane = new VanChartImageMarkerPane() {
-            protected void setImageBackgroundPaneBorder() {
-                getImageBackgroundPane().setPreferredSize(new Dimension((int)TableLayout4VanChartHelper.SECOND_EDIT_AREA_WIDTH, (int)getImageBackgroundPane().getPreferredSize().getHeight()));
+            protected Dimension getImageBackgroundPreferredSize(Dimension dimension) {
+                return new Dimension((int) TableLayout4VanChartHelper.SECOND_EDIT_AREA_WIDTH, (int) dimension.getHeight());
             }
         };