[Java] 3 karakters op 1 key press

[Preau]

Ik zit met een raar probleempje. Ik ben in Java een RPN (Reverse Polish Notation) calculator aan het maken als opdracht voor school. Ik heb nu de GUI helemaal af maar ik moest hem laten werken met het toetsenbord.
Op zich werkt alles, maar als ik 1 keer op een knop drukt registreert hij dat 3 keer.
Dus als ik 1 keer op 4 druk krijg ik 444.
Als ik 1 keer op * druk probeert hij 3 keer te vermenigvuldigen.

Het volgende staat in de constructor van de klasse

Java:

        KeyboardFocusManager kfm = KeyboardFocusManager.getCurrentKeyboardFocusManager();
        kfm.addKeyEventDispatcher(new KeyEvents());

En dit is een interne klasse in dezelfde klasse als die van de constructor

Java:

    private class KeyEvents implements KeyEventDispatcher{

        public boolean dispatchKeyEvent(KeyEvent e) {
            if(Character.isDigit(e.getKeyChar()) || e.getKeyChar() == '.' || e.getKeyChar() == '\n')
            {
                pnlNumberInput.keyPressed(e.getKeyChar());
                return true;
            }
            if(e.getKeyChar() == '+' || e.getKeyChar() == '-' || e.getKeyChar() == '*' || e.getKeyChar() == '/' || e.getKeyChar() == '^')
            {
                pnlOperators.keyPressed("" + e.getKeyChar());
                return true;
            }
            return false;
        }

    }

Na deze methode kan het in principe niet meer fout gaan omdat in de daarop volgende methodes ook het handmatig met de muis op knoppen drukken afgehandeld wordt en daar 1 keer drukken gewoon 1 keer is.

Ik heb al gezocht naar mensen met hetzelfde probleem maar kon niets vinden.

[NMe]

Krijg je niet toevallig een event voor key down, key press en key up? KeyEvent klinkt redelijk generiek. Let wel: ik ben niet zo bekend met Java dus ik denk hier alleen hardop.

[Tiemez]

NMe schreef op donderdag 14 oktober 2010 @ 18:47:
Krijg je niet toevallig een event voor key down, key press en key up? KeyEvent klinkt redelijk generiek. Let wel: ik ben niet zo bekend met Java dus ik denk hier alleen hardop.

klopt, je kan met getID() opvragen welk event

The getID method is inherited from the AWTEvent class. It returns the event type as a numeric value. For a KeyEvent, the event type is returned as a value of type int, which corresponds to one of the following three constants defined in the KeyEvent class:

* KEY_PRESSED
* KEY_RELEASED
* KEY_TYPED

[Preau]

Ah, over het hoofd gezien, nu werkt het naar behoren. Thanks!

[Infinitive]

Aangezien dit een schoolopdracht is, heb ik nog wat advies voor je wat je al dan niet ten harte kunt nemen.

In zijn algemeenheid zou ik nog willen voorstellen om:

code:

char c = e.getKeyChar();

los te schrijven zodat je in de if-then-else expressies naar c kunt refereren, in plaats van iedere keer e.getKeyChar() voluit te schrijven.

Als alternatief voor de if-then-else's, zou je ook nog naar een switch-statement kunnen kijken.

Tot slot, vind ik het op eerste oogopslag vreemd dat je in pnlOperators.keyPressed naar een string converteert, terwijl je het bij pnlNumberInput.keyPressed bij een char laat. Misschien dat je dat nog wat consistenter kunt maken.

[Voor 9% gewijzigd door Infinitive op 14-10-2010 21:39]