🚀 go-pugleaf

W dniu 2015-07-23 o 13:23, Borneq pisze:
> Ma przycisk, którego naciśnięcie powoduje uruchomienie dość długiej
> procedury, tak z 5 sekund. Gdy kliknę na niego kilka razy szybko,
> procedura zostaje uruchamiana wielokrotnie od początku. Próbowałem
> setEnabled(false):

Jeśl deaktywowanie buttona nie działa tak jak oczekujesz, to przychodzą 
mi do głowy dwa rozwiązania:
A) użycie flagi którą sprawdzasz i ustawiasz wewnątrz ActionPerformed
B) przeniesienie długotrwałej operacji do oddzielnego wątku.


-- 
pozdrawiam
Piotr
XLR250&bmw_f650_dakar

W dniu 2015-07-23 o 14:02, Piotr Rezmer pisze:
> Jeśl deaktywowanie buttona nie działa tak jak oczekujesz, to przychodzą
> mi do głowy dwa rozwiązania:
> A) użycie flagi którą sprawdzasz i ustawiasz wewnątrz ActionPerformed
> B) przeniesienie długotrwałej operacji do oddzielnego wątku.

To chyba co innego i chyba dlatego button nie działa tak jak powinien:
zrobiłem:
		public void actionPerformed(ActionEvent e) {
			if (duringOperation == false) {
				duringOperation = true;
				buttonBench.setEnabled(false);
				if (originalName != null)
					printBenchmarks(originalName);
				buttonBench.setEnabled(true);
				duringOperation = false;
			}
			}

to zabezpieczyło by przed zdarzeniami, które przyszły by podczas 
wielokrotnego wołania procedury roboczej. A tutaj jest jakieś 
kolejkowanie, nie wiem, jak bym tu zrobił runnable, to czy nie 
potworzyło by wielu wątków. Potrzebne jest coś w rodzaju globalnego 
czyszczenia zdarzeń kliku myszy. Ale to nie jest takie wazne, to tylko 
procedura porównująca algorytmy, najważniejsze siedzi w algorytmach.

Dnia 23.07.2015 Borneq <borneq@antyspam.hidden.pl> napisał/a:
> Ma przycisk, którego naciśnięcie powoduje uruchomienie dość długiej 
> procedury, tak z 5 sekund. Gdy kliknę na niego kilka razy szybko, 
> procedura zostaje uruchamiana wielokrotnie od początku. Próbowałem 
> setEnabled(false):
>          buttonBench.addActionListener( new ActionListener()
> 	{
> 		public void actionPerformed(ActionEvent e)
> 		{
> 		    	buttonBench.setEnabled(false);
> 		    	if (originalName!=null)
> 		    		printBenchmarks(originalName);
> 		    	buttonBench.setEnabled(true);
> 		}
> 	});
>
> ale coś się dzieje - gromadzi kliknięcia na zablokowanym przycisku i 
> nadal jest źle. Może da się napisać klasę buttona dziedziczącą z JButton 
> gdzie to było by poprawione?

Jeśli poczytasz API dla javax.swing.JComponent.setEnabled (boolean) , to
znajdziesz tam:
Note: Disabling a lightweight component does not prevent it from receiving
MouseEvents.
Wszyskie komponenty Swing są "lightweights" więc będą odbierać myszkę nawet
jeśli są wyłączone (z wszelkimi tego konsekwencjami), dlatego jeśli już
miałbym do tego podejść na Twój sposób, to bym to zrobił na zasadzie:

public void actionPerformed(ActionEvent e) {
  if(buttonBench.isEnabled()) {
    buttonBench.setEnabled(false);
    tu_akcja();
    buttonBench.setEnabled(true);
  }
}

W dniu 2015-07-23 o 15:12, Borneq pisze:
> kolejkowanie, nie wiem, jak bym tu zrobił runnable, to czy nie
> potworzyło by wielu wątków. Potrzebne jest coś w rodzaju globalnego

Tak, w wątkach, ale przy sprawdzaniu terminated:
field:
BenchWorker worker

class BenchWorker extends SwingWorker<Integer, Integer> {
		String originalName;
		protected boolean terminated;

		BenchWorker(String name) {
			originalName = name;
			terminated = false;
		}

		@Override
		protected Integer doInBackground() throws Exception {
			if (originalName != null)
				printBenchmarks(originalName);
			terminated = true;
			return null;
		}
	}


   -------------------
	  buttonBench.addActionListener(new ActionListener() {
	      public void actionPerformed(ActionEvent e) {
		buttonBench.setEnabled(false);
		if (worker == null || worker.terminated == true) {
				worker = new BenchWorker(originalName);
					worker.execute();
				}
				buttonBench.setEnabled(true);
			}
		});

Działa!