Java Scoping с ActionListeners

Question

У меня есть приложение, которое должно сканировать устройства Bluetooth каждые 10 минут в течение времени, установленного пользователем.

У меня есть два javax.swing.Timer (не java.util.Timer) - один управляет вызовом метода сканирования каждые 10 минут, другой останавливает первый таймер (прежде чем остановить сканирование), как только достигнут предел продолжительности ,

durationTimer создается и запускается в actionListener.

Моя проблема заключается в том, что так как durationTimer создается в ActionListener, я не могу остановить таймер от другого ActionListener, так как программа не может «видеть» имя переменной «durationTimer».

Сокращенный код показан ниже ....

public class mainGui extends JFrame

{



public mainGui()

  {

    final ActionListener timerActionEvent = new ActionListener(){

        public void actionPerformed(ActionEvent evt){

            //start a task here

            Timer myTimer2 = (Timer) evt.getSource();

            //Invoke BluetoothScan method

            BluetoothScan(myTimer2);

        }

    };







    final Timer timerDuration;

    final Timer myTimer = new Timer(5000, timerActionEvent); 



    final ActionListener timerDurationActionEvent = new ActionListener(){

        public void actionPerformed(ActionEvent evt){

            //Stops myTimer, so that Bluetooth Stops scanning every 10mins. 

            myTimer.stop();

        }

    };






    ActionListener btnScanAction = new ActionListener(){

    //Action listener for reading data from db

    public void actionPerformed(ActionEvent e){

        int roomID = 0;

        int lecturer = 0;

        int unit;

        int roomIDIndex;

        int lectIDIndex;

        int yearIDIndex;

        int unitIDIndex;

        String[] roomArray;

        String[] lecturerArray;

        String[] unitArray = null;

        int durationIndex;

        String DURATION;

        int durationInt;





        //System.out.println(unitArray.length);

        durationIndex = durCB.getSelectedIndex();

        DURATION = itemDuration[durationIndex];

        durationInt = Integer.parseInt(DURATION);



        //User Selected Duration converted to Milliseconds

        int durationMilliSec = (int)(durationInt*60000);



        ArrayList<String[]> unitYear = null;



        //Store the index ID of the JComboBox Selections

        roomIDIndex = roomCB.getSelectedIndex();

        lectIDIndex = lectCB.getSelectedIndex();

        unitIDIndex = unitCB.getSelectedIndex();

        yearIDIndex = yearCB.getSelectedIndex();



        switch(yearIDIndex)

        {

        case 1: unitYear = Units1; break;

        case 2: unitYear = Units2; break;

        case 3: unitYear = Units3; break;

        case 4: unitYear = UnitsMasters; break;

        }



        //Get the Array contents at index location

        roomArray = rooms.get(roomIDIndex);

        lecturerArray = Lecturers.get(lectIDIndex);

        unitArray = unitYear.get(unitIDIndex);





        if(unitArray==null){

            System.out.println("Please select a unit");

            System.exit(0);

        }



        roomID = Integer.parseInt(roomArray[0]);

        lecturer = Integer.parseInt(lecturerArray[0]);

        unit = Integer.parseInt(unitArray[0]);



        populateComboBoxes pcb = new populateComboBoxes();

        pcb.LabSessionInfo(roomID, lecturer, unit);



        myTimer.start();

        Timer timerDuration = new Timer(durationMilliSec, timerDurationActionEvent);

        timerDuration.start();





        }

    };



public void BluetoothScan(Timer myTimer) {

BluetoothDeviceDiscovery scan = new BluetoothDeviceDiscovery();

try {

    myTimer.stop();

    scan.main();

} catch (IOException e) {

    // TODO Auto-generated catch block

    e.printStackTrace();

} catch (InterruptedException e) {

    // TODO Auto-generated catch block

    e.printStackTrace();

}

myTimer.start();



};



  }

Заранее благодарю за любую помощь

Answer 1

Поскольку вы запускаете сканирование синего зуба в потоке отправки событий, ваш второй таймер не сработает до тех пор, пока сканирование синего зуба не будет завершено.

Вам необходимо запустить новую нить для сканирования синего зуба.

Я предлагаю вам использовать SwingWorker.

Answer 2

Вы можете объявить durationTimer на уровне класса и по-прежнему создавать его в ActionListener.Таким образом, это должно быть видно по всему классу.Просто убедитесь, что он не нулевой, прежде чем пытаться вызвать stop () для него.

Другой вариант - заставить его остановиться сам:

final ActionListener timerDurationActionEvent = new ActionListener(){
    public void actionPerformed(ActionEvent evt){
        //Stops myTimer, so that Bluetooth Stops scanning every 10mins. 
        myTimer.stop();
        ((Timer)evt.getSource()).stop();
    }
};

И еще один (и, возможно, лучший) вариант просто сделать его неповторяющимся:

    Timer timerDuration = new Timer(durationMilliSec, timerDurationActionEvent);
    timerDuration.setRepeats(false);
    timerDuration.start();

Этот последний путь.

Answer 3

Я не знаю, когда btnScanAction срабатывает (вероятно, часто, так как это ActionListener), но это создает новый timerDuration при каждом звонке. Вы, вероятно, в конечном итоге получите несколько таймеров продолжительности, что (я думаю ...) не то, что вы хотите.