Erste Schritte Wert im Array suchen

[dan1996]

Guten Abend, ich möchte überprüfen, ob ein Wert sich im Array befindet und wenn nicht soll es eine IllegalArgumentException auslösen.
Vielen Dank schonmal für die Hilfe

z.b
int x = 2
ist x im int[] a = {12,7,5,9,13,24} ?

ich wollte es in diese Methode einbauen, in meine if anweisung

public static int next(int[] a, int x){

        if(a == null || a.length <= 0){
            throw new IllegalArgumentException("keine Werte im Array");
        }
        int result = 0;
        int lange = a.length;

        for (int i = 0; i < a.length; i++){

            if(...){
                throw new IllegalArgumentException("x befindet sich nicht im Array");
            }
         
            if(x == a[lange -1]){
                result = x;
            }
            if(a[i] == x && a[lange -1] != x){
                result = a[i+1];
            }
        }
        return result;
    }

 

[httpdigest]

Die Tatsache, ob `x` im Array `a` enthalten ist, ist ja nicht abhängig vom aktuellen Schleifenindex, wenn du durch `a` iterierst. Weshalb also willst du den Check innerhalb der Schleife machen und somit auch für jeden Wert im Array erneut machen?
Du solltest vielleicht einmal vorher prüfen, ob `x` in `a` ist. Mit einer weiteren Schleife könnte das so aussehen:

public static int next(int[] a, int x) {
  if (a == null || a.length <= 0) {
    throw new IllegalArgumentException("keine Werte im Array");
  }
  boolean enthalten = false;
  for (int i = 0; i < a.length && !enthalten; i++) {
    enthalten |= a[i] == x;
  }
  if (!enthalten) {
    throw new IllegalArgumentException("x befindet sich nicht im Array");
  }
  int result = 0;
  int lange = a.length;
  for (int i = 0; i < a.length; i++) {
    if (x == a[lange - 1]) {
      result = x;
    }
    if (a[i] == x && a[lange - 1] != x) {
      result = a[i + 1];
    }
  }
  return result;
}

Oder, du verschiebst einen Teil der Ermittlung innerhalb deiner bereits existierenden Schleife, während du `result` berechnest, und dann gibst du eine Exception am Ende aus:

public static int next(int[] a, int x) {
  if (a == null || a.length <= 0) {
    throw new IllegalArgumentException("keine Werte im Array");
  }
  int result = 0;
  boolean enthalten = false;
  int lange = a.length;
  for (int i = 0; i < a.length; i++) {
    enthalten |= a[i] == x;
    if (x == a[lange - 1]) {
      result = x;
    }
    if (a[i] == x && a[lange - 1] != x) {
      result = a[i + 1];
    }
  }
  if (!enthalten) {
    throw new IllegalArgumentException("x befindet sich nicht im Array");
  }
  return result;
}

 

[White_Fox]

Du solltest die Exception vielleicht auch noch an den Aufrufer weiterleiten, wenn du den Fehler nicht behandeln kannst/willst.

 

[httpdigest]

Exceptions, die nicht in der Methode selbst behandelt werden, werden immer an den Aufrufer weitergeleitet. IllegalArgumentException ist keine checked Exception, muss also nicht als `throws IllegalArgumentException` an der Methode deklariert werden - falls du das meintest.

 

[ocsme]

Guten Tag,

kann mir mal jemand erklären was das hier machen soll?

    if (x == a[lange - 1]) {
      result = x;
    }
    if (a[i] == x && a[lange - 1] != x) {
      result = a[i + 1];
    }

Möchtest du den Index zurück geben?

ob ein x im Array a vorkommt hätte ich ansonsten so gemacht

public static boolean kommtVor(int[] a, int x) {
       
        for(int i=0;i<a.length;i++)
            if(a[i]==x)
                return true;
        return false;
    }

 

[dan1996]

ich danke euch

@ocsme
die Aufgabe bestand:
Schreiben Sie eine Methode
static int next(int[] a, int x),
die solch ein Feld a und solch einen Wert x als Parameter erhält. Die Methode soll den kleinsten Wert von a liefern, der größer als x ist.
Falls x der größte Wert von a ist, soll x das Ergebnis sein

habe in einer Methode das Array sortiert und mit der Methode next() den kleinsten Wert von a liefern, der größer als x ist returnt

 

[httpdigest]

Dann ist doch keine Frage, ob `x` im Array enthalten ist. Du sollst ja nur einen Wert finden, der mindestens so groß wie `x` ist. Das kann auch bedeuten, dass x=1 und a=[5,6,7]. Dann käme halt 5 als Ergebnis.
Außerdem braucht man hierfür das Array nicht zu sortieren.
Hätte ich insgesamt so gelöst:

Spoiler

public static int next(int[] a, int x) {
  if (a == null || a.length == 0)
    throw new IllegalArgumentException("Keine Werte im Array");
  int result = a[0];
  for (int i = 1; i < a.length; i++)
    if (a[i] > x && a[i] < result)
      result = a[0];
  if (result < x)
    throw new IllegalArgumentException("Kein Wert im Array ist >= " + x);
  return result;
}

 

[dan1996]

Ich fande es einfacher nach dem sortierten Array, die nächste zahl von x zu nehmen
Das ist die komplette Aufgabe:
Schreiben Sie eine Methode
static int next(int[] a, int x),
die solch ein Feld a und solch einen Wert x als Parameter erhält. Die Methode soll den kleinsten Wert von a liefern, der größer als x ist.
Falls der Parameter a das Null-Objekt ist oder keine Werte enthält oder nicht alle Elemente verschieden sind oder x nicht enthält, soll beim Aufruf der Methode eine IllegalArgumentException ausgelöst werden. Falls x der größte Wert von a ist, soll x das Ergebnis sein. Sie dürfen weder Methoden noch Klassen importieren, aber Hilfsmethoden schreiben. Die Folge a soll durch einen Aufruf von next nicht geändert werden.
Beispiel: Falls a die Folge (12,7,5,9,13,24) und x der Wert 5 ist, soll der Aufruf next(a,x) den Wert 7 liefern, da 7 die nächstgrößere Zahl von 5 ist.

 

[httpdigest]

oder nicht alle Elemente verschieden sind

Okay, das ist der entscheidende Hinweis.

 

[ocsme]

Also wenn ich die Aufgabe nun richtig verstehe willst du einfach nur das nächst größere Minimum ermitteln nach einer Zahl x in einer Reihe a oder?

int[] a = new int[] {12,7,5,9,13,24};
        int x = 5;
      
        int min=Integer.MAX_VALUE;
      
        for(int i=0;i<a.length;i++) {
            if(x < a[i] && a[i] < min)
                min=a[i];
            System.out.println(min);
        }

somit steht in min die Zahl drin Reihenfolge spielt doch dann auch keine Rolle oder
Dann brauchst du nicht zu sortieren und kannst gleich returnen.
Musst leider durch das gesamte Array laufen !

gesamte Lösung nun so:

public static boolean kommtVor(int[] a, int x) {
       
        for(int i=0;i<a.length;i++)
            if(a[i]==x)
                return true;
        return false;
    }
   
    public static int next(int[] a, int x) {
          if (a == null || a.length <= 0) {
            throw new IllegalArgumentException("keine Werte im Array");
          }
         
          if(!kommtVor(a,x))
              throw new IllegalArgumentException("x nicht im Array");
        
          int min=Integer.MAX_VALUE;
          for(int i=0;i<a.length;i++)
                if(x < a[i] && a[i] < min)
                    min=a[i];
          return min;
        }

 

[ocsme]

Sorry ich hab vergessen das muss noch unter die for schleife.

  if(min == Integer.MAX_VALUE)
              return x;

Denn wenn der x Wert ja der größte ist soll nicht 2147483647 raus kommen sondern x oder?

 

[httpdigest]

@ocsme : oder nicht alle Elemente verschieden sind ..., soll beim Aufruf der Methode eine IllegalArgumentException ausgelöst werden.

 

[ocsme]

@ocsme : oder nicht alle Elemente verschieden sind ..., soll beim Aufruf der Methode eine IllegalArgumentException ausgelöst werden.

hupsi das habe ich nicht gelesen sry
dann vergesst das alles von mir
sry!