Programm tut nicht alles, was es soll

[kellog]

Hi,

ich möchte einen Taschenrechner programmieren, bei dem man die Rechenoperation und die Operanden als Kommandozeile eingibt, so dass man das Programm Taschenrechner z.B. mit java Taschenrechner 3 mul 6 aufruft, wenn man wissen möchte, wie viel 3*6 ist.

Mein Code kompiliert ohne Fehlermeldung und das Programm funktioniert auch soweit, nur die Zeilen 8-10 werden nicht ausgeführt, d.h. auch wenn ich weniger als 3 Argumente eingebe, bekomme ich nicht meine Fehlermeldung aus Zeile 9 ausgegeben.
:?: Wer kann mir sagen, was denn an meinem Code nicht passt :?:
Denn ich finde meinen Fehler einfach nicht!!!

public class Taschenrechner{
    public static void main(String[] args) {
        int a = new Integer(args[0]).intValue();
        double x = (double) a;
        int b = new Integer(args[2]).intValue();
        double y = (double) b;

        if (args.length < 3) {
            System.out.println("Ihre Eingabe war fehlerhaft!!!");                            
        }
        
        else {
            if (args[1].equals("add")) {
                System.out.println(x+y);    
            }   else if (args[1].equals("sub")) {
                    System.out.println(x-y);
                }   else if (args[1].equals("mul")) {
                        System.out.println(x*y);
                    }   else if (args[1].equals("div")) {
                            if (y == 0) 
                                System.out.println("Division durch Null ist nicht erlaubt!");
                            else 
                                System.out.println(x/y);
                        }
        }
        System.exit(0);
    }
}

 

[Leroy42]

Du greifst ja schon auf args[2] zu, bevor du die Anzahl der Argumente prüfst.

Mache zuerst die Längenabfrage bevor du auf irgendein Element
von args zugreifst.

P.S.: Warum hast du die ArrayIndexOutOfBounds-Exception nicht gepostet?

 

[sonnendeck]

Du musst die Längenüberprüfung an den Anfang machen:

public class Taschenrechner
{
    public static void main(String[] args)
    {
        if (args.length < 3) // besser: args.length !=3
        {
            System.out.println("Ihre Eingabe war fehlerhaft!!!");
        }
        else
        {
            // die Variablen brauchen nur angelegt werden, wenn die Laenge stimmt
            int a = new Integer(args[0]).intValue();
            double x = (double) a;
            int b = new Integer(args[2]).intValue();
            double y = (double) b;
            
            if (args[1].equals("add"))
            {
                System.out.println(x + y);
            }
            else if (args[1].equals("sub"))
            {
                System.out.println(x - y);
            }
            else if (args[1].equals("mul"))
            {
                System.out.println(x * y);
            }
            else if (args[1].equals("div"))
            {
                if (y == 0)
                    System.out
                            .println("Division durch Null ist nicht erlaubt!");
                else
                    System.out.println(x / y);
            }
        }
        // Das ist nicht nötig
        System.exit(0);
    }
}

Ich hoffe das ist ok.

 

[kellog]

Danke, jetzt klappts!!