Bug finden im Programm

[Nihila]

Hallo liebe Java-Gemeinde,

Bin ein Java-Neuling und soll nun in einem Programm einen Bug finden. Leider finde ich den partout nicht :rtfm:. Wäre echt lieb von euch, wenn ihr mir dabei helfen könntet.

private static String fragezeichen = "?";
  
  private String frage;
  
  public static void main(String[] args) {
    
    String[] s = {"Wann", "Wo", "Warum", "Wann", "Was"};
    FragenUeberFragen[] fragen = new FragenUeberFragen[s.length];
    for (int i = 0; i < fragen.length; i++) {
      fragen[i] = new FragenUeberFragen();
      fragen[i].setFrage(s[i]);
      fragen[i].ergaenzeFragezeichen();
      boolean ausgeben = true;
      for (int j = i - 1; j >= 0 && ausgeben; j--) {
        if (fragen[i] == fragen[j]) {
          System.out.println("Das wurde bereits gefragt.");
          ausgeben = false;
        }
      }
      if (ausgeben) {
        gibFrageAus();
      }
    }
  }
  
  public void setFrage(String frage) {
    this.frage = frage;
  }
  
  public void ergaenzeFragezeichen() {
    this.frage = this.frage + fragezeichen;
  }
  
  public static void gibFrageAus() {
    System.out.println(this.frage);
  }
  
}

Vielleicht könntet ihr mir auch die Frage beantworten, warum kein Konstruktor implementiert wurde und es trotzdem funktioniert ???:L.

Danke euch schon mal im voraus.

Grüße.
Nihila

 

[rme]

Hallo

Leider weiß ich nicht genau, was das Programm machen soll - also wann das Verhalten als korrekt und wann als fehlerhaft gilt. Aber ich vermute Fehler in den Zeilen 15, 23 und 34 (bezogen auf obigen Ausschnitt). Tipp: Unterschied zwischen static-Methoden und nicht-static-Methoden nochmal nachschlagen oder ggf. hier nachfragen

Wenn man in Java für eine Klasse keinen Konstruktor anlegt, wird automatisch ein Standardkonstruktor angelegt, der keine Parameter bekommt und nichts macht. Deshalb ist dieser Teil des Codes in Ordnung.

 

[ARadauer]

Strings vergleicht man mit equals

if (fragen == fragen[j]) { >>>>
if (fragen.equals(fragen[j])) {

 

[rme]

Strings vergleicht man mit equals

if (fragen == fragen[j]) { >>>>
if (fragen.equals(fragen[j])) {

fragen ist kein String, sondern eine Instanz von FragenUeberFragen.

 

[Nihila]

Vielen Dank für die Antworten :toll:. Werde mir das mit static-Methoden und nicht-static-Methoden nochmal durchlesen. Das Programm für jedes im Array gegebene Fragewort ein neues Objekt erzeugen und dem Objekt dann eins der Frageworte zugewiesen werden mit anschließendem Fragezeichen. Auch soll jede Frage nur einmal gestellt werden

Sind an einigen Stellen nicht auch paar this-Angaben obsolet oder kann man das anders machen ?

 

[rme]

Das ist eher eine Geschmacksfrage. Ein Bug ist es jedenfalls nicht, sie hinzuschreiben, obwohl es nicht nötig wäre. Wenn jede Frage nur einmal gestellt werden soll, egal was das Array enthält, ist Zeile 15 das größte Problem. Und bei der Korrektur auch an das denken, was ARadauer angemerkt hat.

 

[Nihila]

Der Compiler spuckt ständig ein Fehler bei Zeile 35 bei "this.frage" aus. Wie Du schon sagtest, hat das was mit static und non-static. Er mosert also, dass eine non-static Variable nicht für einen Static Kontext gelten kann. Ich bin so ein Noob ;(. An welcher Stelle muss ich denn was drehen, damit der Fehler behoben ist. Muss ist da das static umwandeln bzw. umdeklarieren?

 

[rme]

Da die Aufgabe vermutlich darauf abzielt, dein Wissen darüber zu festigen, möchte ich dir nicht einfach die Lösung sagen. Eine static-Methode ist an keine Instanz gebunden und kann deshalb nicht auf Instanzvariablen zugreifen ("frage" ist bei dir eine Instanzvariable). Die Methode darf also nicht static sein, wenn sie die Frage ausgeben soll. Wenn sie nicht mehr static ist, bekommst du in einer anderen Zeile einen Fehler, der dort in Folge behoben werden muss

 

[Nihila]

Je mehr ich darüber lese, umso blöder komm ich mir vor . Bin total frustriert. Ich hab jetzt an zig Stelle rumgedreht, aber nichts ändert sich :autsch:. Ich bekomm noch nicht mal den Compiler-Fehler weg..und dann soll da noch ein ein schwer zu findender Bug drin sein, der nicht vom Compiler erkannt wird und auch keine Runtime-Exception macht :cry:. Ich seh den Wald vor Bäumen nicht.....

 

[rme]

Ganz ruhig

Was ist denn dein mentales Modell davon, was folgende Begriffe bedeuten? - Klasse, Objekt (Instanz), static

Wenn du mir dein momentenanes Verständnis dieser Begriffe beschreibst, kann ich sehen, wo dein Verständnisproblem bzw. eine Wissenslücke liegt und es dir erklären, bis du die Aufgabe lösen kannst

 

[Nihila]

Ja Klassen bilden doch den Rahmen, wo Datentypen deklariert werden und beinhalten doch Attribute und die Methoden, oder nicht? Objekt ist dann die Instanz einer Klasse und verfügt auch über alle Attribute und Methoden der Klasse. Merkmale der Klassendefinition, die nicht von deren Objekten geteilt werden, sind dann static und es kann nur eine davon geben. So weit zur Theorie :rtfm:. Du siehst nur noch halb aussiehtlos .

 

[rme]

Das sieht doch schon ganz gut aus Klassen sind selbst übrigens auch Datentypen.

Wenn man es genauer betrachtet, muss man bei den Attributen zwischen Instanzvariablen und Klassenvariablen unterscheiden. Aber die Theorie hast du glaube ich eh korrekt verstanden: Bei den Instanzvariablen hat jede Instanz ihren eigenen Speicher für den Inhalt der Variable, bei Klassenvariablen (static) teilen sich alle Instanzen diesen Speicher, sodass es nur einen Speicherbereich dafür gibt. Überschreibt eine Instanz diesen Wert, so hat er diesen Wert danach auch in allen Instanzen.

Wichtig ist nun, dass es auch statische Methoden gibt - das sind Methoden, die man aufrufen kann, ohne eine Instanz zu haben. D.h. wenn eine Klasse eine Methode m definiert, kann man Klasse.m aufrufen statt Instanz.m. Der Preis ist dann natürlich, dass diese Methode dann nur auf Klassenvariablen zugreifen kann und nicht auf Instanzvariablen - denn da der Aufrufer keine Instanz mitgegeben hat, sondern nur die Klasse, gibt es innerhalb der Methode kein "this".

Deine Methode verwendet aber "this" innerhalb einer static-Methode. Das geht aus obigen Gründen nicht. Da die Methode aber ziemlich sicher auf die Frage zugreifen soll und Frage eine Instanzvariable ist - und auch bleiben muss, sonst könnte es nur eine Frage geben - ist die Lösung, dass du das "static" bei der Methode entfernst.

Weiter oben im Code wird diese Methode in der Main-Methode aufgerufen. Aber da Main eine statische Methode ist, gibt es auch in main kein this - es können also nur static-Methoden aufgerufen werden! Deshalb lauert da der nächste Fehler, sobald der erste berichtigt ist. Zur Lösung musst du an der Stelle angeben, auf welche Instanz sich der Aufruf der ehemaligen static-Methode beziehen soll. Eine Schleife, die alle vorhandenen Instanzen besucht, ist ja schon vorhanden...

Hilft das ein bisschen?

 

[Nihila]

Vielen Dank für Deine ausführliche Antwort und Hilfe. Theoretisch weiß ich, was was ist, aber ich kann es irgendwie nicht umsetzen . Ich lese und lese und nichts geht ins Gehirn rein . Wärest Du so lieb und könntest mir vllt. ein praktisches Beispiel oder doch die Lösung sagen (notfalls per Mail) ? Normal ist ja besser von selber darauf zu kommen....aber ich hab das Programm schon auseinandergeflückelt und es tut sich gar nichts...ausnahmsweise bevor ich noch nen Anfall bekomme .

 

[knilch]

Hi,
Poste doch bitte mal den Code der Klasse FragenUeberFragen.java bzw. alle Klassen, die zum Ausführen notwendig sind.