'void' type not allowed

[M.I.G]

Hallo ich versuche gerade lineare Suche zu programmieren. Folgende habe ich bereits:

import java.io.*;

public class cRecord implements Serializable
{
  String name;
  String vorname;
  int nummer;

  public cRecord(int pNummer, String pName, String pVorname)
  {
    name = pName;
    vorname = pVorname;
    nummer = pNummer;
  }

  public int getNummer()
  {
    return nummer;
  }

  public void setNummer(int pNummer)
  {
    this.nummer = pNummer;
  }

  public String getName()
  {
    return name;
  }
  
  public void setName(String pName)
  {
    this.name = pName;
  }
  
  public String getVorname()
  {
    return vorname;
  }
  
  public void setVorname(String pVorname)
  {
    this.vorname  = pVorname;
  }
  
  public void zeige()
  {
     System.out.println("Name: "+ name);
     System.out.println("Vorname: "+ vorname );
     System.out.println("Nummer: "+ nummer);
  }
}

import java.io.*;

public class cRecordArray implements  Serializable
{
  cRecord[] daten;

  public cRecordArray(int pLänge)
  {
    daten = new cRecord[pLänge];
  }

  public void zeige_cRecord()
  {
    for(int lauf = 0; lauf < daten.length; lauf++)
    {
      System.out.println("Index "+ lauf +":");
      daten[lauf].zeige();
      System.out.println();
    }
  }
  
  public void fülle()
  {
    System.out.println();
    
    for (int lauf = 0;lauf < daten.length;lauf++)
    {
      daten[lauf] = new cRecord(0 ,"Name", "Vorname");
      daten[lauf].setNummer(lauf);
    }
    for(int buchstabe1=0; buchstabe1<26; buchstabe1++)
    {
      for(int buchstabe2=0; buchstabe2<26; buchstabe2++)
      {
        daten[buchstabe1*26+buchstabe2].setName( (char)(buchstabe1+65)+""+(char)(buchstabe2+65) );
      }
    }
    for(int buchstabe1=0; buchstabe1<26; buchstabe1++)
    {
      for(int buchstabe2=0; buchstabe2<26; buchstabe2++)
      {
        daten[buchstabe1*26+buchstabe2].setVorname( (char)(buchstabe1+65)+""+(char)(buchstabe2+65) );
      }
    }
  }
  
  public int länge()
  {
    return daten.length;
  }
}

import java.io.*;

public class cLineareSuche implements  Serializable
{
  cRecord[] daten;
  
  public void suchen(String targetName)
  {
    for(int lauf = 0; lauf<daten.length;lauf++)
    {
      if(this.daten[lauf].getName().equals(targetName))
      System.out.println(this.daten[lauf].zeige());
      else
      {
        if(lauf == daten.length-1)
        System.out.println("Der gesuchte Name existiert nicht.");
      }
    }
  }
}

Aber ich bekomme immer folgende Fehlermeldung:

cLineareSuche.java:13:48: 'void' type not allowed here
      System.out.println(this.daten[lauf].zeige());
                                                               ^
1 error

Ich hoffe ihr könnt mir helfen danke im voraus.

 

[Marco13]

Aber ich bekomme immer folgende Fehlermeldung:

cLineareSuche.java:13:48: 'void' type not allowed here
      System.out.println(this.daten[lauf].zeige());
                                                               ^
1 error

this.daten[lauf].zeige() liefert "void" zurück. D.h. die Methode liefert NICHT etwa einen String, den man mit System.out. ausgeben könnte, sondern gibt SELBST etwas aus.
System.out.println(this.daten[lauf].zeige());
ersetzen durch
this.daten[lauf].zeige();

Oder in den Daten toString implementieren und
System.out.println(this.daten[lauf]);
aufrufen

 

[trazzag]

Versuch mal die letzte Klasse wie folgt zu gestalten:

import java.io.*;

public class cLineareSuche implements  Serializable
{
  cRecord[] daten;
 
  public void suchen(String targetName)
  {
    for(int lauf = 0; lauf<daten.length;lauf++)
    {
      if(this.daten[lauf].getName().equals(targetName))
        this.daten[lauf].zeige();
      else
      {
        if(lauf == daten.length-1)
        System.out.println("Der gesuchte Name existiert nicht.");
      }
    }
  }
}

Also, einfahc das System.out weglassen - die rufst da ja in der Methode auf...

 

[M.I.G]

Ach ja danke.

 

[M.I.G]

Jetzt hab ich aber ein neues Problem wenn ich in folgendes in die main schreibe:

public static void main(String[] args)
  {
    cRecordArray adressbuch;
    adressbuch = new cRecordArray(26*26);

    adressbuch.fülle();
    adressbuch.zeige_cRecord();
    adressbuch.suchen("HH");
  }

kriege ich folgende Fehlermeldung:

cLineareSuche.java:28:15: cannot find symbol
symbol  : method suchen(java.lang.String)
location: class cRecordArray
    adressbuch.suchen("HH");
              ^
1 error

:bahnhof:

 

[Tobias]

Jo, adressBuch ist vom Typ cRecordArray und cRecordArray hat nun mal keine Methode suchen. Die ist in cLineareSuche. BTW: Du solltest dir bei Gelegenheit mal die Sun Coding Conventions reinziehen.

mpG
Tobias

 

[M.I.G]

Ok ich hab das jetzt vererbt

import java.io.*;

public class cLineareSuche extends cRecordArray
{
  cRecord[] daten;
  
  public cLineareSuche(int lang)
  {
    super(lang);
  }
  
  public void suchen(String targetName)
  {
    for(int lauf = 0; lauf<daten.length;lauf++)
    {
      if(this.daten[lauf].getName().equals(targetName))
      this.daten[lauf].zeige();
      else
      {
        if(lauf == daten.length-1)
        System.out.println("Der gesuchte Name existiert nicht.");
      }
    }
  }
  public static void main(String[] args)
  {
    cLineareSuche adressbuch;
    adressbuch = new cLineareSuche(26*26);

    adressbuch.fülle();
    adressbuch.zeige_cRecord();
    adressbuch.suchen("HH");
  }
}

, aber jetzt klappt es immer noch nicht reibungslos der Array wird angezeigt aber suchen klappt nicht denn ich krieg immer folgendes ausgegeben:

.......

Index 673:
Name: ZX
Vorname: ZX
Nummer: 673

Index 674:
Name: ZY
Vorname: ZY
Nummer: 674

Index 675:
Name: ZZ
Vorname: ZZ
Nummer: 675

Exception in thread "main" java.lang.NullPointerException
        at cLineareSuche.suchen(cLineareSuche.java:15)
        at cLineareSuche.main(cLineareSuche.java:33)

Hoffe ihr könnt mir helfen.

@Tobias:
Meinst du weil ich die Klassennamen klein schreibe und die Klammern nich so setze wie in den Sun Coding Conventions, oder ist daran noch mehr auszusetzen, weil die 2 genannten Punkte verlangt unser Lehrer.

 

[Tobias]

Mh, das habe ich befürchtet - also das mit dem Lehrer. Nein, dein Code sieht recht vernünftig aus, bis eben auf die Klammern und das c vor jedem Klassennamen.

Zur Vererbung: cLineareSuche benutzt doch nur ein cRecordArray, ist aber selbst keines. Vererbung bedeutet man installiert eine "ist-ein"-Beziehung, während bei dir eine "hat-ein"-Beziehung angebrachter wäre ("Hat-ein"-Beziehungen lassen sich zum Beispiel über Instanzvariablen etablieren).
Oder du verzichtest komplett auf die Klasse cLineareSuche und verschiebst stattdessen die Funktionalität in cRecordArray (die Lösung würde ich bevorzugen, den Klassen, die nur eine Tätigkeit beschreiben, sind meistens creepy).

Zu deinem eigentlichen Problem: Welche ist den Zeile 15? Markier das mal bitte im Code. Am einfachsten wirst du den Fehler mit einem Debugger im Einzelstepmodus finden. Meine Vermutung ist, das dein Array weniger Einträge enthält als Plätze vorhanden sind, so das die Suche irgendwann auf einen nicht-initialisierten Array-Index stößt, von dem du ohne Prüfung die zeige()-Methode aufrufst. Das knallt dann eben.

mpG
Tobias

 

[M.I.G]

Das mit dem cLineareSuche velangt leider auch mein Lehrer
Ich weiß leider nich was du mit Instanzvariablen etablieren meinst und was ein Debugger ist :bahnhof:
Und Zeile 15 ist gleich Zeile 15 in der Codierung.

 

[Tobias]

Ok, mach folgendes:

import java.io.*;

public class cLineareSuche implements  Serializable
{
  cRecordArray daten; // cLineareSuche _hat ein_ cRecordArray
 
  public void suchen(String targetName)
  {
    for(int lauf = 0; lauf<daten.länge();lauf++)
    {
       // daten ist jetzt kein Array mehr, also müssen wir Methoden zum Zugriff benutzen
      if(this.daten.hole(lauf).getName().equals(targetName)) {
          System.out.println(this.daten.hole(lauf).zeige());
      }
      else
      {
        if(lauf == daten.länge()-1)
        System.out.println("Der gesuchte Name existiert nicht.");
      }
    }
  }
}

In deinem cRecordArray mußt du jetzt noch eine Methode hole(int lauf) einfügen, die dir das cRecord mit der Nummer lauf zurückgibt. Diese Implementierung ist dir zu Übungszwecken überlassen (1 Zeile; mit Kopf und Klammern in eigenen Zeilen 4 Zeilen).

mpG
Tobias

P.S.: Das Problem bei deinem Code ist, das dein Feld daten in cLineareSuche das Feld daten in cRecordArray überdeckt. Die Zugriffe aus cLineareSuche auf dieses Feld bezogen sich also alle auf das leere, uninitialisierte Feld daten in cLineareSuche und nicht, wie du erwartet hat, aus daten in cRecordArray. Verwirrt hast du mich allerdings mit der Ausgabe - die kam nämlich nicht von cLineareSuche sondern von irgendwo anders her ...

P.P.S.: Alle meine Codes sind ungetestet, funktionieren aber zumindest prinzipiell.

 

[M.I.G]

Leider klappt suchen() immer noch nicht.

cLineareSuche.java:30:15: cannot find symbol
symbol  : method suchen(java.lang.String)
location: class cRecordArray
    adressbuch.suchen("HH");
              ^
1 error

Und was meinst du eigentlich mit dem was in deiner Klammer steht?
(1 Zeile; mit Kopf und Klammern in eigenen Zeilen 4 Zeilen)

 

[Tobias]

Na, die Methode suchen() gehört ja auch immer noch zu cLineareSuche und NICHT zu cRecordArray. Das wolltest du doch so?

Und meine Klammer gibt dir nur die ungefähre Größe der Methode hole(int lauf) an, die du noch schreiben mußt - 1 Zeile nach traditioneller Zählung ohne Methodenkopf etc, 4 Zeilen wenn man deine Coding Conventions und den Methodenkopf berücksichtigt.

Was in meinem obigen Code noch fehlt ist das Initialisieren der Beziehung zwischen cRecordArray und cLineareSuche. Hab ich a) vergessen und b) will ich dir ja nicht den ganzen Spaß rauben .

import java.io.*;

public class cLineareSuche implements  Serializable
{
  cRecordArray daten; // cLineareSuche _hat ein_ cRecordArray

  public cLineareSuche(cRecordArray daten) {
    this.daten = daten; // Beziehung herstellen
  }

  public void suchen(String targetName)
  {
    for(int lauf = 0; lauf<daten.länge();lauf++)
    {
       // daten ist jetzt kein Array mehr, also müssen wir Methoden zum Zugriff benutzen
      if(this.daten.hole(lauf).getName().equals(targetName)) {
          System.out.println(this.daten.hole(lauf).zeige());
      }
      else
      {
        if(lauf == daten.länge()-1)
        System.out.println("Der gesuchte Name existiert nicht.");
      }
    }
  }
}

In der Main dann

public static void main(String[] args) {
    cRecordArray adressbuch;
    adressbuch = new cLineareSuche(26*26);

    adressbuch.fülle();
    adressbuch.zeige_cRecord();
    
    cLineareSuche s = new cLineareSuche(adressBuch);
    s.suche("HH");

mpG
Tobias

 

[M.I.G]

Dann sind da aber immer noch Fehler

cLineareSuche.java:28:18: cannot find symbol
symbol  : constructor cLineareSuche(int)
location: class cLineareSuche
    adressbuch = new cLineareSuche(26*26);
                 ^
cLineareSuche.java:34:6: cannot find symbol
symbol  : method suche(java.lang.String)
location: class cLineareSuche
    s.suche("HH");
     ^
2 errors

 

[Tobias]

Ja, das eine muss logischerweise new cRecordArray(26*26); heißen (copy-paste-Fehler) und das andere ist ein Tippfehler, muss halt "suchen" statt "suche" heißen. Das ich dir helfe, heißt übrigens nicht, das du dein (bislang bemerkenswertes) Maß an Eigeninitiative jetzt einstellen darfst (Kein Vorwurf, nur eine Anmerkung, weil dir die beiden Flüchtigkeitsfehler auch selbst hätten auffallen können.).

mpG
Tobias :wink:

 

[M.I.G]

Jo sorry liegt wohl daran ,dass ich da schon länger dran sitze und ein bisschen unter Zeitdruck stehe.
Aber tausend dank

 

[Tobias]

Läuft's denn jetzt?

mpG
Tobias

 

[M.I.G]

Ja aber ich hab noch ein Bug. Weil immer: "der name existiert nicht "angezeigt wird, außer ich suche nach dem letzten Namen.Gibt es ein Befehl mit dem ich die for-Schleife abbreachen kann, weil dann würds funktionieren.

 

[Tobias]

Es gibt die Anweisung break mit der eine Schleife verlassen werden kann.

mpG
Tobias

 

[M.I.G]

Ok thx jetzt klappt alles reibungslos