Problem mit Files

[comp89]

hi nun bin ich mit meinem kalender so gut wie fertig.
aber dieser teil funktioniert leider noch nicht so ganz. in dem teil wird aus den textfiles die termine der nächsten 5 tage in die jtable mit DefaultTableModel (model) geschrieben, das ganze wird jedoch nur einmal ausgeführt und nur die termine des heutigen tages werden in dem jtable angezeigt. hab ich mit den files was falsch gemacht?

    private void getTableData(){
        byte day = Zeit.getDayOfMonth();
        byte month = Zeit.getMonthNumber();
        short year = Zeit.getYear();
        GregorianCalendar next = new GregorianCalendar();
        File f;
        for(int i = 0; i < 5; i++){ // soll 5 mal ausgeführt werden(für 5 tage)
            StringBuffer buffer = new StringBuffer();
            f = new File("Kalender" +  File.separator + "daten" + File.separator + year + File.separator + month + File.separator + day);
            if(f.canRead()){
                try{
                    BufferedReader in = new BufferedReader(new FileReader(f));
                    for (String b;(b = in.readLine()) != null;buffer.append(b));
                    in.close();
                    
                    StringTokenizer term = new StringTokenizer(buffer.toString(), "-----");
                    while(term.hasMoreTokens()){
                        StringTokenizer teil = new StringTokenizer(term.nextToken(), "||");
                        model.addRow(new Object[]{(Object)(day + "." + month + "." + year), uhr(teil.nextToken(), teil.nextToken()), teil.nextElement(), teil.nextElement()});
                    }
                    next = Zeit.getNextDay(next);
                    day = (byte)next.get(GregorianCalendar.DAY_OF_MONTH);
                    month = (byte)next.get(GregorianCalendar.MONTH);
                    year = (short)next.get(GregorianCalendar.YEAR);
                }
                catch(Exception exc){}
            }
        }
    }

 

[foobar]

catch(Exception exc){}

Was soll denn das schon wieder sein? Lass dir mal den Stacktrace und eine paar Werte ausgeben, damit du siehst an welcher Stelle der Fehler steckt.

 

[comp89]

nein es gibt keinen fehler und währenddessen hab ich eh schon verbessert

                catch(IOException exc){
                    JOptionPane.showMessageDialog(null, "Beim Einlesen der Termine trat ein Fehler auf!", "ERROR", JOptionPane.ERROR_MESSAGE);
                }

 

[comp89]

das mit den werten ausgeben:

ich hab versucht mit System.err.println(i) am Anfang der for-Schleife da wird 0 1 2 3 4 ausgegeben also richtig
2. inm try-block ebenfalls mit System.err.println(day), da wird mir nur 15 (also der heutige tag ausgegeben)

edit: aber in den catch block kommt man nicht

 

[foobar]

catch(IOException exc){
                    JOptionPane.showMessageDialog(null, "Beim Einlesen der Termine trat ein Fehler auf!", "ERROR", JOptionPane.ERROR_MESSAGE);
                }

Das ist schon besser als gar nichts, aber trotzdem weißt du noch nicht was für eine Exception es war ;-)

catch(IOException exc)
{
   exc.printStacktrace();
   JOptionPane.showMessageDialog(null, "Beim Einlesen der Termine trat ein Fehler auf!", "ERROR", JOptionPane.ERROR_MESSAGE);
 }

Hast du dir mal ein paar Werte ausgeben lassen, um zu sehen an welchen Stellen was ankommt und wo nicht?

 

[comp89]

ja hab ich eh schon geschrieben, und in den catch-block kommt man ja gar nicht

 

[foobar]

Lass dir einfach mal alle relevanten Werte ausgeben z.b. buffer , die einzelnen Tokens, einen Wert in der While-Schleife, dann siehst du immer genau was passiert.

odel.addRow(new Object[]{(Object)(day + "." + month + "." + year), uhr(teil.nextToken(), teil.nextToken()), teil.nextElement(), teil.nextElement()});

Warum greifst du zuerst auf nextToken und dann auf nextElement zu?

 

[comp89]

weil nextToken() hat als rückgabetyp einen string, nextElement() ein objekt
bei dem aufruf der methode uhr(String stunde, String minute)
und da da ganz sicher int drinnen sind, und man int und String besser verbinden kann als int und Object mach ichs einfach so, sollt ich das ändern?

 

[foobar]

Probiers mal aus, ich weiß nicht ob das der Fehler ist.

 

[comp89]

nein war nicht der fehler
das war eine gute idee mit dem alle werte ausgeben.
das problem war das ich int month = 12; (dezember) mein, der gregoriancalendar für dezember aber 11 zurück gibt, also hat er im nächsten schritt (für den nächsten tag) die termine vom 16.november gesucht und da warn klarerweise keine da
danke