try/catch-Block bei for-Schleife

[drei_und_20]

Guten Abend. Ich versuche bei dieser Methode zum Array einlesen einen try/catch-Block mit einer InputMismatchException einzufügen, so dass bei Eingabe eines Buchstabens die for-Schleife und Eingabe über den Scanner erneut abläuft. Leider wird dieser bei bisher jedem meiner Versuch abgebrochen.

    public void arrayEinlesen() {
        Scanner sc = new Scanner(System.in);
        for (int i = 0; i < array.length; i++) {
            System.out.println("Geben sie eine Zahl ein. <0 zum beenden>");
            array[i] = sc.nextInt();
            if (array[i] <= this.obereGrenze && array[i] >= this.untereGrenze && array[i] != 0) {
                System.out.println("Zahl korrekt");
                anzKorrekt++;
            } else if (array[i] >= this.obereGrenze || array[i] <= this.untereGrenze && array[i] != 0) {
                System.out.println("Zahl nicht korrekt");
                i--;
                anzFalsch++;
            } else if (array[i] == 0) {
                break;
            }
        }
        arrayNeu = Arrays.copyOf(array, anzKorrekt);

    }

Kann mir dabei evt jemand helfen?
Vielen Dank und lg

 

[Blender3D]

Leider wird dieser bei bisher jedem meiner Versuch abgebrochen.

Zerlege das Ganze in Funktionen.
[CODE lang="java" highlight="9-13,16-34,36-53"]import java.util.Arrays;
import java.util.InputMismatchException;
import java.util.Scanner;

public class StartArrayInput {
static Scanner input = new Scanner(System.in);
public static void main(String[] args) {
int[] array = new int[10];
System.out.println("Abruch durch eine Zahl <0!");
InputStatistic statistic = inputIntArray(array, 10, 30);
System.out.println(Arrays.toString(array));
System.out.println(statistic);
}

public static InputStatistic inputIntArray(int[] array, int min, int max) {
InputStatistic statistic = new InputStatistic();
for (int i = 0; i < array.length; i++) {
try {
array = inputInt((i + 1) + ". Zahl bitte eingeben", min, max);
if (array < 0) {
array = 0;
return statistic;
}
System.out.println("Zahl korrekt");
statistic.correct();
} catch (InputMismatchException e) {
System.out.println("Zahl nicht korrekt");
statistic.error();
i--;
}
}
return statistic;
}

private static int inputInt(String label, int min, int max) throws InputMismatchException {
boolean ok = false;
int value = 0;
do {
System.out.print(label + " [" + min + "-" + max + "]" + " >");
try {
value = Integer.parseInt(input.next());
if (value < 0)
return value;
if (value < min || value > max)
throw new InputMismatchException("Wert liegt nicht im Bereich [" + min + "-" + max + "]");
ok = true;
} catch (NumberFormatException e) {
System.out.println("Bitte eine ganze Zahl eingeben!");
}
} while (!ok);
return value;
}
}[/CODE]
[CODE lang="java" title="InputStatistic"]public class InputStatistic {
private int error = 0;
private int correct = 0;

public void correct() {
correct++;
}

public void error() {
error++;
}

@Override
public String toString() {
return "RICHTIG " + correct + ", FALSCH " + error;
}
}[/CODE]

 

[Jw456]

Na den Code habe ich doch erst vor kurzen gesehen hier im Forum.

Der User / Account der das gepostet hatte gibt es nicht mehr, das scheinst du gewesen zu sein.
Wieso ein neuer User?

Du hattest Probleme mit dem Konstruktor habe da auch Antwort gegeben jetzt ist der User gelöscht schon merkwürdig bei jedem Problem einen neuen User anzulegen.

 

[Jw456]

Ich würde es nicht mit Try/Catch machen.

Ich würde mit eine while prüfen ob es ein int ist.
Die solange läuft bis ein int eingeben wurde.
Falsche Eingaben musst du natürlich in der Schleife abholen. Sonnst geht es nicht weiter.
deshalb das String stline = sc.nextLine();

public void arrayEinlesen() {
        Scanner sc = new Scanner(System.in);
        for (int i = 0; i < array.length; i++) {
            System.out.println("Geben sie eine Zahl ein. <0 zum beenden>");
         
            while (!sc.hasNextInt()) {
                String stline = sc.nextLine();
      
            }     
                     
            array[i] = sc.nextInt();
            .....

 

[Mart]

Ich würde es nicht mit Try/Catch machen.

eine Ausführliche Begrüdnung wäre , wenn du durch abfragen auf Gültigkeit nicht in das try catch läufst bist du besser unterwegs als mit try catch

zb eine additions Methode

public int addieren(int summandEins, int summandZwei)
{
    return summandEins + summandZwei;
}

diese Methode kannst du aufteilen auf eine Public und private variante
i n der public fangst du "dämliche"werte ab zb wenn 1e zahl 0 ist

public int addieren(int summandEins,int summandZwei)
{
    if(summandZwei == 0){
        return summandEins;
    }
    return addierenIntern(...);
}
private int addierenIntern(...){
    ...
}

gut macht das beispiel viel sinn es hier anzuwenden? nien...

1 Beispiel wo genau das system angewandt wird ist beim Binär Baum, wenn du die Knoten Zählen musst die drin sind, wenn der Bamu aber leer ist läufst du in eine Nullpointer Exception aber die Methode müsste eig richtig sein wenn

if(baum = leer)
{
    return 0;
}

stimmt ja der baum hat 0 knoten

die exception wäre hier aber kontra produktiv gewesen, weil das ergebnis falsch gewesen wäre... die methode musste halt nur den Grenzfall von leer sein abdecken ( manchmal nur mit exception möglich )

 

[Jw456]

Es geht ja erst mal darum das das Zeichen was von der Tastatur kommt auch eine Ziffer ist und nicht etwa ein Buchstabe oder Sonderzeichen was dann die Scanner nextint Methode nicht in einen int parsen kann.

Ja man kann oder sollte es in eine eigene Methode packen.


Aber dein Beispiel mit der Addition
ist am Tehma vorbei.
Es geht um das prüfen eines Daten Typs.

 

[drei_und_20]

Na den Code habe ich doch erst vor kurzen gesehen hier im Forum.

Der User / Account der das gepostet hatte gibt es nicht mehr, das scheinst du gewesen zu sein.
Wieso ein neuer User?

Du hattest Probleme mit dem Konstruktor habe da auch Antwort gegeben jetzt ist der User gelöscht schon merkwürdig bei jedem Problem einen neuen User anzulegen.

Nein ich hab keinen neuen Account ich musste nur den Beitrag löschen weil ich ne Meldung bekommen habe, dass ich die Aufgabenstellung nicht hätte hier veröffentlichen dürfen. Daher jetzt ein neuer Beitrag mit nur diesem Teilaspekt und ohne Aufgabenstellung.

 

[Jw456]

Na dann weißt du ja was ich schon gesagt geschrieben hatte.
Wirst es dir wohl kopiert haben.

Wir du das Thema lösen kannst habe ich ja nun gezeigt.

 

[Mart]

Nein ich hab keinen neuen Account ich musste nur den Beitrag löschen weil ich ne Meldung bekommen habe, dass ich die Aufgabenstellung nicht hätte hier veröffentlichen dürfen.

ich lehn mich ma weit ausm fenster und finde nach paar minuten im internet die gleiche aufgabe ...

 

[Jw456]

@Mart das glaube ich dir.

@drei_und_20 wer hat dich den darauf hingewiesen das du das nicht darfst?
Dein Lehrer/Prof oder ein Admin/User vom Forum?

 

[drei_und_20]

ich lehn mich ma weit ausm fenster und finde nach paar minuten im internet die gleiche aufgabe ...

Tut mir leid, ich finde die Aufgabe nicht. Aber ich verstehe auch nicht so ganz was das Problem ist. Ich sollte die Aufgabenstellung rausnehmen und habe jetzt eine neue Frage gestellt nur ohne die konkrete Aufgabe zu veröffentlichen. Trotzdem danke für die Hilfe.

 

[Mart]

public boolean isInt(String input) {
    try {
        Integer.parseInt(input);
        return true;
    } catch (final NumberFormatException e) {
        return false;
    }
}

so könntest du die Überprüfungs methode schreiben die ausgeführt wird bevor du weiter machst aber das war ja am content vorbei

 

[Jw456]

ich habe ihm doch schon eine Lösung gegeben ohne try & catch.

für was gibst es denn sonst die "has..... " Methoden von dem Scanner. Doch wohl um zu prüfen was im Tastaturbuffer ist.

while (!sc.hasNextInt()) {
     String stline = sc.nextLine();
     
  }

PS @drei_und_20 schon kommisch das du nicht zu den genanten Lösungen sagst .

 

[drei_und_20]

while (!sc.hasNextInt()) {
     String stline = sc.nextLine();
    
  }

Tut mir leid, habe ich versucht, aber irgendetwas scheine ich doch falsch eingegeben zu haben, sodass das bei mir nicht zur gewünschten Lösung führt. Weiß einfach nicht was ich mit dem neuen String dann machen soll. Aber wenn man gerade erst mit Java anfängt sieht man manchmal die einfachsten Dinge nicht...

 

[Jw456]

Eigentlich brauchst nicht damit machen dient nur zum abholen des Tastatur Buffers wenn kein gültiger "int" im Buffer ist.

Ok du bleibst in der Schleife bekommst keine Ausgabe wen etwas falsches Eingegeben wurde.
Die Schleife wartet bis ein echter „int“ Wert eingegeben wurde. Eine Eingabe ist immer mit Return zu beenden erst dann wird geprüft.
Ist aber bei nextInt() auch so das da auf das Return gewartet wird. Und erst jetzt das Parsen beginnt.
Wenn ein int im Buffer ist geht es nicht in schleife sondern weiter zum nextInt() und das parsen erfolgt ohne Fehler.
Wenn es in die Schleife geht musst du das falsche aus dem Buffer holen, das machst mit der hilfst Variable . Zu mehr ist sie nicht da.

Wenn du eine Fehler Ausgabe haben willst etwa so.

for (int i = 0; i < array.length; i++) {
            System.out.println("Geben sie eine Zahl ein. <0 zum beenden>");

            while (!sc.hasNextInt()) {
                String stline = sc.nextLine();
                if (!stline.trim().isEmpty()) {
                    System.out.println("nur ganze Zahlen");
                }
            }

            array[i] = sc.nextInt();
            System.out.println("es wurde " +array[i]+ " eingeben");