我目前正在研究现有的GWT项目。在这个项目中,我在视图上添加了一个列表,看起来像这样:


标签1:[文本框]
标签2:[DateBox]
Label3:[文本框]
Label4:[文本框]
标签5:[DateBox]
Label6:[文本框]


我制作了一个从FlowPanel派生的小部件,在其中创建了这些Label-InputBox对:

import java.sql.Date;

import com.google.gwt.dom.client.Style.Display;
import com.google.gwt.user.client.ui.FlowPanel;
import com.google.gwt.user.client.ui.Label;
import com.google.gwt.user.client.ui.TextBox;
import com.google.gwt.user.client.ui.Widget;
import com.google.gwt.user.datepicker.client.DateBox;

public class LabelInputWidget extends FlowPanel {

  private final Label      label;
  private final Widget     inputField;

  private static final int LABEL_AND_INPUTFIELD_WIDTH = 200;

  public LabelInputWidget(final String labelText, final Widget inputField) {
    super();

    this.label = new Label(labelText);
    this.label.getElement().getStyle().setDisplay(Display.INLINE_BLOCK); // Used to put the inputfield next to the label, instead of below it
    this.label.setWidth(LABEL_AND_INPUTFIELD_WIDTH + "px");
    this.add(this.label);

    this.inputField = inputField;
    // The DateBox width looks smaller when the same width is set, so we'll add 4 pixels to line it up with the other Input-Widgets
    int inputWidth = LABEL_AND_INPUTFIELD_WIDTH;
    if (inputField instanceof DateBox) {
      inputWidth += 4;
    }
    this.inputField.setWidth(inputWidth + "px");
    this.add(this.inputField);
  }

  // TODO KC: Use a less ugly way of determine which InputField is used
  // POTENTIAL TODO KC: Use custom widgets for Numeric / Currency input fields
  public void setInputValue(final String value) {
    if (this.inputField instanceof TextBox) {
      ((TextBox) this.inputField).setValue(value);
    } else if (this.inputField instanceof DateBox) {
      ((DateBox) this.inputField).setValue(value != null && value.length() > 0 ? Date.valueOf(value) : null);
    }
  }
}


在创建这些字段的文件中,我具有以下内容:

private void fillEditedProperties(final List<Property> list) {
  for (final Property p : list) {
    if (!containsProperty(p)) {
      this.editedProperties.add(p);

      // The moment the property is loaded and known, we add it to the view
      Widget inputWidget = new TextBox();
      if (Type.DATUM.name().equals(p.getPdf().getType().name())) {
        final DateBox dateBox = new DateBox();
        dateBox.setFormat(new DateBox.DefaultFormat(DateTimeFormat.getFormat("dd-MM-yyyy")));
        inputWidget = new DateBox();
      }
      final LabelInputWidget labelInput = new LabelInputWidget(p.getPdf().getName(), inputWidget);
      labelInput.setInputValue(p.getValue());
      labelInput.getElement().getStyle().setDisplay(Display.BLOCK); // Used to put the LabelInputWidgets below each other
      this.labelInputFieldsContainer.add(labelInput);
    }
  }
}


而且我还想在用户输入内容(待办事项/正在进行的工作)时验证输入。

如您所见,将常规com.google.gwt.user.client.ui.Widget用作TextBox和DateBox的共享基类非常难看。因为它们不共享InputBox类之类的东西,所以我必须在确定使用哪个(Input)Widget之后设置它们的值,并且可能对ValueChangedHandlers进行同样的操作以验证用户将要创建的用户输入。

有谁知道在同一个LabelInputWidgets列表中同时使用TextBoxes和DateBoxes的更好解决方案,而不是在几乎所有地方都使用丑陋的instanceof,而仅使用一种通用方法?

如果不是这样,我可能会将这种行为隐藏在某个地方,以使代码更具可读性。

PS:我不是100%地确定这是适用于StackOverflow的问题,因此我也对CodeReview.StackExchange.com提出了疑问。我也将其发布在这里有两个原因:


有人可能会以完全不同的方式为此找到合适的解决方案。
这里的社区更大,所以我可能会更快地得到答案。

最佳答案

好吧,您可以使LabelInputWidget抽象,并使其具有两种实现(基于日期的实现和基于Textbox的实现),并重载方法。

关于java - GWT TextBox,DateBox等不共享相同的基本输入类,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/35365674/

10-10 13:24