Iterator

[Mozza]

Hallo, ich bin recht neu in der Java Welt und habe ein kleines Problem mit einem Quelltext.

hier mal der Auszug der betreffenden Methode:

    public static Set<Integer> getRelevanteZiehungen(Set<Integer> getipteZahlen)
    {
      
        for(int i = 1; i<=ziehungsSpeicher.size();i++)
        {
            Set<Integer> gezogen = ziehungsSpeicher.get(i);
            int getroffeneAnzahl = 0;
            for(Iterator<Integer> getipteZahl = getipteZahlen.iterator(); getipteZahl.hasNext();) // <- this
            {
  
                if (gezogen.contains(getipteZahl))
                {
                    getroffeneAnzahl++;
                }
            }
           System.out.println("getroffene Anzahl" + getroffeneAnzahl +" bei ziehung " + i);
      }
        return getipteZahlen;
    }
}

Wenn ich das Programm compiliere, erhalte ich jedes mal eine Null.Pointer.Exception in der markierten Zeile, ich weiß allerdings nicht warum. wahrscheinlich benutze ich den Iterator verkehrt?!

Ich hoffe jemand kann mir dabei helfen

 

[Elenteria]

ich geh mal davon aus das getippeZahlen niergens initialisiert wurde.

Außerdem hast du noch weitere Fehler drin. Zum einen hast du dir eine Endlosschleife gebaut und zum anderen wird getroffeneAnzahl niemals erhöht werden da du hier prüfst ob gezogen getippteZahl enthält was niemals der Fall sein wird.

 

[Joose]

Wenn ich das Programm compiliere, erhalte ich jedes mal eine Null.Pointer.Exception in der markierten Zeile, ich weiß allerdings nicht warum.

Beim kompilieren kannst du keine Exception bekommen sondern einen Compile Error.
Eine Exception kann erst zur Laufzeit passieren wenn dein Programm ausgeführt wird.

Eine NullPointerException fliegt immer dann wenn wo vergessen wurde einer Variable ein Objekt zuzuweisen und man damit arbeiten will.
Elenteria hat dir schon gesagt wo du ansetzen solltest

 

[Mozza]

Ehrlich gesagt helfen mir eure Antworten nicht wirklich weiter

Wenn ich die Variable Iterator<Integer> getipteZahl = null; z.B. so initialisiere dann sagt er mir das die lokale Variable doppelt vorhanden ist. wie kann ich sie andersweitig initialisieren?

 

[Harry Kane]

Wenn ich die Variable Iterator<Integer> getipteZahl = null; z.B. so initialisiere dann sagt er mir das die lokale Variable doppelt vorhanden ist. wie kann ich sie andersweitig initialisieren?

Wie rufst du denn die getRelevanteZiehungen Methode auf? Check mal, ob du nicht null als Argument übergibst.

 

[Mozza]

getRelevanteZiehungen wird in meinem BenutzerMenue aufgerufen.
Die entsprechende Zeile schaut so aus

ZiehungsSpeicher.getRelevanteZiehungen(getipteZahlen);

 

[Harry Kane]

Und wo kommt der Parameter "getipteZahlen" her? Er fällt ja nicht vom Himmel.

 

[fhoffmann]

Ich wundere mich, dass der Code compilierbar sein soll. getipteZahl ist ein Iterator und kein Integer. Deshalb sollte der Aufruf gezogen.contains(getipteZahl) zu einem Compilerfehler führen.

 

[Harry Kane]

@fhoffmann: nein, da liegst du falsch. Collection<E>.contains() erwartet tatsächlich ein Object und kein E als Parameters.

 

[Mozza]

getipteZahlen bekommt seine Werte hier rüber:

getipteZahlen = new Tipp().getInput();

in newTipp().getInput() kann der Benutzer über einen Scanner seinen Tipp abgeben, dieser wird dann als return zurück gegeben

 

[InfectedBytes]

und dann ist noch die frage, warum du den iterator überhaupt manuell verwendest.
für genau sowas gibt es ja extra die for-each schleife

for(Integer i : getippteZahlen) {

}

 

[Mozza]

Mit der habe ich es zu erst versucht, kenne mich damit aber leider überhaupt nicht aus und dachte die Exception kommt daher

 

[Harry Kane]

getipteZahlen = new Tipp().getInput();
in newTipp().getInput() kann der Benutzer über einen Scanner seinen Tipp abgeben, dieser wird dann als return zurück gegeben

Das heisst, du hast irgendeine "Masterklasse", die new Tipp().getInput() aufruft und den Rückgabewert der Methode als Argument für ZiehungsSpeicher.getRelevanteZiehungen(). Und wie ist der Quellcode von getInput() aus der Tipp-Klasse? Überprüfst du an irgendeiner Stelle, ob getipteZahlen ungleich null ist?

 

[Elenteria]

Na dann würde ich sagen schau mal ob new Tip().getInput() überhaupt das richtige zurück liefert. Wenn dem so sein sollte dann würde ich mal schauen ob die Aufrufreihenfolge korrekt ist. Nicht das du das alles so gebaut hast das getRelevanteZahlen vor der getInput() methode aufgerufen wird.

 

[Mozza]

Genau. in getInput() werden mehrere eingaben des Benutzer in ein HashSet gegeben. somit ist ja ausgeschlossen das der Wert null ist.

 

[InfectedBytes]

zeig doch einfach die getinput methode, dann kann man sehen ob da vielleicht doch ein fehler drin ist

 

[JStein52]

Hast du mal im Debugger geschaut was dein Parameter getipteZahlen denn nun wirklich enthält ? Die For-each-Schleife von @InfectedBytes ist auf jeden Fall am lesbarsten und richtig. Dein Fehler kommt woanders her.

 

[Mozza]

Die getInput() sieht so aus

Set<Integer> input = new HashSet<>();
 
    public Set<Integer> getInput()
    {
        Scanner sc = new Scanner(System.in);
        Set<Integer> input = new HashSet<>();
       
          for(int i = 0; input.size() < 6; i++)
            {
              System.out.print("Eingabe der " + (i + 1) + ". Zahl: ");
              Integer obj = new Integer(sc.nextInt());
              if(obj <= 49)
                  {
                      input.add(obj);
                  }
              else
                  {
                      i--;
                  }
            }
          System.out.println("Ihr Tipp lautet \n" + input);
           return input;
    }

und zum Debugger... weiß leider nicht genau wie ich damit umgehen muss

 

[Harry Kane]

Allmählich wirds anstrengend:
In dieser Zeile

for(Iterator<Integer> getipteZahl = getipteZahlen.iterator(); getipteZahl.hasNext();)

fliegt eine NPE. Entweder getipteZahlen ist null, oder gibt einen null-Iterator zurück. Letzteres kann ich mir nicht vorstellen. Ergo vermute ich sehr stark, das getipteZahlen null ist. Hast du das mal getestet?

 

[InfectedBytes]

du hast eine objektvariable namens input und eine lokale variable namens input.
ich würde mal vermuten du übergibst die Objektvariable, welche null ist.

 

[Harry Kane]

du hast eine objektvariable namens input und eine lokale variable namens input.
ich würde mal vermuten du übergibst die Objektvariable, welche null ist.

Sowohl die Instanz- als auch die lokale Variable sind beide initialisiert. Und die getInput() Methode müsste die lokale Variable zurückgeben, es sei denn, der TE hat vor dem Posten ein paar this aus dem Code gelöscht.

 

[JStein52]

Aber wo siehst du nun dass er diese auch wieder übergibt ?

 

[Harry Kane]

Aber wo siehst du nun dass er diese auch wieder übergibt ?

Aus seinem Code? In seinem zuletzt geposteten Schnipsel hat er eine lokale input-Variable und eine ausserhalb der Methode stehende, die wohl eine Instanzvariable sein soll. In der Methode steht nirgendwo this, so dass die lokale Variable zurückgegeben wird. Dann wäre die Instanzvariable natürlich überflüssig.
Ob tatsächlich der Rückgabewert dieser Methode auch tatsächlich an ZiehungsSpeicher.getRelevanteZiehungen() übergeben wird, steht auf einem anderen Blatt. Das liesse sich erst klären, wenn der TE mal zusammenhängenden Code postet statt Puzzleteile.

 

[InfectedBytes]

es ging auch nicht um den inhalt dieser methode, sondern um den Rückgabewert.
Hab mich zugegebenermaßen etwas falsch ausgedrückt^^
Meine Vermutung ist halt dass er eben nicht den Rückgabewert benutzt, sondern die Objekt Variable

 

[Harry Kane]

es ging auch nicht um den inhalt dieser methode, sondern um den Rückgabewert.

Den Rückgabewert wovon? Von der getInput-Methode? Dann verstehe ich nicht, was du meinst.
So wie die getInput-Methode gepostet wurde, wird im Methodenrumpf eine Variable namens input deklariert und initialisiert und schliesslich mit return zurückgegeben.
Welche Variable er tatsächlich an die getRelevanteZiehungen Methode übergibt, ist allerdings unklar.

 

[InfectedBytes]

die getInput methode scheint soweit korrekt zu sein.
Allerdings ist halt die Frage ob er den Rückgabewert dieser Methode überhaupt benutzt oder ob er die Objektvariable verwendet, weil er glaubt dass beides das gleiche ist.