Probleme mit dem Konstruktor

[janhei]

Hallo,

ich habe folgendes Programm geschreiben:

import Prog1Tools.IOTools;

class Datensatz {
	
  protected double Betrag;
  protected int Tag;
  protected int Monat;
  protected int Haeufigkeit;
  protected String Beschreibung;
  protected String Glaeubiger;
  
  public Datensatz()
  {
    IOTools.readLine("");
    Beschreibung=IOTools.readLine("Beschreibung: ");
    Glaeubiger=IOTools.readLine("Glaeubiger: ");
    Tag=IOTools.readInteger("Tag: ");
  	Monat=IOTools.readInteger("Monat: ");
  	Haeufigkeit=IOTools.readInteger("Haeufigkeit: ");
  	Betrag=IOTools.readDouble("Betrag: ");
  	IOTools.readLine("");
  	System.out.println();
  }
  
  public void zeigeDaten()
  {  	
  	System.out.println("Glaubiger:    \t"+Glaeubiger);
  	System.out.println();
  	System.out.println("Beschreibung: \t"+Beschreibung);
  	System.out.println("Datum:        \t"+Tag+"."+Monat);
  	System.out.println("Haeufigkeit:  \t"+Haeufigkeit);
  	System.out.println();  	
  	System.out.println("Betrag:       \t"+Betrag+" EUR");  	
  }
  
}

public class Ausgaben { 

  protected Datensatz [] Verpfl;
  
  public Ausgaben (int n)
  {
  	Verpfl = new Datensatz[n];
  }

  public void zeigeAusgaben()
  {
  	double summe = 0;
  	System.out.println("----------------------------------------------");
  	for(int i=0; i<Verpfl.length; i++)
  	{
  	  Verpfl[i].zeigeDaten();  // Zeile aus der Fehler-Meldung
  	  summe = summe + Verpfl[i].Betrag;
  	  System.out.println("----------------------------------------------");
  	}
  	System.out.println("Summe der Ausgaben: "+summe+" EUR");
  	System.out.println("----------------------------------------------"); 
  	System.out.println(); 	
  }
  
  public void sortiereAusgabe()
  {
  	Datensatz temp=new Datensatz();
  	for (int i=(Verpfl.length-2); i==0; i--)
      for (int j=0; j<=i; j++)
        if (istBesser(Verpfl[j], Verpfl[j+1])) { 
          temp=Verpfl[j]; 
          Verpfl[j] = Verpfl[j+1]; 
          Verpfl[j+1] = temp; 
        } 
  }
  
  public boolean istBesser(Datensatz D1, Datensatz D2)
  {
  	if (D1.Betrag>=D2.Betrag) {return true;} else {return false;}  	
  }

  public static void main(String[ ] args) 
  {
  	int n=0;
  	
  	System.out.println();
    System.out.println(" A U S G A B E N");
    System.out.println();     
    n=IOTools.readInteger("Anzahl der Datensaetze: ");
    Ausgaben a = new Ausgaben(n);    
    IOTools.readLine("");
    System.out.println();
    System.out.println("Vor dem Sortieren:");
    a.zeigeAusgaben();  // Zeile aus der Fehler-Meldung
    System.out.println("Nach dem Sortieren:");
    a.sortiereAusgabe();
    a.zeigeAusgaben();    
  }
  

}

(Die verwendete Klasse IOTools ist selbst geschrieben und funktioniert auch)

Folgenes Problem besteht. Ich kann das Prog. zwar ohne Probleme komblieren, aber beim ausführen bekomme ich folgende Fehlermeldung:
Exception in thread "main" java.lang.NullPointerException at Ausgaben.zeigeAusgaben(Ausgaben.java:54)
at Ausgaben.main(Ausgaben.java:92)

(Das mit den Zeilen 54 und 92 sollte passen!)

Ich habe herausgefunden, dass wenn ich nur eine Instaz der Klasse Datensatz aufrufe alles wunderbar geht. Es kommt nur bei der "instanziierung eines ganzen Arreys (wie im Prog. nötig, zu dem Fehler.
Kann mir jemand helfen? Bin am verzweilen und habe schon das ganze Netz durchforscht.

 

[Beni]

Wenn man einen Array hat, muss man ihn auch füllen.

for( ... )
  array[i] = new ...

 

[mic_checker]

public class Ausgaben { 

  protected Datensatz [] Verpfl;
  
  public Ausgaben (int n)
  {
  	Verpfl = new Datensatz[n];
  }

Vielleicht solltest du noch sicherstellen das n >= 0

  public boolean istBesser(Datensatz D1, Datensatz D2)
  {
  	if (D1.Betrag>=D2.Betrag) {return true;} else {return false;}  	
  }

Geht auf einfacher:

public boolean istBesser(Datensatz D1, Datensatz D2)
{
	return (D1.Betrag >= D2.Betrag);   	
}

Müsste zumindest so gehen...

 

[janhei]

Wenn man einen Array hat, muss man ihn auch füllen.

for( ... )
  array[i] = new ...

Das Stimmt auf jeden Fall. Aber ich glaube ich habe in dem Prog. alle Arrays gefüllt, oder habe ich das was übersehen?
Die einzelnen Instanzen sollen doch im Konstruktor der Klasse Datensatz gefüllt werden. Das geht aber nur wenn man einzelne instanzen erstellt. Nicht aber bei einem Array. Keine Ahnung warum. Habe ich aber getestet.

 

[mic_checker]

  protected Datensatz [] Verpfl;
 
  public Ausgaben (int n)
  {
     Verpfl = new Datensatz[n];
  }

Du hast das Array deklariert und im Konstruktor teilst du ihm die Größe mit. Danach zeigen die Referenzen "standardmäßig" erstma auf null.
kann in deinem source nichts derart finden....hast wohl vergessen.

 

[janhei]

Ich bin da jetzt vielleicht etwas schwer von Begriff, aber warum klappt das ganze dann ohne Probleme wenn ich den Aufruf mache:

  public Ausgaben (int n)
  {
     Datensatz test = new Datensatz();

Dann bekomme ich keine Fehlermeldung, das heißt, dann wird der Konstruktor Datensatz() aufgerufen. Warum nicht, wenn ich ein Array erzeuge?

Es handelt sich ja hier nicht um ein Array von Variablen (ich weiß, sind eigentlich auch klassen). Sondern um ein Array der selbst erstellten Klasse Datensatz.

 

[mic_checker]

Ich bin da jetzt vielleicht etwas schwer von Begriff, aber warum klappt das ganze dann ohne Probleme wenn ich den Aufruf mache:

  public Ausgaben (int n)
  {
     Datensatz test = new Datensatz();

Dann bekomme ich keine Fehlermeldung, das heißt, dann wird der Konstruktor Datensatz() aufgerufen. Warum nicht, wenn ich ein Array erzeuge?

Du deklarierst mit deiner ursprünglichen Anweisung ein Feld der Größe n, welches Objekte vom Typ Datensatz aufnehmen kann.

Nach der Deklaration zeigen die Referenzen auf null, allerdings versuchst du in deiner Ausgabe Methode schon eine Methode der Klasse aufzurufen....Teste es einfach mit dem andern Code.

 

[Beni]

Stell es dir so vor: mit "Datensatz test = new Datensatz()" kaufst du dir ein Buch.
Mit "Datensatz[] test = new Datensatz[ 123 ]" kaufst du dir einen Kasten, der mit Bücher gefüllt werden kann (es aber noch nicht ist), die Bücher für den Kasten musst du selbst noch kaufen.

Deshalb musst du deinen Array noch füllen, nachdem du ihn hergestellt hast.

 

[janhei]

Oh man, das ist mir jetzt richtig peinlich. Du hattest voll recht, ich hab's nur nicht beblickt, was du meinst. Habe einfach folgendes eingefügt:

protected Datensatz [] Verpfl;
  
  public Ausgaben (int n)
  {
  	Verpfl = new Datensatz[n];
         for(int i=0; i<Verpfl.length; i++)
  	{
  	  Verpfl[i]=new Datensatz();
  	}
  }

jetzt klappt alles! DANKE!