Argumente auf plausibilität prüfen...

[petergunter]

Hey ich bin (komplett) unerfahren und hoffe jemand hilft mir...
Ich möchte das erste Argument (von 3) auf plausibilität prüfen... Es wird eine Zahl eingegeben und diese darf nicht 0 sein, muss positiv und keine Kommazahl sein. Verzweifle irgendwie :/
Mfg!

 

[Flown]

Was hast du schon versucht? Wie sieht deine Methoden Signatur aus?

 

[petergunter]

public static int checkErsteZahl(args [0] string eingabe, int) schon bei der signatur hab ich probleme...

 

[Flown]

Was soll denn deine Methode an Argumenten erhalten? Ein String array?

 

[Xyz1]

So funzt es:

/**
* @author
*/
public class Gunter {

    public static int checkErsteZahl(String... args) {
        if (args == null || args.length < 1 /* || args[0].isEmpty() */) {
            throw new IllegalArgumentException("args empty...");
        }
        if (!isValid(args[0])) {
            throw new IllegalArgumentException("not valid... " + args[0]);
        }
        return Integer.parseInt(args[0]);
    }

    private static boolean isValid(String arg) {
        try {
            int i = Integer.parseInt(arg);
            return i > 0;
        } catch (NumberFormatException ex) {
            return false;
        }
    }

    public static void main(String[] args) {
        try {
            System.out.println(checkErsteZahl(null));
        } catch (Exception e) {
            System.out.println(e);
        }
        try {
            System.out.println(checkErsteZahl(""));
        } catch (Exception e) {
            System.out.println(e);
        }
        try {
            System.out.println(checkErsteZahl("null"));
        } catch (Exception e) {
            System.out.println(e);
        }
        try {
            System.out.println(checkErsteZahl("-1.0"));
        } catch (Exception e) {
            System.out.println(e);
        }
        try {
            System.out.println(checkErsteZahl("-1"));
        } catch (Exception e) {
            System.out.println(e);
        }
        try {
            System.out.println(checkErsteZahl("0.0"));
        } catch (Exception e) {
            System.out.println(e);
        }
        try {
            System.out.println(checkErsteZahl("0"));
        } catch (Exception e) {
            System.out.println(e);
        }
        try {
            System.out.println(checkErsteZahl("1.0"));
        } catch (Exception e) {
            System.out.println(e);
        }
        System.out.println(checkErsteZahl("+1"));
    }
}

----
Und Ausgabe:

java.lang.IllegalArgumentException: args empty...
java.lang.IllegalArgumentException: not valid...
java.lang.IllegalArgumentException: not valid... null
java.lang.IllegalArgumentException: not valid... -1.0
java.lang.IllegalArgumentException: not valid... -1
java.lang.IllegalArgumentException: not valid... 0.0
java.lang.IllegalArgumentException: not valid... 0
java.lang.IllegalArgumentException: not valid... 1.0
1

 

[Xyz1]

Noch etwas vergessen, wenn man checkErsteZahl(new String[]{null}) aufruft, krachts...
Deswegen:

        if (args == null || args.length < 1 || args[0] == null || args[0].isEmpty()) {
            throw new IllegalArgumentException("args empty...");
        }
        // ...

 

[petergunter]

danke für die hilfe =)

 

[Xyz1]

Also, du brauchst erst eine Methode, die prüft, ob das Argument gültig ist, und wenn nicht, dann wird "oft" eine bestimmte Fehlermeldung zurückgegeben. Wenn das alles nicht der Fall ist, dann geht's weiter im Programm (Integer.parseInt(args[0])).

 

[Xyz1]

Und das lässt sich weiter aufblähen:

    public static int checkErsteZahl(String... args) {
        if (args == null) {
            throw new IllegalArgumentException("args null...");
        }
        if (args.length < 1) {
            throw new IllegalArgumentException("args < 1...");
        }
        if (args[0] == null) {
            throw new IllegalArgumentException("arg null...");
        }
        if (args[0].isEmpty()) {
            throw new IllegalArgumentException("arg empty...");
        }
        if (!isValid(args[0])) {
            throw new IllegalArgumentException("not valid... " + args[0]);
        }
        return Integer.parseInt(args[0]);
    }

    private static boolean isValid(String arg) {
        try {
            int i = Integer.parseInt(arg);
            return i > 0;
        } catch (NumberFormatException ex) {
            return false;
        }
    }