warum will punkt 2 nicht?

Frage zur Aufgabe

In der Diskussion

schaut was ich für einen schönen Code geschrieben habe ha! :D ihr glaubt nicht wie stolz ich darauf bin, danke

Größter gemeinsamer Teiler (GGT).
Gib 2 positive ganze Zahlen über die Tastatur ein.
Zeige den größten gemeinsamen Teiler an.

Anforderungen:

Das Programm muss zwei Zeilen von der Tastatur einlesen.
Wenn die eingegebenen Zeilen nicht in positive ganze Zahlen umgewandelt werden können, löse eine Ausnahme aus.
Das Programm muss Daten auf dem Bildschirm anzeigen.
Das Programm muss den größten gemeinsamen Teiler (GGT) der von der Tastatur gelesenen Zahlen anzeigen und sich dann erfolgreich beenden.

package de.codegym.task.task14.task1420; /* GGT */ import java.util.Scanner; import java.util.ArrayList; public class Solution { public static void main(String[] args) throws Exception { Scanner scan = new Scanner(System.in); try { int a = scan.nextInt(); int b = scan.nextInt(); System.out.println(GGT(a, b)); } catch (Exception e){ System.out.println(e); } } public static int GGT (int a, int b){ ArrayList<Integer> liste = new ArrayList<>(); int big; if(a > b) big = a; else big = b; for(int i = 1; i < big; i++){ if(a % i == 0 && b % i == 0){ liste.add(i); } } int groesste = Integer.MIN_VALUE; for(int i = 0; i < liste.size(); i++){ if(liste.get(i) > groesste){ groesste = liste.get(i); } } return groesste; } }

Kommentare (7)

Beliebt
Neu
Alt

Du musst angemeldet sein, um einen Kommentar schreiben zu können

Thomas

Level 41 , Bayreuth, Germany

5 Juni 2020, 18:13

Weil du keine Ausnahme auslöst (throw). Du fängst nur welche (catch).

hidden #10625598

Level 23

5 Juni 2020, 20:07

wo throw ich denn da noch was? nach catch? mit if (a != Integer) irgendwie?

Thomas

Level 41 , Bayreuth, Germany

5 Juni 2020, 20:23

Wenn die eingegebenen Zeilen nicht in positive ganze Zahlen umgewandelt werden können, löse eine Ausnahme aus. Wenn die Zahlen also <=0 sind throw NumberFormat oder WasAuchImmerWasException (irgendwas, das es gibt und das halbwegs passt. Kannst natürlich auch deine eigene Exception schreiben), die wird dann gefangen durch dein catch statement. Du sollst sie aber werfen, deswegen hier noch mal ein rethrow statt Stacktrace drucken.

hidden #10625598

Level 23

5 Juni 2020, 20:41

und die darstellung? hier findet er das symbol nicht try { int a = scan.nextInt(); int b = scan.nextInt(); if(a <= 0){ throw NumberFormatException; } System.out.println(GGT(a, b)); } oder damit kommt auch ganz ein komischer error :D throw (NumberFormatException e){ System.out.println(e); }

Thomas

Level 41 , Bayreuth, Germany

5 Juni 2020, 20:54

Alles in Java ist ein Objekt ;) .... if (a <= 0 || b <= 0) throw new NumberFormatException("Eingegebene Zahl zu klein"); .... } catch(NumberFormatException e) { throw e; }

hidden #10625598

Level 23

6 Juni 2020, 04:01

danke, macht sinn :D noch eine frage, wenn ich die exception provoziere, dann bekomme ich die richtige catch Meldung mit einem Error/ "roter text" hinterher, woran liegt das? z.B: bei <0 java.lang.NumberFormatException: Zahl zu klein java.lang.NumberFormatException: Zahl zu klein <--- rot at de.codegym.task.ta.... etc. oder bei dezimal: java.util.InputMismatchException java.util.InputMismatchException<--- rot at java.util.Scanner.t.....etc. ich habe folgenden code: try { int a = scan.nextInt(); int b = scan.nextInt(); if(a <= 0 || b <= 0){ throw new NumberFormatException("Zahl zu klein"); } System.out.println(GGT(a, b)); } catch (Exception e){ System.out.println(e); throw e; } finally { scan.close(); }

Thomas

Level 41 , Bayreuth, Germany

6 Juni 2020, 06:49

Weiß... hier hast du die Exception selbst behandelt (System.out.println(e); im catch Block) Rot, das ist das throw (e). Nachdem nichts mehr da ist, das die Exception fängt, wird sie durchgereicht, bis das Programm zu Ende ist und dann eben mit den default Exceptionfarben angezeigt.