JTable verursacht NullPointerException

[mayer]

package Frame;

import DBAccess.Abfrage;
import DBAccess.Connect;
import Data.Fahrzeug;
import Data.FahrzeugListe;

import Model.AutoTableModel;
import java.sql.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.awt.Rectangle;

public class AutoVerwaltungFrame extends BaseFrame
{
  private JFrame hauptAuswahlFrame;
  private JLabel autoVerwLabel = new JLabel();
  private JTable autoTable;
  private JScrollPane autoTableJsp;
  private Connect con;
  private Abfrage autoAbfrage;
  private Fahrzeug fz;
  private FahrzeugListe fahrzeugListe = new FahrzeugListe();
  
  public AutoVerwaltungFrame(JFrame hauptAuswahlFrame, Connect con) 
  {
    this.hauptAuswahlFrame = hauptAuswahlFrame;
    this.con = con;
    readCarPoolFromDb();
    
    try 
    {
      jbInit();
    }
    
    catch(Exception e) 
    {
      System.out.println("Fehler beim Maskenaufbau von AutoVerwaltung");
    }
  }
  
  public void jbInit() throws Exception 
  {
    autoTable = new JTable(new AutoTableModel(fahrzeugListe));
    autoTableJsp = new JScrollPane(autoTable);

    autoTableJsp.setBounds(new Rectangle(40, 150, 950, 260));
    
    autoVerwLabel.setText("Auto - Verwaltung:");
    autoVerwLabel.setBounds(new Rectangle(35, 35, 295, 60));
    autoVerwLabel.setHorizontalTextPosition(SwingConstants.CENTER);
    autoVerwLabel.setHorizontalAlignment(SwingConstants.LEFT);
    autoVerwLabel.setFont(new Font("Tahoma", 3, 30));
   
    this.getContentPane().add(autoVerwLabel, null);
    this.getContentPane().add(autoTableJsp, null);
    this.setVisible(true);
    this.repaint();
  }
  
  private void readCarPoolFromDb() 
  {
    ResultSet result = null;
    autoAbfrage = new Abfrage(this.con.getConnection());
    result = autoAbfrage.getResult("select * from autopool");
    
    try 
    {
      while(result.next()) 
      {
        fz = new Fahrzeug();
        fz.setInventarNummer(result.getInt("InventarNummer"));
        fz.setMarke(result.getString("Marke"));
        fz.setBezeichnung(result.getString("Bezeichnung"));
        fz.setLeistung(result.getInt("Leistung"));
        fz.setHubraum(result.getInt("Hubraum"));
        fz.setFarbe(result.getString("Farbe"));
        fz.setPreis(result.getDouble("Preis"));
        fz.setZustand(result.getString("Zustand"));
        fz.setKmStand(result.getDouble("Km_Stand"));
        fz.setErstZulassung(result.getDate("Erstzulassung"));
        fz.setExtras(result.getString("Extras"));
        fz.setKontakt(result.getString("Kontakt"));
        fahrzeugListe.addFahrzeug(fz);
      }
    }
    
    catch(Exception e) 
    {
      System.out.println("Fehler beim Lesen von Datenbank");
    }
  }
}

-------------------
Fehlermeldung:
-------------------
java.lang.NullPointerException

at javax.swing.JTable.prepareRenderer(JTable.java:3731)

at javax.swing.plaf.basic.BasicTableUI.paintCell(BasicTableUI.java:1149)

at javax.swing.plaf.basic.BasicTableUI.paintCells(BasicTableUI.java:1051)

at javax.swing.plaf.basic.BasicTableUI.paint(BasicTableUI.java:974)

at javax.swing.plaf.ComponentUI.update(ComponentUI.java:142)

at javax.swing.JComponent.paintComponent(JComponent.java:541)

at javax.swing.JComponent.paint(JComponent.java:808)

at javax.swing.JComponent.paintChildren(JComponent.java:647)

at javax.swing.JComponent.paint(JComponent.java:817)

at javax.swing.JViewport.paint(JViewport.java:722)

at javax.swing.JComponent.paintChildren(JComponent.java:647)

at javax.swing.JComponent.paint(JComponent.java:817)

at javax.swing.JComponent.paintChildren(JComponent.java:647)

at javax.swing.JComponent.paint(JComponent.java:817)

at javax.swing.JComponent.paintChildren(JComponent.java:647)

at javax.swing.JComponent.paint(JComponent.java:817)

at javax.swing.JLayeredPane.paint(JLayeredPane.java:557)

at javax.swing.JComponent.paintChildren(JComponent.java:647)

at javax.swing.JComponent.paintWithOffscreenBuffer(JComponent.java:4794)

at javax.swing.JComponent.paintDoubleBuffered(JComponent.java:4740)

at javax.swing.JComponent.paint(JComponent.java:798)

at java.awt.GraphicsCallback$PaintCallback.run(GraphicsCallback.java:21)

at sun.awt.SunGraphicsCallback.runOneComponent(SunGraphicsCallback.java:60)

at sun.awt.SunGraphicsCallback.runComponents(SunGraphicsCallback.java:97)

at java.awt.Container.paint(Container.java:1312)

at sun.awt.RepaintArea.paint(RepaintArea.java:177)

at sun.awt.windows.WComponentPeer.handleEvent(WComponentPeer.java:260)

at java.awt.Component.dispatchEventImpl(Component.java:3678)

at java.awt.Container.dispatchEventImpl(Container.java:1627)

at java.awt.Window.dispatchEventImpl(Window.java:1606)

at java.awt.Component.dispatchEvent(Component.java:3477)

at java.awt.EventQueue.dispatchEvent(EventQueue.java:456)

at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:201)

at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:151)

at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:145)

at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:137)

at java.awt.EventDispatchThread.run(EventDispatchThread.java:100)

 

[foobar]

Wie sieht denn das AutoTablemodel aus?

BTW das repaint kannste dir sparen.

 

[ronny]

uh oh! ist das bei deiner while schleife wirklich so gewünscht?

du willst sicher nich

while (result.next())

haben! das sollte doch eher

while (result.hasNext())

sein! dann in deiner schleife machst du:

(MeineGewünschteKlasse) obj = (MeineGewünschteKlasse) result.next();

usw.

ansonsten verstehe ich den sinn dieser schleife nicht.....

Falls du mit eclipse arbeitest: da kannst du im Debug-Mode unter dem Panel Breakpoints
exceptions "catchen" -> d. h. wenn du dein prog im debug-mode durchlaufen läßt bleibt
er genau beim nullpointer hängen... dann siehst du ganz genau, welchen objekt evtl.
noch nicht instanziert wurde! (gibts sicher im jbuilder auch irgendwo....)

 

[mayer]

An ronny:
Habe schon alles durchdebuggt. Dabei meldet er mir keinen Fehler. Habe auch deinen Lösungsvorschlag ausporbiert, dieser funktioniert aber auch nicht.
-----------------------------------------------


An footbar:

Das AutoTableModel sieht folgendermaßen aus:

package Model;

import Data.FahrzeugListe;
import java.sql.Date;
import javax.swing.table.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class AutoTableModel extends AbstractTableModel
{
    public FahrzeugListe fahrzeugListe;
    
    public AutoTableModel(FahrzeugListe fl) 
    {
        this.fahrzeugListe = fl;
    }
    
    public int getColumnCount() 
    {
        return 12;
    }
    
    public int getRowCount() 
    {
        return fahrzeugListe.getRowCount();
    }
    
    public Object getValueAt(int rowIndex, int columnIndex) 
    {
        if(columnIndex == 0)
          return new Integer(fahrzeugListe.getFahrzeug(rowIndex).getInventarNummer());
          
        if(columnIndex == 1)
          return fahrzeugListe.getFahrzeug(rowIndex).getMarke();
          
        if(columnIndex == 2)
          return fahrzeugListe.getFahrzeug(rowIndex).getBezeichnung();
          
        if(columnIndex == 3)
          return new Integer(fahrzeugListe.getFahrzeug(rowIndex).getLeistung());
          
        if(columnIndex == 4)
          return new Integer(fahrzeugListe.getFahrzeug(rowIndex).getHubraum());
          
        if(columnIndex == 5)
          return fahrzeugListe.getFahrzeug(rowIndex).getFarbe();
          
        if(columnIndex == 6)
          return new Double(fahrzeugListe.getFahrzeug(rowIndex).getPreis());
          
        if(columnIndex == 7)
          return fahrzeugListe.getFahrzeug(rowIndex).getZustand();
          
        if(columnIndex == 8)
          return new Double(fahrzeugListe.getFahrzeug(rowIndex).getKmStand());
          
        if(columnIndex == 9)
          return fahrzeugListe.getFahrzeug(rowIndex).getErstZulassung();
        
        if(columnIndex == 10)
          return fahrzeugListe.getFahrzeug(rowIndex).getExtras();
          
        else
          return fahrzeugListe.getFahrzeug(rowIndex).getKontakt();
    }
    
    public boolean isCellEditable(int rowIndex, int columnIndex) 
    {
        return false;
    }
 
    
    public Class getColumnClass(int columnIndex) 
    {
        if(columnIndex == 0 || columnIndex == 3 || columnIndex == 4)
          return Integer.class;
          
        if(columnIndex == 6 || columnIndex == 8)
          return double.class;
          
        if(columnIndex == 9)
          return Date.class;
          
        else
          return String.class;
    }
           
    public  String getColumnName(int column) 
    {
        if(column == 0)
          return "InventarNr.";
          
        if(column == 1)
          return "Marke";
          
        if(column == 2)
          return "Bezeichnung";
          
        if(column == 3)
          return "Leistung";
          
        if(column == 4)
          return "Hubraum";
          
        if(column == 5)
          return "Farbe";
          
        if(column == 6)
          return "Preis";
          
        if(column == 7)
          return "Zustand";
          
        if(column == 8)
          return "Km-Stand";
          
        if(column == 9)
          return "Erst-Zulassung";
        
        if(column == 10)
          return "Extras";
          
        else
          return "Kontakt";
    }
  
}

 

[abollm]

Wenn du jetzt noch deine restlichen Klassen postest, könnte man dir sicher noch besser helfen.

 

[mayer]

Klasse: Fahrzeug

package Data;

import java.sql.*;

public class Fahrzeug 
{
  private int inventarNummer;
  private String marke;
  private String bezeichnung;
  private int leistung;
  private int hubraum;
  private String farbe;
  private double preis;
  private String zustand;
  private double kmStand;
  private Date erstZulassung;
  private String extras;
  private String kontakt;
  
  public void setInventarNummer(int inventarNummer) 
  {
    this.inventarNummer = inventarNummer;
  }
  
  public void setMarke(String marke) 
  {
    this.marke = marke;
  }
  
  public void setExtras(String extras) 
  {
    this.extras = extras;
  }
  
  public void setKontakt(String kontakt)
  {
    this.kontakt = kontakt;
  }
  
  public void setBezeichnung(String bezeichnung) 
  {
    this.bezeichnung = bezeichnung;
  }
  
  public void setLeistung(int leistung) 
  {
    this.leistung = leistung;
  }
  
  public void setHubraum(int hubraum) 
  {
    this.hubraum = hubraum;
  }
  
  public void setFarbe(String farbe) 
  {
    this.farbe = farbe;
  }
  
  public void setPreis(double preis) 
  {
    this.preis = preis;
  }
  
  public void setZustand(String zustand) 
  {
    this.zustand = zustand;
  }
  
  public void setKmStand(double kmStand) 
  {
    this.kmStand = kmStand;
  }
  
  public void setErstZulassung(Date erstZulassung) 
  {
    this.erstZulassung = erstZulassung;
  }
  
  public int getInventarNummer() 
  {
    return inventarNummer;
  }
  
  public String getMarke() 
  {
    return marke;
  }
  
  public String getBezeichnung() 
  {
    return bezeichnung;
  }
  
  public int getLeistung() 
  {
    return leistung;
  }
  
  public int getHubraum()
  {
    return hubraum;
  }
  
  public String getFarbe() 
  {
    return farbe;
  }
  
  public double getPreis() 
  {
    return preis;
  }
  
  public String getZustand() 
  {
    return zustand;
  }
  
  public double getKmStand()
  {
    return kmStand;
  }
  
  public Date getErstZulassung()
  {
    return erstZulassung;
  }
  
  public String getExtras() 
  {
    return extras;
  }
  
  public String getKontakt()
  {
    return kontakt;
  }
}

Klasse FahrzeugList:

package Data;

public class FahrzeugListe 
{
  private Fahrzeug[] fahrzeugPool = new Fahrzeug[30];
  private int index = 0;
  
  public void addFahrzeug(Fahrzeug f) 
  {
    if(index >= fahrzeugPool.length)
    {
      System.out.println("Max. Fahrzeuganzahl erreicht ("+fahrzeugPool.length+")!!");
      return;
    }
    
    fahrzeugPool[index] = f;
    index++;
  }
  
  public Fahrzeug getFahrzeug(int i) 
  {
    return fahrzeugPool[i];
  }
  
  public int getRowCount() 
  {
    return index;
  }
}

Klasse Connect:

package DBAccess;

import java.sql.*;
import javax.swing.JFrame;
import javax.swing.JOptionPane;


public class Connect {
    
    private Connection con = null;
    private JFrame owner;
        
    public Connect(String benutzer, char[] passwort, JFrame owner) {

        this.owner = owner;
        
//      Registrieren des JDBC-Treibers
        try {
            Class.forName("org.gjt.mm.mysql.Driver"); 
        }
        
        catch(ClassNotFoundException cnfe) {
            JOptionPane.showMessageDialog(owner, 
                                          "JDBC Treiber konnte nicht geladen werden!",
                                           "ACHTUNG", JOptionPane.INFORMATION_MESSAGE);
            System.exit(0);
        }
        
//      Verbindung zur Datenbank aufbauen
        try {
            con = DriverManager.getConnection("jdbc:mysql://10.1.1.3:3306/autohaus",benutzer,new String(passwort));
        }
        
        catch(SQLException sle) {
            JOptionPane.showMessageDialog(owner, 
                                          "Es konnte keine Verbindung zur Datenbank hergestellt werden!",
                                           "ACHTUNG", JOptionPane.INFORMATION_MESSAGE);
            return;
        }
    }
    
    
    public Connection getConnection() {
        return con;
    }
    
    public void closeConnection() {
        try {
            con.close();
        }
        
        catch(SQLException sle) {
            System.out.println("Verbindung konnte nicht geschlossen werden");
            System.exit(0);
        }
    }
    
}

Klasse HauptAuswahlFrame:

package Frame;

import ActionListener.*;
import DBAccess.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.sql.*;


public class HauptAuswahlFrame extends BaseFrame
{
 
  private ImageIcon kundeIcon = new ImageIcon("C:/Dokumente und Einstellungen/Manuel/Eigene Dateien/HTBL-EDVO/Preu/Opel-Projekt/StenggSoftware/AdminSoft/person.gif");
  private ImageIcon autoIcon = new ImageIcon("C:/Dokumente und Einstellungen/Manuel/Eigene Dateien/HTBL-EDVO/Preu/Opel-Projekt/StenggSoftware/AdminSoft/auto.gif");
  private ImageIcon ersatztIcon = new ImageIcon("C:/Dokumente und Einstellungen/Manuel/Eigene Dateien/HTBL-EDVO/Preu/Opel-Projekt/StenggSoftware/AdminSoft/schraubenschluessel.gif");
  private ImageIcon logoLinks = new ImageIcon("C:/Dokumente und Einstellungen/Manuel/Eigene Dateien/HTBL-EDVO/Preu/Opel-Projekt/StenggSoftware/AdminSoft/car.gif");
  
  private JButton kundenVerw = new JButton();
  private JButton autoVerw = new JButton();
  private JButton ersatztVerw = new JButton();
  private JLabel autohausLabel = new JLabel();
  private JLabel logoLinksLabel = new JLabel(logoLinks);
  private Connect con;

  public HauptAuswahlFrame(final Connect con)
  { 
    this.con = con;
  
//  Beim Klick auf Schließen
    this.addWindowListener(new WindowAdapter() 
       {
          public void windowClosing(WindowEvent e) 
          {
//          Verbindung trennen
            con.closeConnection();
            System.exit(0);
          }
       });

//  Aufbau der GUI
    try
    {
      jbInit();
    }
    catch(Exception e)
    {
      System.out.println("Fehler beim Aufbau von jbInit bei HauptAuswahlFrame");
    }

    
  }

  private void jbInit() throws Exception
  {
    kundenVerw.setIcon(kundeIcon);
    kundenVerw.setText("KUNDEN - VERWALTUNG                                ");
    kundenVerw.setBounds(new Rectangle(375, 265, 315, 55));
    kundenVerw.setIconTextGap(50);
    kundenVerw.setHorizontalAlignment(SwingConstants.LEFT);
    kundenVerw.setFont(new Font("Arial Narrow", 1, 14));
    kundenVerw.setBorder(BorderFactory.createRaisedBevelBorder());
    kundenVerw.setActionCommand("kundenVerw");
    kundenVerw.addActionListener(new HauptAuswahlAction(this,con));
    
    autoVerw.setIcon(autoIcon);
    autoVerw.setText("AUTO - VERWALTUNG                                  ");
    autoVerw.setBounds(new Rectangle(375, 375, 315, 55));
    autoVerw.setHorizontalAlignment(SwingConstants.LEFT);
    autoVerw.setIconTextGap(40);
    autoVerw.setFont(new Font("Arial Narrow", 1, 14));
    autoVerw.setBorder(BorderFactory.createRaisedBevelBorder());
    autoVerw.setActionCommand("autoVerw");
    autoVerw.addActionListener(new HauptAuswahlAction(this,con));
    
    ersatztVerw.setIcon(ersatztIcon);
    ersatztVerw.setText("ERSATZTEIL - VERWALTUNG                        ");
    ersatztVerw.setBounds(new Rectangle(375, 480, 315, 55));
    ersatztVerw.setHorizontalAlignment(SwingConstants.LEFT);
    ersatztVerw.setIconTextGap(35);
    ersatztVerw.setFont(new Font("Arial Narrow", 1, 14));
    ersatztVerw.setBorder(BorderFactory.createRaisedBevelBorder());
    ersatztVerw.setActionCommand("ersatztVerw");
    ersatztVerw.addActionListener(new HauptAuswahlAction(this,con));
    
    autohausLabel.setText("AUTOHAUS - VERWALTUNG");
    autohausLabel.setBounds(new Rectangle(320, 125, 455, 65));
    autohausLabel.setHorizontalAlignment(SwingConstants.CENTER);
    autohausLabel.setHorizontalTextPosition(SwingConstants.CENTER);
    autohausLabel.setFont(new Font("Tahoma", 3, 31));
    
    logoLinksLabel.setBounds(new Rectangle(10, 265, 325, 240));
    logoLinksLabel.setBackground(SystemColor.control);
    
    
    this.getContentPane().add(logoLinksLabel, null);
    this.getContentPane().add(kundenVerw, null);
    this.getContentPane().add(autohausLabel, null);
    this.getContentPane().add(ersatztVerw, null);
    this.getContentPane().add(autoVerw, null);
    
  }
  

}

Klasse HauptAuswahlAction:

package ActionListener;

import DBAccess.Connect;
import Frame.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.sql.*;

public class HauptAuswahlAction implements ActionListener
{
    private JFrame hauptAuswahlFrame;
    private Connect con;
    
    
    public HauptAuswahlAction(JFrame hauptAuswahlFrame, Connect con) 
    {
      this.hauptAuswahlFrame = hauptAuswahlFrame;
      this.con = con;
      
    }
    
    public void actionPerformed(ActionEvent e) 
    {
    
//    Modul Auto-Verwaltung
      if(e.getActionCommand().equals("autoVerw")) 
      {
        hauptAuswahlFrame.setVisible(false);
        hauptAuswahlFrame.dispose();
        
        AutoVerwaltungFrame autoVerw = new AutoVerwaltungFrame(hauptAuswahlFrame,con);
      }


//    Modul Kunden-Verwaltung        
      if(e.getActionCommand().equals("kundenVerw"))
      {
        System.out.println("kundenVerw");
      }

//    Modul Ersatzteil-Verwaltung    
      if(e.getActionCommand().equals("ersatztVerw"))
      {
        System.out.println("ersatztVerw");
      }
      
    }
}

 

[abollm]

Hi,

vielleicht wäre es besser, wenn du künftig bei einer derartigen Größe des Projekts lieber den Code per Mail oder PN an jemanden Interessierten auf Nachfrage sendest. Ich habe nicht vermutet, dass du gleich so viele Klassen hast. Den Code habe ich mir gezogen, allerdings fehlen mir z.B. noch die Klassen Abfrage und BasicFrame.

Wenn du willst, kannst du mir das gesamte Projekt einmal per E-mail möglichst gepackt senden:

bollm at gmx dot de

Ich würde dann versuchen, das Problem zu analysieren, hoffentlich die Lösung finden und dir mitteilen und die wesentlichen Erkenntniss anschließend hier posten.

Einverstanden?

 

[ronny]

@mayer:

sorry, sorry, hab mich bei der schleife geirrt... dachte das resultset hat methoden wie ein iterator... -> das kommt davon, wenn man nich gleich das ausprobiert, was man da hinschreibt und wenn man schon länger nimmer damit gearbeitet hat...

kann sonst leider nix ungewöhnliches beim drüberschauen erkennen....
hoffentlich findet abollm den fehler...

 

[abollm]

Für alle Interessierten:

Nach Analyse des Codes und diversen Test steht m.E. fest, dass hier ein Design-Fehler bei der Klassenstruktur der Grund für die Fehlermeldungen ist.

Ein entscheidender Hintergrund ist dabei, dass die meisten Swing-Methoden nicht thread-sicher sind. Deshalb sollen insbesondere Prozesse, die u.U. länger andauern (-> DB-Zugriffe) können, nicht im Event-Thread durchgeführt werden. Hierfür sollte ein neuer Thread erzeugt werden.

Wenn der längere Zeit benötigende Code Swing-Komponenten der GUI aktualisieren muss, kann auch die SwingUtilities-Klasse im Event-Thread benutzt werden.