REPORT-1814 表单tab块的改良bug

代码质量，抽出父类

designer_form/src/com/fr/design/designer/creator/cardlayout/TabMoveAction.java

@@ -0,0 +1,80 @@
+package com.fr.design.designer.creator.cardlayout;
+
+import com.fr.design.designer.beans.actions.FormUndoableAction;
+import com.fr.design.mainframe.FormDesigner;
+import com.fr.form.ui.CardSwitchButton;
+import com.fr.form.ui.container.cardlayout.WTabFitLayout;
+import com.fr.general.FRLogger;
+
+/**
+ * Created by zhouping on 2017/2/17.
+ */
+public class TabMoveAction extends FormUndoableAction {
+    private XCardSwitchButton xCardSwitchButton;
+
+    public TabMoveAction(FormDesigner t, XCardSwitchButton xCardSwitchButton) {
+        super(t);
+        this.xCardSwitchButton = xCardSwitchButton;
+    }
+
+    @Override
+    public boolean executeActionReturnUndoRecordNeeded() {
+        XCardSwitchButton xCardSwitchButton = getxCardSwitchButton();
+        XWCardTagLayout xwCardTagLayout = xCardSwitchButton.getTagLayout();
+        XWCardLayout xwCardLayout = xCardSwitchButton.getCardLayout();
+        CardSwitchButton currentButton = (CardSwitchButton) xCardSwitchButton.toData();
+        try {
+            int currentIndex = currentButton.getIndex();
+            int maxIndex = xwCardTagLayout.getComponentCount();
+            XWTabFitLayout xCurrentTab = (XWTabFitLayout) xwCardLayout.getXCreator(currentIndex);
+            WTabFitLayout currentTab = (WTabFitLayout) xCurrentTab.toData();
+            xwCardTagLayout.setSwitchingTab(true);
+
+            changeTabIndex(xwCardTagLayout, xwCardLayout, currentIndex, maxIndex);
+
+            moveTabAction(xwCardTagLayout, currentButton, xCurrentTab, currentTab);
+
+            xwCardTagLayout.setSwitchingTab(false);
+        }catch (Exception e){
+            xwCardTagLayout.setSwitchingTab(false);
+            FRLogger.getLogger().error(e.getMessage());
+            return false;
+        }
+        return true;
+    }
+
+    private void moveTabAction(XWCardTagLayout xwCardTagLayout, CardSwitchButton currentButton, XWTabFitLayout xCurrentTab, WTabFitLayout currentTab) {
+        int move2Index = getTabMoveIndex(currentButton);
+        XWCardLayout xwCardLayout = xCardSwitchButton.getCardLayout();
+        xwCardTagLayout.remove(xCardSwitchButton);
+        xwCardTagLayout.add(xCardSwitchButton, move2Index);
+        xwCardLayout.remove(xCurrentTab);
+        xwCardLayout.add(xCurrentTab, move2Index);
+        currentButton.setIndex(move2Index);
+        currentTab.setIndex(move2Index);
+        currentTab.setTabNameIndex(move2Index);
+        xwCardLayout.toData().setShowIndex(move2Index);
+        xwCardLayout.showCard();
+    }
+
+    //改变Tab的索引号
+    protected void changeTabIndex(XWCardTagLayout xwCardTagLayout, XWCardLayout xwCardLayout, int currentIndex, int maxIndex) {
+    }
+
+    /**
+     * 获取tab移动的目的索引：首位，末尾，下一个，上一个
+     * @param currentButton 当前按钮
+     * @return 索引
+     */
+    protected int getTabMoveIndex(CardSwitchButton currentButton) {
+        return currentButton.getIndex();
+    }
+
+    public XCardSwitchButton getxCardSwitchButton() {
+        return xCardSwitchButton;
+    }
+
+    public void setxCardSwitchButton(XCardSwitchButton xCardSwitchButton) {
+        this.xCardSwitchButton = xCardSwitchButton;
+    }
+}

designer_form/src/com/fr/design/designer/creator/cardlayout/TabMoveCustomAction.java

@@ -1,32 +1,17 @@
 package com.fr.design.designer.creator.cardlayout;
 
-import com.fr.base.BaseUtils;
-import com.fr.design.designer.beans.actions.FormUndoableAction;
 import com.fr.design.mainframe.FormDesigner;
+import com.fr.general.IOUtils;
 
 /**
  * Created by zhouping on 2017/2/9.
  */
-public class TabMoveCustomAction extends FormUndoableAction {
+public class TabMoveCustomAction extends TabMoveAction {
-    private XCardSwitchButton xCardSwitchButton;
 
     public TabMoveCustomAction(FormDesigner t, XCardSwitchButton xCardSwitchButton) {
-        super(t);
+        super(t, xCardSwitchButton);
         this.setName("");
-        this.setSmallIcon(BaseUtils.readIcon("/com/fr/design/images/control/refresh.png"));
+        this.setSmallIcon(IOUtils.readIcon("/com/fr/design/images/control/refresh.png"));
-        this.xCardSwitchButton = xCardSwitchButton;
     }
 
-    @Override
-    public boolean executeActionReturnUndoRecordNeeded() {
-        return false;
-    }
-
-    public XCardSwitchButton getxCardSwitchButton() {
-        return xCardSwitchButton;
-    }
-
-    public void setxCardSwitchButton(XCardSwitchButton xCardSwitchButton) {
-        this.xCardSwitchButton = xCardSwitchButton;
-    }
 }

designer_form/src/com/fr/design/designer/creator/cardlayout/TabMoveEndAction.java

@@ -1,37 +1,24 @@
 package com.fr.design.designer.creator.cardlayout;
 
-import com.fr.base.BaseUtils;
-import com.fr.design.designer.beans.actions.FormUndoableAction;
 import com.fr.design.mainframe.FormDesigner;
 import com.fr.form.ui.CardSwitchButton;
 import com.fr.form.ui.container.cardlayout.WTabFitLayout;
-import com.fr.general.FRLogger;
+import com.fr.general.IOUtils;
 import com.fr.general.Inter;
 
 /**
  * Created by zhouping on 2017/2/9.
  */
-public class TabMoveEndAction extends FormUndoableAction {
+public class TabMoveEndAction extends TabMoveAction {
-    private XCardSwitchButton xCardSwitchButton;
 
     public TabMoveEndAction(FormDesigner t, XCardSwitchButton xCardSwitchButton) {
-        super(t);
+        super(t, xCardSwitchButton);
         this.setName(Inter.getLocText("FR-Designer-Move_Tab_End"));
-        this.setSmallIcon(BaseUtils.readIcon("com/fr/design/images/control/rightright.png"));
+        this.setSmallIcon(IOUtils.readIcon("com/fr/design/images/control/rightright.png"));
-        this.xCardSwitchButton = xCardSwitchButton;
     }
 
-    @Override
+    //改变Tab的索引号
-    public boolean executeActionReturnUndoRecordNeeded() {
+    protected void changeTabIndex(XWCardTagLayout xwCardTagLayout, XWCardLayout xwCardLayout, int currentIndex, int maxIndex) {
-        XWCardTagLayout xwCardTagLayout = xCardSwitchButton.getTagLayout();
-        XWCardLayout xwCardLayout = xCardSwitchButton.getCardLayout();
-        CardSwitchButton currentButton = (CardSwitchButton) xCardSwitchButton.toData();
-        try {
-            int currentIndex = currentButton.getIndex();
-            int maxIndex = xwCardTagLayout.getComponentCount();
-            XWTabFitLayout xCurrentTab = (XWTabFitLayout) xwCardLayout.getXCreator(currentIndex);
-            WTabFitLayout currentTab = (WTabFitLayout) xCurrentTab.toData();
-            xwCardTagLayout.setSwitchingTab(true);
         //修改当前tab往后所有tab的索引号
         for (int i = currentIndex + 1; i < maxIndex; i++) {
             CardSwitchButton tempBtn = (CardSwitchButton) xwCardTagLayout.getXCreator(i).toData();
@@ -40,29 +27,10 @@ public class TabMoveEndAction extends FormUndoableAction {
             tempTab.setIndex(i - 1);
             tempTab.setTabNameIndex(i - 1);
         }
-            xwCardTagLayout.remove(xCardSwitchButton);
-            xwCardTagLayout.add(xCardSwitchButton);
-            xwCardLayout.remove(xCurrentTab);
-            xwCardLayout.add(xCurrentTab);
-            currentButton.setIndex(maxIndex - 1);
-            currentTab.setIndex(maxIndex - 1);
-            currentTab.setTabNameIndex(maxIndex - 1);
-            xwCardLayout.toData().setShowIndex(maxIndex - 1);
-            xwCardLayout.showCard();
-            xwCardTagLayout.setSwitchingTab(false);
-        }catch (Exception e){
-            xwCardTagLayout.setSwitchingTab(false);
-            FRLogger.getLogger().error(e.getMessage());
-            return false;
-        }
-        return true;
     }
 
-    public XCardSwitchButton getxCardSwitchButton() {
+    @Override
-        return xCardSwitchButton;
+    protected int getTabMoveIndex(CardSwitchButton btn) {
-    }
+        return getxCardSwitchButton().getTagLayout().getComponentCount() - 1;
-
-    public void setxCardSwitchButton(XCardSwitchButton xCardSwitchButton) {
-        this.xCardSwitchButton = xCardSwitchButton;
     }
 }

designer_form/src/com/fr/design/designer/creator/cardlayout/TabMoveFirstAction.java

@@ -1,36 +1,24 @@
 package com.fr.design.designer.creator.cardlayout;
 
-import com.fr.base.BaseUtils;
-import com.fr.design.designer.beans.actions.FormUndoableAction;
 import com.fr.design.mainframe.FormDesigner;
 import com.fr.form.ui.CardSwitchButton;
 import com.fr.form.ui.container.cardlayout.WTabFitLayout;
-import com.fr.general.FRLogger;
+import com.fr.general.IOUtils;
 import com.fr.general.Inter;
 
 /**
  * Created by zhouping on 2017/2/9.
  */
-public class TabMoveFirstAction extends FormUndoableAction {
+public class TabMoveFirstAction extends TabMoveAction {
-    private XCardSwitchButton xCardSwitchButton;
 
     public TabMoveFirstAction(FormDesigner t, XCardSwitchButton xCardSwitchButton) {
-        super(t);
+        super(t, xCardSwitchButton);
         this.setName(Inter.getLocText("FR-Designer-Move_Tab_First"));
-        this.setSmallIcon(BaseUtils.readIcon("com/fr/design/images/control/leftleft.png"));
+        this.setSmallIcon(IOUtils.readIcon("com/fr/design/images/control/leftleft.png"));
-        this.xCardSwitchButton = xCardSwitchButton;
     }
 
     @Override
-    public boolean executeActionReturnUndoRecordNeeded() {
+    protected void changeTabIndex(XWCardTagLayout xwCardTagLayout, XWCardLayout xwCardLayout, int currentIndex, int maxIndex) {
-        XWCardTagLayout xwCardTagLayout = xCardSwitchButton.getTagLayout();
-        XWCardLayout xwCardLayout = xCardSwitchButton.getCardLayout();
-        CardSwitchButton currentButton = (CardSwitchButton) xCardSwitchButton.toData();
-        try {
-            int currentIndex = currentButton.getIndex();
-            XWTabFitLayout xCurrentTab = (XWTabFitLayout) xwCardLayout.getXCreator(currentIndex);
-            WTabFitLayout currentTab = (WTabFitLayout) xCurrentTab.toData();
-            xwCardTagLayout.setSwitchingTab(true);
         //修改当前tab往前所有tab的索引号
         for (int i = currentIndex - 1; i >= 0; i--) {
             CardSwitchButton tempBtn = (CardSwitchButton) xwCardTagLayout.getXCreator(i).toData();
@@ -39,29 +27,10 @@ public class TabMoveFirstAction extends FormUndoableAction {
             tempTab.setIndex(i + 1);
             tempTab.setTabNameIndex(i + 1);
         }
-            xwCardTagLayout.remove(xCardSwitchButton);
-            xwCardTagLayout.add(xCardSwitchButton, 0);
-            xwCardLayout.remove(xCurrentTab);
-            xwCardLayout.add(xCurrentTab, 0);
-            currentButton.setIndex(0);
-            currentTab.setIndex(0);
-            currentTab.setTabNameIndex(0);
-            xwCardLayout.toData().setShowIndex(0);
-            xwCardLayout.showCard();
-            xwCardTagLayout.setSwitchingTab(false);
-        }catch (Exception e){
-            xwCardTagLayout.setSwitchingTab(false);
-            FRLogger.getLogger().error(e.getMessage());
-            return false;
-        }
-        return true;
-    }
-
-    public XCardSwitchButton getxCardSwitchButton() {
-        return xCardSwitchButton;
     }
 
-    public void setxCardSwitchButton(XCardSwitchButton xCardSwitchButton) {
+    @Override
-        this.xCardSwitchButton = xCardSwitchButton;
+    protected int getTabMoveIndex(CardSwitchButton btn) {
+        return 0;
     }
 }

designer_form/src/com/fr/design/designer/creator/cardlayout/TabMoveNextAction.java

@@ -1,66 +1,34 @@
 package com.fr.design.designer.creator.cardlayout;
 
-import com.fr.base.BaseUtils;
-import com.fr.design.designer.beans.actions.FormUndoableAction;
 import com.fr.design.mainframe.FormDesigner;
 import com.fr.form.ui.CardSwitchButton;
 import com.fr.form.ui.container.cardlayout.WTabFitLayout;
-import com.fr.general.FRLogger;
+import com.fr.general.IOUtils;
 import com.fr.general.Inter;
 
 /**
  * Created by zhouping on 2017/2/9.
  */
-public class TabMoveNextAction extends FormUndoableAction {
+public class TabMoveNextAction extends TabMoveAction {
-    private XCardSwitchButton xCardSwitchButton;
 
     public TabMoveNextAction(FormDesigner t, XCardSwitchButton xCardSwitchButton) {
-        super(t);
+        super(t, xCardSwitchButton);
         this.setName(Inter.getLocText("FR-Designer-Move_Tab_Next"));
-        this.setSmallIcon(BaseUtils.readIcon("com/fr/design/images/control/right.png"));
+        this.setSmallIcon(IOUtils.readIcon("com/fr/design/images/control/right.png"));
-        this.xCardSwitchButton = xCardSwitchButton;
     }
 
     @Override
-    public boolean executeActionReturnUndoRecordNeeded() {
+    protected void changeTabIndex(XWCardTagLayout xwCardTagLayout, XWCardLayout xwCardLayout, int currentIndex, int maxIndex) {
-        XWCardTagLayout xwCardTagLayout = xCardSwitchButton.getTagLayout();
-        XWCardLayout xwCardLayout = xCardSwitchButton.getCardLayout();
-        CardSwitchButton currentButton = (CardSwitchButton) xCardSwitchButton.toData();
-        try {
-            int currentIndex = currentButton.getIndex();
-            XWTabFitLayout xCurrentTab = (XWTabFitLayout) xwCardLayout.getXCreator(currentIndex);
-            WTabFitLayout currentTab = (WTabFitLayout) xCurrentTab.toData();
-            xwCardTagLayout.setSwitchingTab(true);
         //修改下一个tab的索引号
         CardSwitchButton nextBtn = (CardSwitchButton) xwCardTagLayout.getXCreator(currentIndex + 1).toData();
         nextBtn.setIndex(currentIndex);
         WTabFitLayout nextTab = (WTabFitLayout) xwCardLayout.getXCreator(currentIndex + 1).toData();
         nextTab.setIndex(currentIndex);
         nextTab.setTabNameIndex(currentIndex);
-
-            xwCardTagLayout.remove(xCardSwitchButton);
-            xwCardTagLayout.add(xCardSwitchButton, currentIndex + 1);
-            xwCardLayout.remove(xCurrentTab);
-            xwCardLayout.add(xCurrentTab, currentIndex + 1);
-            currentButton.setIndex(currentIndex + 1);
-            currentTab.setIndex(currentIndex + 1);
-            currentTab.setTabNameIndex(currentIndex + 1);
-            xwCardLayout.toData().setShowIndex(currentIndex + 1);
-            xwCardLayout.showCard();
-            xwCardTagLayout.setSwitchingTab(false);
-        }catch (Exception e){
-            xwCardTagLayout.setSwitchingTab(false);
-            FRLogger.getLogger().error(e.getMessage());
-            return false;
-        }
-        return true;
     }
 
-    public XCardSwitchButton getxCardSwitchButton() {
+    @Override
-        return xCardSwitchButton;
+    protected int getTabMoveIndex(CardSwitchButton currentButton) {
-    }
+        return currentButton.getIndex() + 1;
-
-    public void setxCardSwitchButton(XCardSwitchButton xCardSwitchButton) {
-        this.xCardSwitchButton = xCardSwitchButton;
     }
 }

designer_form/src/com/fr/design/designer/creator/cardlayout/TabMovePrevAction.java

@@ -1,66 +1,34 @@
 package com.fr.design.designer.creator.cardlayout;
 
-import com.fr.base.BaseUtils;
-import com.fr.design.designer.beans.actions.FormUndoableAction;
 import com.fr.design.mainframe.FormDesigner;
 import com.fr.form.ui.CardSwitchButton;
 import com.fr.form.ui.container.cardlayout.WTabFitLayout;
-import com.fr.general.FRLogger;
+import com.fr.general.IOUtils;
 import com.fr.general.Inter;
 
 /**
  * Created by zhouping on 2017/2/9.
  */
-public class TabMovePrevAction extends FormUndoableAction {
+public class TabMovePrevAction extends TabMoveAction {
-    private XCardSwitchButton xCardSwitchButton;
 
     public TabMovePrevAction(FormDesigner t, XCardSwitchButton xCardSwitchButton) {
-        super(t);
+        super(t, xCardSwitchButton);
         this.setName(Inter.getLocText("FR-Designer-Move_Tab_Prev"));
-        this.setSmallIcon(BaseUtils.readIcon("com/fr/design/images/control/left.png"));
+        this.setSmallIcon(IOUtils.readIcon("com/fr/design/images/control/left.png"));
-        this.xCardSwitchButton = xCardSwitchButton;
     }
 
     @Override
-    public boolean executeActionReturnUndoRecordNeeded() {
+    protected void changeTabIndex(XWCardTagLayout xwCardTagLayout, XWCardLayout xwCardLayout, int currentIndex, int maxIndex) {
-        XWCardTagLayout xwCardTagLayout = xCardSwitchButton.getTagLayout();
-        XWCardLayout xwCardLayout = xCardSwitchButton.getCardLayout();
-        CardSwitchButton currentButton = (CardSwitchButton) xCardSwitchButton.toData();
-        try {
-            int currentIndex = currentButton.getIndex();
-            XWTabFitLayout xCurrentTab = (XWTabFitLayout) xwCardLayout.getXCreator(currentIndex);
-            WTabFitLayout currentTab = (WTabFitLayout) xCurrentTab.toData();
-            xwCardTagLayout.setSwitchingTab(true);
         //修改上一个tab的索引号
         CardSwitchButton prevBtn = (CardSwitchButton) xwCardTagLayout.getXCreator(currentIndex - 1).toData();
         prevBtn.setIndex(currentIndex);
         WTabFitLayout prevTab = (WTabFitLayout) xwCardLayout.getXCreator(currentIndex - 1).toData();
         prevTab.setIndex(currentIndex);
         prevTab.setTabNameIndex(currentIndex);
-
-            xwCardTagLayout.remove(xCardSwitchButton);
-            xwCardTagLayout.add(xCardSwitchButton, currentIndex - 1);
-            xwCardLayout.remove(xCurrentTab);
-            xwCardLayout.add(xCurrentTab, currentIndex - 1);
-            currentButton.setIndex(currentIndex - 1);
-            currentTab.setIndex(currentIndex - 1);
-            currentTab.setTabNameIndex(currentIndex - 1);
-            xwCardLayout.toData().setShowIndex(currentIndex - 1);
-            xwCardLayout.showCard();
-            xwCardTagLayout.setSwitchingTab(false);
-        }catch (Exception e){
-            xwCardTagLayout.setSwitchingTab(false);
-            FRLogger.getLogger().error(e.getMessage());
-            return false;
-        }
-        return true;
     }
 
-    public XCardSwitchButton getxCardSwitchButton() {
+    @Override
-        return xCardSwitchButton;
+    protected int getTabMoveIndex(CardSwitchButton currentButton) {
-    }
+        return currentButton.getIndex() - 1;
-
-    public void setxCardSwitchButton(XCardSwitchButton xCardSwitchButton) {
-        this.xCardSwitchButton = xCardSwitchButton;
     }
 }