Aufgabe: Array auf doppelte Zahl prüfen

[MisterX]

Guten Tach zusammen,

folgende Aufgabe muss ich bewältigen:
5 Zahlen eingeben, dann das Array auf doppelte Zahlen prüfen und bei Fund das Programm beenden.

Ich habe es jetzt so gemacht:

2 Schleifen und dann die Kontrolle:

 if(dipple[i]==dipple[z])
           {
             System.out.println("Doppelte Zahl gefunden");
             System.exit(1);
           }

Ist das so sauber mit dem System.exit?
Oder ist das mit dem Holzhammer draufgeschlagen?

Viele Grüße

 

[SlaterB]

Holzhammer, aber wenn das Programm danach zu Ende sein soll,
dann in einem einfachen Programm auch nicht ganz schlimm

 

[MisterX]

Danke.
Aber sonst weiß ich nicht wie ich aus den beiden Schleifen rauskomme.
Mit "break" verlasse ich nur die zweite. Die erste kehrt dann ja wieder in die erste zurück...

 

[SlaterB]

return;
gibts auch noch

für doppelte Schleife könntest du einen boolean auf false setzen und den am Ende der äußeren Schleife prüfen -> noch ein break

 

[Landei]

Man kann mit Labels mehrere Schleifenlevels "abbrechen":

outer:
for(int i = 1; i < 100; i++) {
   for(int j = 1; j < 100; j++) {
      if (i*j > 400) {
         break outer;  
      }
   }
}

 

[maki]

Sauberer wäre es jede schleife in eine eigene Methode auszulagern und dann mit return zu arbeiten.

 

[Orri]

Mir wurde beigebracht die boolean zu benutzen um aus schleifen rauszukommen

boolean abbruch=true;
while(abbruch)
{
...
if(dipple[i]==dipple[z])
{
    System.out.println("Doppelte Zahl gefunden");
    abbruch=false;
}//endif
}//endwhile

 

[Civilazi]

Mir wurde beigebracht die boolean zu benutzen um aus schleifen rauszukommen

boolean abbruch=true;
while(abbruch)
{
...
if(dipple[i]==dipple[z])
{
    System.out.println("Doppelte Zahl gefunden");
    abbruch=false;
}//endif
}//endwhile

Geht auch wunderbar, aber da kriegt man ja Knoten im Kopf, wenn du
abbrichst, genau dann, wenn !abbruch. Also dann vielleicht den Namen geeigneter wählen, vor allem, wenn auch andere deinen Code mal überfliegen sollen

 

[SchonWiederFred]

2 Schleifen

Damit hast Du quadratische Komplexität. Schneller geht's mit einem HashSet.
Außerdem würde ich einfach eine Methode basteln, die einen boolean zurückgibt, anstatt das Programm zu beenden.

public static boolean einzigartig(int[] array)
{
    Set<Integer> untersucht = new HashSet<Integer>(2 * array.length);
    for (int zahl: array)
        if (!untersucht.add(zahl))
            return false;
    return true;
}