установка команды слушателя предпочтения стиля сомнения - PullRequest
Новая
       2

установка команды слушателя предпочтения стиля сомнения

2 голосов
/ 06 сентября 2011

Почему некоторые люди предпочитают использовать 

.setCommandListener(this)

над 

.setCommandListener(new CommandListener(){})

чаще? В каком случае я должен использовать второй и почему? Я предполагаю, что это просто вопрос стиля или есть конкретная проблема?

Ответы [ 2 ]

1 голос
/ 06 сентября 2011

Если вы используете «this», вы должны реализовать слушатель класса, а затем вы можете получить доступ к полям класса в реализованных методах слушателя.

Если вы используете второй (новый Слушатель ...), то он может быть более читабельным, если вам не требуется доступ ко многим другим вещам в вашем классе.

0 голосов
/ 06 сентября 2011

setCommandListener(this) заметно легче читать в «игрушечном коде».Я думаю, именно поэтому я видел, что он используется во многих учебниках начального уровня, где авторы просто не хотят углубляться.

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

Хотя для более сложного кода, по моему опыту, setCommandListener(new CommandListener(){/*..*/}) было намного проще поддерживать и читать.

Обратите внимание, что выможет получить доступ к полям класса в обоих случаях, только последний требует использования Квалифицировано это : 

//import javax.microedition.midlet.*;
//import javax.microedition.lcdui.*;

abstract class ListenerTest extends MIDlet {
    protected Display display;

    protected void startApp() {
        display = Display.getDisplay(this);
        Form form = new Form("welcome");
        form.addCommand(new Command("go", Command.OK, 1));
        form.setCommandListener(new CommandListener() {
            public void commandAction(Command c, Displayable d) {
                // qualified this - see JLS 15.8.4
                ListenerTest.this.cmdAction(c, d);
            }
        });
        // display "welcome" screen with "go" command
        display.setCurrent(form);
    }

    protected void pauseApp() { }

    protected void destroyApp(boolean unconditional) {
        notifyDestroyed();
    }

    protected abstract void displayNext();

    private void cmdAction(Command c, Displayable d) {
        // invoke from listener to display next screen
        displayNext();
    }
} // ListenerTest

class NextTest extends ScreenTest {

    protected void displayNext() {
        Form form = new Form("bye-bye");
        form.addCommand(new Command("EXIT", Command.EXIT, 1));
        form.setCommandListener(new CommandListener() {
            public void commandAction(Command c, Displayable d) {
                notifyDestroyed();
            }
        });
        // display "bye-bye" screen with "exit" command
        display.setCurrent(form);
    }
} // NextTest

Кстати, я упоминал, что вышеупомянутый подход также безопаснее?Это гарантирует, что слушатель, которого вы ожидаете для определенного экрана, именно тот, который вы установили.

Скажем, если вы сделаете прямую перезапись на setCommandListener (this) и запустите его, выобратите внимание на странное поведение - команда "go" теперь будет выходить из мидлета вместо отображения следующего экрана: 

    // don't do that
    abstract class ListenerTest extends MIDlet implements CommandListener {
        protected Display display;

        protected void startApp() {
            display = Display.getDisplay(this);
            Form form = new Form("welcome");
            form.addCommand(new Command("go", Command.OK, 1));
            form.setCommandListener(this);
            // display "welcome" screen with "go" command
            display.setCurrent(form);
        }

        protected void pauseApp() { }

        protected void destroyApp(boolean unconditional) {
            notifyDestroyed();
        }

        protected abstract void displayNext();

        public void commandAction(Command c, Displayable d) {
            // invoke from listener... really??? check the subclass
            displayNext();
        }
    } // ListenerTest

    class NextTest extends ScreenTest implements CommandListener {

        protected void displayNext() {
            Form form = new Form("bye-bye");
            form.addCommand(new Command("EXIT", Command.EXIT, 1));
            form.setCommandListener(this);
            // display "bye-bye" screen with "exit" command
            display.setCurrent(form);
        }

        public void commandAction(Command c, Displayable d) {
            // you may not notice but...
            notifyDestroyed();
            // ...this actually overrides superclass implementation
        }
    } // NextTest
...