SQLite NullPointerException in preparedStatement.close();

[ve987]

Hallo,

beim Aufruf der insertInitialValues(); Methode, bekomme ich eine NullPointerException in preparedStatement.close();

Kann mir bitte dabei jemand helfen?

Anbei mein Code:

MAIN KLASSE:

public class MainClass {

    public static void main(String[] args) {
    
        
        Connection connection = new dbConnection().getConnection();
        
        if (connection != null) {
            System.out.println("verbunden");
        }

        dbStatement dbStatement= new dbStatement();
        
//        try {
//            dbStatement.createTable();
//        } catch (Exception e) {
//            e.printStackTrace();
//        }
    
      
          
        try {
         
            dbStatement.insertInitialValues();
        } catch (Exception e) {
        e.printStackTrace();
        }
        
        
    }

}

getConnection KLASSE:

public class dbConnection { 
    
    private static final String DB_Name= "usersTest.db";
    private static final String PATH = "C:\\Users\\xxxxx\\Desktop\\Java\\src\\";
    private static final String CONNECTION_STRING ="jdbc:sqlite:" + PATH + DB_Name; 
    
    
    public static Connection getConnection() {
        
        
        Connection connection = null;

        try {
            connection = DriverManager.getConnection(CONNECTION_STRING);
        } catch (SQLException e) {
        
            e.printStackTrace();
        }
            
        return connection;
                      
    }

}

"Hauptklasse"

public class dbStatement {
    

    private final String COLUMN_ID = "id";
    private final String COLUMN_CATEGORY = "category";
    private final String COLUMN_PRICE = "price";

    private final String TABLE_NAME = "expenses";

    Connection connection = null;
    PreparedStatement preparedStatement = null;
    ResultSet resultSet = null;
    
public void createTable() throws SQLException { 
        
        String createTable = "CREATE TABLE IF NOT EXISTS " + TABLE_NAME + 
        "(" + 
                COLUMN_ID + " INTEGER PRIMARY KEY, " +    
                COLUMN_CATEGORY + " TEXT, " +
                COLUMN_PRICE + " REAL "
                
                +")";
                

        executeQuery(createTable);
                
    }
    
    public void executeQuery(String query) throws SQLException {
        
        try {
            connection = dbConnection.getConnection(); 
            preparedStatement = connection.prepareStatement(query); 
            
            if (preparedStatement != null) {
                preparedStatement.execute(); 
            }
            
        } finally {
            preparedStatement.close(); 
            connection.close(); 
            
        }
    }
    
    public void insertInitialValues() throws SQLException {
        
        String insertInitialValues= "INSERT INTO " + TABLE_NAME +
                "(COLUMN_CATEGORY, COLUMN_PRICE)"
                + "VALUES (?, ?)";
        
        try {
            connection= dbConnection.getConnection(); 
            preparedStatement= connection.prepareStatement(insertInitialValues);
            
             preparedStatement.setString(1, "test");
            preparedStatement.setDouble(2, 3.3);
            
            
            
            
            if (preparedStatement != null) {
                 preparedStatement.executeUpdate();
            }
            
        } finally {
        
                preparedStatement.close();
                connection.close();
            
            
            
        }
    }

}

 

[Barista]

Offensichtlich ist das PreparedStatement null.

Ein if drum herum wird helfen.

if (preparedStatement != null) {
    preparedStatement.close();
}

Aber sicher gibt es noch einen Grund, warum das PreparedStatement null ist, dieser Fehler wird durch die NullPointerException verdeckt.

 

[ve987]

Danke Barista. Ich habe es ergänzt. Und jetzt kommt der nächste Fehler:

[SQLITE_ERROR] SQL error or missing database (table expenses has no column named COLUMN_CATEGORY)

in:

preparedStatement= connection.prepareStatement(insertInitialValues);

Die Tabelle wurde aber erfolgreich erstellt...

 

[kneitzel]

Du hast ja auch die Variablennamen in den SQL Befehl geschrieben und nicht den Inhalt der Variablen.

 

[ve987]

Problem gelöst ich habe eine neue Tabelle erstellt und wurde alles übernommen.

public void createTable() throws SQLException {
        
        String createTable = "CREATE TABLE IF NOT EXISTS " + TABLE_NAME +
        "(" +
           COLUMN_ID + " INTEGER PRIMARY KEY, " +   
           COLUMN_CATEGORY + " TEXT, " +
           COLUMN_PRICE + " TEXT "
                
                +")";
                
        try {
            connection = dbConnection.getConnection();
            preparedStatement = connection.prepareStatement(createTable);
            
            if (preparedStatement != null) {
                preparedStatement.execute();
            }
            
        } finally {
            if (preparedStatement != null) {
                preparedStatement.close();
            }
            
            connection.close();
            
        }

    }

    public void insertInitialValues() throws SQLException {
        
        
        String insertInitialValues= "INSERT INTO " +  TABLE_NAME 
            +  " VALUES (?,?,?)" ;
        
        try {
            connection= dbConnection.getConnection();
            preparedStatement= connection.prepareStatement(insertInitialValues);
            
            if (preparedStatement != null) {
                System.out.println("verknüpft");
            }
            

            preparedStatement.setString(2, "test");
            preparedStatement.setString(3, "test");
            
            
            
            if (preparedStatement != null) {
                 preparedStatement.executeUpdate();
            }
            
        } finally {
        
            if (preparedStatement != null) {
                preparedStatement.close();
            }
                
                connection.close();
            
        }
    }

 

[kneitzel]

Also nur um es noch einmal im Detail zu erläutern, da ich das Gefühl habe, dass die Ursache nicht verstanden wurde.

        String insertInitialValues= "INSERT INTO " + TABLE_NAME +
                "(COLUMN_CATEGORY, COLUMN_PRICE)"
                + "VALUES (?, ?)";

Hier hast Du also als Spalten die Namen angegeben: COLUMN_CATEGORY und COLUMN_PRICE. Die Spalten heissen aber anders und das sind nur die Namen der Konstanten.

Damit Du den Inhalt der Konstanten nimmst, musst Du das so machen wie auch bei dem create table Statement: Den String beenden und dann mit + die Variable anhängen und dann mit + weiter machen (als eine Möglichkeit). Das wäre dann gewesen:

        String insertInitialValues= "INSERT INTO " + TABLE_NAME +
                "(" + COLUMN_CATEGORY + ", " + COLUMN_PRICE + ") "
                + "VALUES (?, ?)";

Die Spaltennamen weglassen geht prinzipiell auch, aber wenn dann bei der Datenbank die Reihenfolge der Spalten sich ändert oder so, dann hast Du ein Problem. Das ist ein Grund, wieso ich oft SELECT * und so auch vermeide.

 

[ve987]

Vielen lieben Dank für die ausführliche Erklärung! Jetzt verstehe ich das und werde auf jeden Fall berücksichtigen in Zukunft

 

[fhoffmann]

dbStatement dbStatement= new dbStatement();

Möglicherweise ist das in Java sogar erlaubt; ich finde es aber sehr verwirrend, wenn eine Variable genauso heißt wie eine Klasse.
Du solltest dich an die Konvention halten, dass Klassennamen mit einem Großbuchstaben beginnen und Variablennamen mit einem Kleinbuchstaben beginnen.

 

[ve987]

Möglicherweise ist das in Java sogar erlaubt; ich finde es aber sehr verwirrend, wenn eine Variable genauso heißt wie eine Klasse.
Du solltest dich an die Konvention halten, dass Klassennamen mit einem Großbuchstaben beginnen und Variablennamen mit einem Kleinbuchstaben beginnen.

Stimmt, du hast völlig recht, das habe ich übersehen