Hinweis zu Fehler im Kalender-Code - Aufgabe 5.3 aus Grundkurs Programmieren in Java 7. Auflage

[schnugur]

Nabend,

ich (lernender) habe eine kurze Frage bezüglich einer Aufgabe aus meinen Buch
"Grundkurs Programmieren in Java (7.Auflage)". Natürlich könnte ich mir auch die Lösung direkt anschauen, jedoch möchte ich zuvor den steinigen Weg dahin begreifen 🙂

Folgende Aufgabe soll ich lösen:

"Seite 130 - Aufgabe 5.3
Schreiben Sie ein Kalenderprogramm, das Termine fuer die Jahre 2000 bis 2009 verwaltet.
Verwenden Sie hierzu ein vierdimensionales Feld, und beruecksichtigen Sie
auch Schaltjahre."

Bevor ich mich mit den 10 Jahren und den Schlatjahren beschäftige, wollte ich erst einmal den Code insofern bearbeiten, das ich neben der (funktionierenen) Auswahl von [Tag][Stunde] auf [Monat][Tag][Stunde] komme. Wenn ich meinen Code in "Java-Editor 13.27" starten möchte kommt folgende Fehlermeldung:

"Exception in thread "main" java.lang.NullPointerException
at JahresKalenderVereinfacht.main(JahresKalenderVereinfacht.java:28)"

import Prog1Tools.IOTools;
public class JahresKalenderVereinfacht {
  public static void main (String[] args) {
    //Seite 125 - folgende Zeile bewirkt exakt das Gleiche
    //kann nur bei "rechteckigen" Arrays erzeugt werden die nur
    //aus Zeilen gleicher Länge bestehen
    String[][][] termine;
    //Es wird ein Feld in der ersten Dimension mit 12 Einträgen belegt,
    //die jeweils auf ein zweidimensionales Feld verweisen
    termine = new String[12][][];
   
    termine[0] = new String[31][24]; // Januar
    termine[1] = new String[28][24]; // Februar
    termine[2] = new String[31][24]; // März
    termine[2] = new String[30][24]; // April
    termine[2] = new String[31][24]; // Mai
    termine[2] = new String[30][24]; // Juni
    termine[2] = new String[31][24]; // Juli
    termine[2] = new String[31][24]; // August
    termine[2] = new String[30][24]; // September
    termine[2] = new String[31][24]; // Oktober
    termine[10]= new String[30][24]; // November
    termine[11]= new String[31][24]; // Dezember
   
    //im Vorfeld muss das Feld mit Startwerten belegt werden (leere String)
    //dies wird ermöglicht durch folgende verschachtelte Schleife - Seite 126
    for (int i = 0; i < termine.length; i++)
      for (int j = 0; j < termine[i].length; j++)
        for (int k = 0; k < termine[i][j].length; k++)
          termine[i][j][k] = "";
   
    //Das Hauptprogramm in einer Schleife
    boolean fertig=false;
    while (!fertig) {
      //Zuerst ein Bildschirmmenü
      System.out.println("1 = Neuer Eintrag");
      System.out.println("2 = Termine ausgaben");
      System.out.println("3 = Programm beenden");
      int auswahl=IOTools.readInteger("Ihre Wahl:");
      //Nun eine Fallunterscheidung
      switch (auswahl) {
        case  1: //Termine eingeben
          int monat=IOTools.readInteger("Welcher Monat?");
          if (monat<1 || monat>12) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          int tag=IOTools.readInteger("Welcher Tag?");
          if (tag<1 || tag>31) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          int nummer=IOTools.readInteger("Wie viel Uhr?");
          if (nummer<0 || nummer>23) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          String eingabe=IOTools.readLine("Termin:");
          termine[monat-1][tag-1][nummer]=eingabe; //Termine einordnen
          break;
        case  2: //Termine ausgeben
          int m=IOTools.readInteger("Welcher Monat?");
          if (m<1 || m>12) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          int t=IOTools.readInteger("Welcher Tag?");
          if (t<1 || t>31) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          for (int j=0;j<termine[m-1].length; j++) {
            for (int i=0; i<termine[t-1].length; i++) {
              System.out.println(i + " Uhr: " + termine[t-1][i]);
            } // end of for
          } // end of for
          break;
        case  3: //Programm beenden
          fertig=true;
          break;
        default: //Falsche Zahl eingegeben
          System.out.println("Eingabefehler!");
      } // end of switch
    } // end of while
  } // end of for
}

So wie ich die Fehlermeldung jetzt verstehe, muss dieser in der Zeile 28 sein. Jedoch fehlt mir vielleicht der Durchblick bei den Problem. Habe ich bei der Definition einen Fehler gemacht?

Ausgegeben soll die Frage nach:
1. Neuer Eintrag im Kalender
2. Termine aus dem Kalender ausgeben
3. Programm beenden

Welcher Monat -> Eingabe
Welcher Tag -> Eingabe
Welche Stunde -> Eingabe

Danke für die Hilfestellung 🙂

 

[schnugur]

termine[0] = new String[31][24]; // Januar
termine[1] = new String[28][24]; // Februar
termine[2] = new String[31][24]; // März
termine[2] = new String[30][24]; // April
termine[2] = new String[31][24]; // Mai
termine[2] = new String[30][24]; // Juni
termine[2] = new String[31][24]; // Juli
termine[2] = new String[31][24]; // August
termine[2] = new String[30][24]; // September
termine[2] = new String[31][24]; // Oktober
termine[10]= new String[30][24]; // November
termine[11]= new String[31][24]; // Dezember

Gut hab den Fehler selber gefunden, habe wohl unbemerkt immer [2] geschrieben statt dies fortzuführen 😛

 

[schnugur]

So nächster fehler...

import Prog1Tools.IOTools;
public class JahresKalenderVereinfacht {
  public static void main (String[] args) {
    //Seite 125 - folgende Zeile bewirkt exakt das Gleiche
    //kann nur bei "rechteckigen" Arrays erzeugt werden die nur
    //aus Zeilen gleicher Länge bestehen
    String[][][] termine;
    //Es wird ein Feld in der ersten Dimension mit 12 Einträgen belegt,
    //die jeweils auf ein zweidimensionales Feld verweisen
    termine = new String[12][][];
   
    termine[0] = new String[31][24]; // Januar
    termine[1] = new String[28][24]; // Februar
    termine[2] = new String[31][24]; // März
    termine[3] = new String[30][24]; // April
    termine[4] = new String[31][24]; // Mai
    termine[5] = new String[30][24]; // Juni
    termine[6] = new String[31][24]; // Juli
    termine[7] = new String[31][24]; // August
    termine[8] = new String[30][24]; // September
    termine[9] = new String[31][24]; // Oktober
    termine[10]= new String[30][24]; // November
    termine[11]= new String[31][24]; // Dezember
   
    //im Vorfeld muss das Feld mit Startwerten belegt werden (leere String)
    //dies wird ermöglicht durch folgende verschachtelte Schleife - Seite 126
    for (int i = 0; i < termine.length; i++)
      for (int j = 0; j < termine[i].length; j++)
        for (int k = 0; k < termine[i][j].length; k++)
          termine[i][j][k] = "";
   
    //Das Hauptprogramm in einer Schleife
    boolean fertig=false;
    while (!fertig) {
      //Zuerst ein Bildschirmmenü
      System.out.println("1 = Neuer Eintrag");
      System.out.println("2 = Termine ausgaben");
      System.out.println("3 = Programm beenden");
      int auswahl=IOTools.readInteger("Ihre Wahl:");
      //Nun eine Fallunterscheidung
      switch (auswahl) {
        case  1: //Termine eingeben
          int monat=IOTools.readInteger("Welcher Monat?");
          if (monat<1 || monat>12) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          int tag=IOTools.readInteger("Welcher Tag?");
          if (tag<1 || tag>31) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          int nummer=IOTools.readInteger("Wie viel Uhr?");
          if (nummer<0 || nummer>23) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          String eingabe=IOTools.readLine("Termin:");
          termine[monat-1][tag-1][nummer]=eingabe; //Termine einordnen
          break;
        case  2: //Termine ausgeben
          int m=IOTools.readInteger("Welcher Monat?");
          if (m<1 || m>12) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          int t=IOTools.readInteger("Welcher Tag?");
          if (t<1 || t>31) {
            System.out.println("Eingabefehler!");
            break;
          } // end of if
          for (int j=0;j<termine[m-1].length; j++) {
            for (int i=0; i<termine[t-1].length; i++) {
              System.out.println(i + " Uhr: " + termine[m-1][t-1][i]);
            } // end of for
          } // end of for
          break;
        case  3: //Programm beenden
          fertig=true;
          break;
        default: //Falsche Zahl eingegeben
          System.out.println("Eingabefehler!");
      } // end of switch
    } // end of while
  } // end of for
}

"Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 24
at JahresKalenderVereinfacht.main(JahresKalenderVereinfacht.java:74)"

Zumindest klappt nun die Eintragung und Abrufung, jedoch kommt es am Ende zu dieser oben genannten Meldung....Weiter gehts 🙂

 

[kneitzel]

Also die Fehlerbeschreibung kann ich jetzt nicht ganz nachvollziehen. Evtl. kannst Du immer genauer angeben, was Du machst und dann bitte die Zeile markieren - hier im Forum sehe ich nicht, was Zeile 74 in JahresKalenderVereinfacht.java ist.

Was ich aber sehe ist ein Problem in

         for(int j=0;j<termine[m-1].length; j++){
           for(int i=0; i<termine[t-1].length; i++){
             System.out.println(i +" Uhr: "+ termine[m-1][t-1][i]);
           }// end of for
         }// end of for

Also äußere Schleife iteriert über die Tage des Monats m. So Du das willst, ist es ok. Aber dann hast Du eine zweite Iteration bei der Du über die Tage eines Monats iterierst - jedoch gibt der eingegebene Tag t den Monat an. Das ist natürlich falsch.
Bei der Anzeige hast Du es dann ja prinzipiell richtig gemacht - da hast Du termine[m-1][t-1].

Von der Logik her: Du brauchst die äußere for schleife ja nicht. Die kannst Du also direkt weglassen.

Und schon einmal darüber nachgedacht, den Code etwas zu optimieren / lesbarer zu gestalten?
a) Du hast derzeit alles in main - das ist nicht ganz so toll. Wie wäre es mit einer Klasse, die dann die termine als Instanzvariable hat? Die Initialisierung käme dann in den Konstruktor. Und Zugriffe würdest Du dann in Funktionen packen a.la. terminHinzufuegen(int monat, int tag, int zeit, string termin) oder termineAusgeben(int monat, int tag).
b) Du hast oft (fast) identischen Code. Den kannst Du vereinfachen, indem Du das teilweise in Funktionen auslagerst oder eben automatisierst. So könntest Du die Initialisierung des Arrays auch über eine Schleife machen. Dazu erstellst Du einfach nur ein Array mit der Anzahl der Tage als { 31, 28, 31, 30, 31, .... } und schon hast Du da auch eine Schleife in der Du

termine[i] = new String[AnzahlTage[i]][24]();

setzt.

Das aber nur als Anregung für weitere Verbesserungen.

 

[kneitzel]

Ach so - ganz vergessen:
c) versuch Variablen besser zu benenen. m und t sind zum Beispiel nicht gut. monat und tag könnten da noch bessere Namen sein.
d) (Gehört eigentlich zu den ersten Punkten) Kommentare nach den } sind aus meiner Sicht nicht nötig. Wenn Du das Gefühl hast, dass dies notwendig ist, dann könnte das ein Zeichen sein, dass Du deutlich zu lange und zu sehr gestaffelte Funktionen hast.
Hier wäre es also sinnvoll, einfach vieles auszugliedern. Dann hätte man die Initialisierung in einer eigenen Funktion. Oder so ein Switch wäre dann nur noch etwas wie:

switch (auswahlEinholen()) {
  case 1: 
    terminEingeben();
    break;
  case 2:
    terminAusgeben();
    break;
  case 3: 
    fertig=true;
    break;
}

Das aber nur als weitere Anregungen. Viel Spass beim weiter basteln.