Redesign TabBase to get rid of mandatory inheritance of dialog element interfaces

plugins/org.chromium.debug.ui/src/org/chromium/debug/ui/launcher/TabBase.java

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.debug.ui.launcher;
 
 import java.util.Collection;
 import java.util.List;
 import java.util.Map;
 
@@ skipping 41 matching lines @@
       }
     };
 
     tabElements = createElements(parent, modifyListener);
   }
 
   protected abstract ELEMENTS createElements(Composite parent, Runnable modifyListener);
 
   @Override
   public void setDefaults(ILaunchConfigurationWorkingCopy configuration) {
-    for (TabField<?, ?, ?, PARAMS> field : getTabFields()) {
-      field.setDefault(configuration, getParams());
-    }
+    getTabFields().setDefaults(configuration, getParams());
   }
 
   @Override
   public void initializeFrom(ILaunchConfiguration configuration) {
-    for (TabField<?, ?, ? super ELEMENTS, ?> field : getTabFields()) {
-      field.initializeFrom(tabElements, configuration);
-    }
+    getTabFields().initializeFrom(tabElements, configuration);
   }
 
   @Override
   public void performApply(ILaunchConfigurationWorkingCopy configuration) {
-    for (TabField<?, ?, ? super ELEMENTS, ?> field : getTabFields()) {
-      field.saveToConfig(tabElements, configuration);
-    }
+    getTabFields().saveToConfig(tabElements, configuration);
   }
 
   @Override
   public boolean isValid(ILaunchConfiguration config) {
     MessageData messageData;
     try {
       messageData = isValidImpl(config);
     } catch (CoreException e) {
       ChromiumDebugPlugin.log(new Exception("Unexpected storage problem", e)); //$NON-NLS-1$
       messageData = new MessageData(true, "Internal error " + e.getMessage()); //$NON-NLS-1$
@@ skipping 31 matching lines @@
       this.message = message;
     }
     public boolean isValid() {
       return valid;
     }
     public String getMessage() {
       return message;
     }
   }
 
-  protected abstract List<? extends TabField<?, ?, ? super ELEMENTS, PARAMS>> getTabFields();
+  protected abstract TabFieldList<? super ELEMENTS, ? super PARAMS> getTabFields();
 
   /**
    * A dialog window tab field description. It is a static description -- it has no reference to
    * a particular element instance.
    * @param <P> physical type of field as stored in config; used internally
    * @param <L> logical type of field used in runtime operations
    * @param <E> interface to dialog elements
    * @param <C> context that holds immutable input parameter of the tab dialog
    */
   static class TabField<P, L, E, C> {
@@ skipping 244 matching lines @@
       }
     };
 
     for (Button button : buttons) {
       if ((button.getStyle() & SWT.NO_RADIO_GROUP) == 0) {
         throw new IllegalArgumentException();
       }
       button.addSelectionListener(selectionListener);
     }
   }
+
+  /**
+   * Incapsulate tab field list together with a way of accessing them. This includes adapting

[apavlov, 2012/08/06 08:19:06]

Encapsulate

[Peter Rybin, 2012/08/06 22:40:35]

Done.

+   * type of dialog elements structure (<E>) to a type accepted by fields.
+   * @param <E> type of elements structure that provides getters to dialog elements
+   * @param <P> type of dialog window parameters
+   */
+  interface TabFieldList<E, P> {
+    void setDefaults(ILaunchConfigurationWorkingCopy configuration, P params);
+    void initializeFrom(E elements, ILaunchConfiguration configuration);
+    void saveToConfig(E elements, ILaunchConfigurationWorkingCopy configuration);
+  }
+
+  /**
+   * Create a plain implementation of {@link TabFieldList} over a list of tab fields.
+   */
+  static <E, P> TabFieldList<E, P> createFieldListImpl(
+      final List<? extends TabField<?, ?, ? super E, P>> tabFields) {
+    return new TabFieldList<E, P>() {
+      public void setDefaults(ILaunchConfigurationWorkingCopy configuration, P params) {
+        for (TabField<?, ?, ?, P> field : tabFields) {
+          field.setDefault(configuration, params);
+        }
+      }
+
+      @Override
+      public void initializeFrom(E elements, ILaunchConfiguration configuration) {
+        for (TabField<?, ?, ? super E, ?> field : tabFields) {
+          field.initializeFrom(elements, configuration);
+        }
+      }
+
+      @Override
+      public void saveToConfig(E elements, ILaunchConfigurationWorkingCopy configuration) {
+        for (TabField<?, ?, ? super E, ?> field : tabFields) {
+          field.saveToConfig(elements, configuration);
+        }
+      }
+    };
+  }
+
+  interface Adapter<F, T> {
+    T get(F from);
+  }
+
+  /**
+   * Creates {@link TabFieldList} implementation that adapts dialog elements type using the adapter.
+   * @param list of tab fields that accepts alternative type of dialog elements structure
+   * @param elementsAdapter converts one dialog elements structure type to another

[apavlov, 2012/08/06 08:19:06]

what are "one" and "another" in this context? This should be explained in the
javadoc

[Peter Rybin, 2012/08/06 22:40:35]

Done.

+   */
+  static <E, INNER, P> TabFieldList<E, P> createFieldListAdapting(
+      final TabFieldList<? super INNER, ? super P> list, final Adapter<E, INNER> elementsAdapter) {
+    return new TabFieldList<E, P>() {
+      @Override public void setDefaults(ILaunchConfigurationWorkingCopy configuration, P params) {
+        list.setDefaults(configuration, params);
+      }
+      @Override public void initializeFrom(E elements, ILaunchConfiguration configuration) {
+        list.initializeFrom(elementsAdapter.get(elements), configuration);
+      }
+      @Override
+      public void saveToConfig(E elements, ILaunchConfigurationWorkingCopy configuration) {
+        list.saveToConfig(elementsAdapter.get(elements), configuration);
+      }
+    };
+  }
+
+  static <E, P> TabFieldList<E, P> createCompositeFieldList(
+      final List<? extends TabFieldList<? super E, ? super P>> listList) {
+    return new TabFieldList<E, P>() {
+      @Override
+      public void setDefaults(ILaunchConfigurationWorkingCopy configuration,
+          P params) {
+        for (TabFieldList<?, ? super P> list : listList) {
+          list.setDefaults(configuration, params);
+        }
+      }
+
+      @Override
+      public void initializeFrom(E elements, ILaunchConfiguration configuration) {
+        for (TabFieldList<? super E, ?> list : listList) {
+          list.initializeFrom(elements, configuration);
+        }
+      }
+
+      @Override
+      public void saveToConfig(E elements,
+          ILaunchConfigurationWorkingCopy configuration) {
+        for (TabFieldList<? super E, ?> list : listList) {
+          list.saveToConfig(elements, configuration);
+        }
+      }
+    };
+  }
 }