Обработчики активности не удаляются - PullRequest
Новая
       35

Обработчики активности не удаляются

2 голосов
/ 02 ноября 2011

Я пытаюсь освоить GWT Activity and Places. Я тестирую некоторый исходный код, изначально найденный в этом хорошем блоге .

Я считаю, что обработчики, которые добавляются во время bind (), никогда не удаляются. Мое небольшое понимание javadoc Activity * заставило меня подумать, что они должны быть автоматически удалены к тому моменту, когда вызывается метод onStop () Activity.

Все обработчики событий, которые он зарегистрировал, будут удалены до этого метод называется.

Но каждый раз, когда я нажимаю кнопку, соответствующий обработчик вызывается n + 1 раз.

Что мне не хватает? Пожалуйста, дайте мне знать, если есть больше информации, которую я могу предоставить.

Вот соответствующий фрагмент кода: 

public class ContactsActivity extends AbstractActivity {

private List<ContactDetails> contactDetails;
private final ContactsServiceAsync rpcService;
private final EventBus eventBus;
private final IContactsViewDisplay display;
private PlaceController placeController;

public interface IContactsViewDisplay {
    HasClickHandlers getAddButton();
    HasClickHandlers getDeleteButton();
    HasClickHandlers getList();
    void setData(List<String> data);
    int getClickedRow(ClickEvent event);
    List<Integer> getSelectedRows();
    Widget asWidget();
}

public ContactsActivity(ClientFactory factory) {
    GWT.log("ContactActivity: constructor");

    this.rpcService = factory.getContactServiceRPC();
    this.eventBus = factory.getEventBus();
    this.display = factory.getContactsView();
    this.placeController = factory.getPlaceController();
}

@Override
public void start(AcceptsOneWidget container, EventBus eventBus) {
    GWT.log("ContactActivity: start()");

    bind();
    container.setWidget(display.asWidget());
    fetchContactDetails();

}

public void bind() {

    GWT.log("ContactActivity: bind()");

    display.getAddButton().addClickHandler(new ClickHandler() {
        public void onClick(ClickEvent event) {
            GWT.log("Add button clicked");
            ContactsActivity.this.placeController.goTo(new NewContactPlace(""));
        }
    });

    display.getDeleteButton().addClickHandler(new ClickHandler() {
        public void onClick(ClickEvent event) {
            GWT.log("ContactActivity: Delete button clicked");
            deleteSelectedContacts();
        }
    });

    display.getList().addClickHandler(new ClickHandler() {
        public void onClick(ClickEvent event) {
            GWT.log("ContactActivity: List clicked");
            int selectedRow = display.getClickedRow(event);

            if (selectedRow >= 0) {
                String id = contactDetails.get(selectedRow).getId();
                ContactsActivity.this.placeController.goTo(new EditContactPlace(id));
            }
        }
    });
}

Ответы [ 2 ]

7 голосов
/ 02 ноября 2011

События зарегистрированы через. EventBus, переданный AbstractActivity#start(), будет незарегистрирован ко времени вызова onStop(). Однако обработчики событий, зарегистрированные в вышеуказанном методе bind(), не регистрируются через EventBus и не видны абстрактному базовому классу. Вам нужно отменить их регистрацию самостоятельно: 

public class ContactsActivity extends AbstractActivity {
  private List<HandlerRegistration> registrations = new ArrayList();

  private void bind() {
    registrations.add(display.getAddButton().
      addClickHandler(new ClickHandler() { ... }));
    registrations.add(display.getDeleteButton().
      addClickHandler(new ClickHandler() { ... }));
    registrations.add(display.getList().
      addClickHandler(new ClickHandler() { ... }));
  }

  @Override
  public void onStop() {
    for (HandlerRegistration registration : registrations) {
      registration.removeHandler();
    }

    registrations.clear();
  }
}
0 голосов
/ 20 ноября 2017

Я считаю, что лучше всего обрабатывать регистрацию в представлении - возложить на нее ответственность только за то, чтобы удерживать активным только один щелчок для каждой кнопки.

Вместо: 

class View {
    Button commitButton;

    public HasClickHandlers getCommit () {return commitButton;}
}

.. и ссылкик этому в Деятельности: 

view.getCommit.addClickHandler(new Clickhandler()...

Сделайте это в Представлении: 

    class View {
        private Button commitButton;        
        private HandlerRegistration commitRegistration = null;

        public void setCommitHandler (ClickHandler c) {
            commitRegistraion != null ? commitRegistration.removeRegistration ();
            commitRegistration = commitButton.addClickHandler (c);
        }
    }

И в Деятельности: 

view.setCommitHandler (new ClickHandler () ...

Надежда, которая помогает.

...