Wpis z mikrobloga

@MrFisherman: Przede wszystkim robisz tak żeby rozwiązanie było czytelne dla programisty, który później siądzie i będzie grzebał w Twoim kodzie. Takie optymalizacje nic nie dają, a powodują że jakbym zobaczył 2 sposób to pewnie miałbym ochotę się powiesić.
@MrFisherman: Zdecydowanie czytelniejsze będzie zrobienie seta obiektów dozwolonych i sprawdzenie czy e.getSource() jest zawarty w tym secie.
W tym wypadku kod będzie czytelniejszy, obiekt stworzy się w momencie kompilacji, a sprawdzenie zawierania w secie jest wykonywane w czasie logarytmicznym.
@MrFisherman: Powiedzmy że rozważyłbym postrzelenie się co jest na pewno mniej bolesne ;p Zdecydowanie wyrzuć cały warunek do funkcji np isButtonJakisTam(e.getSource()), zrób np. unmodifiable set buttonów które spełniają ten warunek i w tej funkcji go sprawdzaj.