[JAVA]Hulp nodig bij switch statement

[NEOeo]

Ik heb een rekenmachine gemaakt in java, deze werkt zoals ik wil maar nu ik de knoppen MC, MR, MS en M+ erbij heb gezet wil het niet lukken. Het is namelijk zo dat mijn switch statement naar het 1e character kijkt...
Kan iemand mij helpen hiermee?

Het deel van de code:

code:

    private class KnopHandler implements ActionListener { 
        public void actionPerformed(ActionEvent e) {
            String knop = e.getActionCommand();

            switch (knop.charAt(0)) {
                case '=':
                    schermwaarde = vak.getText();
                    getal2 = Double.parseDouble(schermwaarde);
                    vak.setText("");
                    waarde = "";
                    punt = false;
                    if (bewerking =="plus") {
                    uitkomst = getal1 + getal2;
                    }
                    if (bewerking =="min") {
                    uitkomst = getal1 - getal2;
                    }
                    if (bewerking =="delen") {
                    uitkomst = getal1 / getal2;
                    }
                    if (bewerking =="maal") {
                    uitkomst = getal1 * getal2;
                    }
                    vak.setText(""+uitkomst+"");
                    break;
                    
                case '+':
                    schermwaarde = vak.getText();
                    waarde = "";
                    getal1 = Double.parseDouble(schermwaarde);
                    bewerking = "plus";
                    punt = false;
                    break;
                    
                case '-':
                    schermwaarde = vak.getText();
                    waarde = "";
                    getal1 = Double.parseDouble(schermwaarde);
                    bewerking = "min";
                    punt = false;
                    break;
                    
                case '*':
                    schermwaarde = vak.getText();
                    waarde = "";
                    getal1 = Double.parseDouble(schermwaarde);
                    bewerking = "maal";
                    punt = false;
                    break;
                    
                case '/':
                    schermwaarde = vak.getText();
                    waarde = "";
                    getal1 = Double.parseDouble(schermwaarde);
                    bewerking = "delen";
                    punt = false;
                    break;
                    
                case '.':
                    if ((waarde !="") && (punt == false)){
                        waarde = waarde + ".";
                        vak.setText(waarde);
                        punt = true;}
                    break;
                    
                case 'MC':
                    waarde = "";
                    getal3 = 0;
                    punt = false;
                    break;
                
                case 'MR':
                    vak.setText(""+getal3+"");
                    waarde = "";
                    punt = false;
                    break;
                    
                case 'MS':
                    schermwaarde = vak.getText();
                    getal3 = Double.parseDouble(schermwaarde);
                    waarde = "";
                    punt = false;
                    break;
                    
                case 'M+':
                    schermwaarde = vak.getText();
                    getal4 = Double.parseDouble(schermwaarde);
                    getal3 = getal3 + getal4;
                    waarde = "";
                    vak.setText(""+getal3+"");
                    punt = false;
                    break;
                    
                default:
                    waarde = waarde + knop;
                    vak.setText(waarde);
                    break;
            }
        }
    }

[Marcj]

Je hebt een 'MC' etc.. als character, terwijl dit een string is. Een character is namelijk altijd een letter (in dit geval dus de 'M'). Wat je zou kunnen doen is een case 'M': aanmaken en onder deze een switch die de tweede letter controleerd.

Is de java compiler niet duidelijk genoeg? Want die zegt hier toch duidelijk: "Invalid character constant" op regel 66, 72, 78 en 85.

[ Voor 21% gewijzigd door Marcj op 22-02-2007 11:12 ]

[Foutlook]

Eh, controleren in de case "M" met behulp van een if-else welke "M-knop" je te pakken hebt?

[Wouter Tinus]

Volgens mij kun je beter aparte event handlers bouwen voor al je knoppen. Dit is niet echt de geest van OO .

[RobIII]

^^Wat hij zegt

Daarnaast is het niet de bedoeling dat je hier een ("flinke") lap code post en dan van ons verwacht dat we je zo de oplossing geven of het voor je voorkauwen (zie ook scriptrequest / quickfix)

NEOeo schreef op donderdag 22 februari 2007 @ 11:06:
Het is namelijk zo dat mijn switch statement naar het 1e character kijkt...

Je weet gewoon wat het probleem is; zo inventief hoef je ook weer niet te zijn om te beseffen dat je dan ook naar het 2e karakter moet kijken. Je kunt dus, als je op deze manier door wil breien, Marcj's oplossing prima gebruiken.

[ Voor 46% gewijzigd door RobIII op 22-02-2007 11:17 ]

[NEOeo]

marcj zou je mij opweg kunnen helpen?

[Marcj]

NEOeo schreef op donderdag 22 februari 2007 @ 11:26:
marcj zou je mij opweg kunnen helpen?

Als ik jouw was zou ik eerst wat meer tutorials over Java gaan bekijken. Als je hier niet uitkomt met de hint die ik je gegeven heb, zul je toch eerst wat meer over Java moeten leren. Kijk daarnaast ook naar was Wouter zegt, dit is niet echt een nette manier van OO programmeren. Op deze manier blijven uitbreiden is lastig en gevoelig voor errors. Wat gaat er bijvoorbeeld gebeuren als je een knop met de tekst "++" gaat toevoegen?? Het is beter om aparte event handlers daarvoor aan te maken.

[whoami]

Sorry, maar zo werkt het hier niet.
Zoals RobIII al aangeeft, willen we echt wel helpen, maar verwachten we toch wel wat inzet van jou. Het is echt niet de bedoeling dat je anderen jouw code laat fixen, zonder dat je ook maar enige inzet toont.
Zowiezo kan je -zoals gezegd- beter met event-handlers werken; indien je toch bij je switch statement wil blijven, dan moet je eens kijken wat je checked in je switch statement (het eerste character), en waarop je test (in sommige gevallen op 2 characters, wat een string is).