Java NullPointerException

[alaska94]

Hallo,

ich versuche mich gerade an Java und Datenbanken.

Ich habe folgende zwei Klassen geschrieben.

import java.sql.ResultSet;
import java.sql.SQLException;

public class Ausgaben {

	/**
	 * @param args
	 * @throws SQLException 
	 */
	public static void main(String[] args) throws SQLException {
		// TODO Auto-generated method stub
		Datenbank db=new Datenbank();
		ResultSet result;
		db.verbinden();
		result=db.auslesen("Select * from benutzer");
		System.out.println(result.getString(1));
	}

}

import java.sql.*;

public class Datenbank {

	private Connection con;
	private Statement stm;
	private String treiber="jdbc:mysql://localhost/user";
	private String user="root";
	private String pwd="";
	private ResultSet result;

	public void verbinden(){
		try
	    {
	      Class.forName("com.mysql.jdbc.Driver");
	    }
	    catch ( ClassNotFoundException e )
	    {
	      System.err.println( "Keine Treiber-Klasse!" );
	      return;
	    }
	    try {
			con=DriverManager.getConnection(treiber,user,pwd);
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	}
	
	public ResultSet auslesen(String sql){
		
		try {
			result=stm.executeQuery(sql);
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
		return result;
	}
	
	
	
}

Beim Ausführen erhalte ich folgende Meldung

• 
  Exception in thread "main" java.lang.NullPointerException
  at Datenbank.auslesen(Datenbank.java:33)
  at Ausgaben.main(Ausgaben.java:15)

Wer kann mir hier weiterhelfen?

 

[foobar]

Dein Statement ist null, ist doch klar daß das knallt.
http://java.sun.com/j2se/1.3/docs/api/java/sql/Connection.html#createStatement()

public ResultSet auslesen(String sql){
      stmt = con.createStatement();
      try {
         result=smt.executeQuery(sql);
      } catch (SQLException e) {
         // TODO Auto-generated catch block
         e.printStackTrace();
      }
      return result;
   }

 

[SnooP]

du hast von der Klasse Statement kein Objekt erstellt... stm ist demnach null ... wenn man darauf ne Methode aufrufen will: stm.executeQuery sprich null.executeQuery geht das nicht, weil null numal nix mit executeQuery kann - dies führt zu einer allseits beliebten NullPointerException.
Sprich du musst vorher dafür sorgen, dass nen Statement Objekt erstellt wird (new Statement...) oder vielmehr createStatement Methode von Connection aufrufen...

edit: mist - zu spät

 

[alaska94]

Jetzt gibt er mir die Daten leider nicht aus.
Meine meine Klassen wie folgt umgeändert.

import java.sql.*;

public class Datenbank {

	private Connection con;
	private Statement stm;
	private String treiber="jdbc:mysql://localhost/user";
	private String user="root";
	private String pwd="";
	private ResultSet result;

	public void verbinden(){
		try
	    {
	      Class.forName("com.mysql.jdbc.Driver");
	      
	    }
	    catch ( ClassNotFoundException e )
	    {
	      System.err.println( "Keine Treiber-Klasse!" );
	      return;
	    }
	    try {
	    	con=DriverManager.getConnection(treiber,user,pwd);
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
			System.err.println(e.getSQLState());
			System.err.println(e.getErrorCode());
		}
	}
	
	public ResultSet auslesen(String sql) {
	      try {
			stm = con.createStatement();
		} catch (SQLException e1) {
			// TODO Auto-generated catch block
			e1.printStackTrace();
			System.err.println(e1.getSQLState());
		    System.err.println(e1.getErrorCode());
		}
	      try {
	         result=stm.executeQuery(sql);
	      } catch (SQLException e) {
	         // TODO Auto-generated catch block
	         e.printStackTrace();
	         System.err.println(e.getSQLState());
			 System.err.println(e.getErrorCode());
	      }
	     
	     return result;
	    } 
	
	
	
}

import java.sql.ResultSet;
import java.sql.SQLException;

public class Ausgaben {

	/**
	 * @param args
	 * @throws SQLException 
	 */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Datenbank db=new Datenbank();
		String abfrage="Select * from benutzer";
		ResultSet result;
		db.verbinden();
		result=db.auslesen(abfrage);
		try {
			System.out.println(result.getString(2));
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
			System.err.println("Sqlstate= "+e.getSQLState());
			System.err.println(e.getErrorCode());
		}
		try {
			result.close();
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	}

}

Folgende Fehlermeldung erhalte ich.

java.sql.SQLException: Before start of result set
at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:910)
at com.mysql.jdbc.ResultSet.checkRowPos(ResultSet.java:692)
at com.mysql.jdbc.ResultSet.getStringInternal(ResultSet.java:5487)
at com.mysql.jdbc.ResultSet.getString(ResultSet.java:5431)
at Ausgaben.main(Ausgaben.java:18)
S1000
0
Was habe ich nun schon wieder falsch gemacht :-(

 

[RaoulDuke]

Ein next() auf das ResultSet musst du noch machen damit du auf dem ersten Datensatz stehst.

Sagt die SQLException ja auch: "Before start of result set"

Mach es so:

if (result.next()){
    System.out.println(result.getString(2)); 
    }
else{
    System.out.println("Keine Daten gefunden"); 
}

Oder wenn du mehr als nur die erste Zeile willst:

while(result.next()){
    System.out.println("Feld1: "+result.getString(1)); 
    System.out.println("Feld2: "+result.getString(2)); 
}

Das gibt dann alle gefundenen Zeilen aus. Bzw jeweils deren ersten beiden Felder.

 

[alaska94]

Danke dir, ich hätte mich doof gesucht. Hier in meinem Javabuch steht das nicht drin.

 

[RaoulDuke]

Siehe: www.galileocomputing.de/openbook/javainsel5/javainsel20_007.htm

Im Abschnitt "20.8.2 Ergebnisse einer Abfrage in ResultSet"

Ein Aufruf der next()-Methode von ResultSet setzt den internen Cursor auf die erste Zeile der geladenen Ergebnisse.

 

[RaoulDuke]

Ausserdem nutzt dir folgendes nur begrenzt:

try
       {
         Class.forName("com.mysql.jdbc.Driver");
        
       }
       catch ( ClassNotFoundException e )
       {
         System.err.println( "Keine Treiber-Klasse!" );
         return;
       }

Da du nirgendwo prüfst ob die db.verbinden() funktioniert hat. Das bringt dir eigentlich nur das du eine Ausgabe bekommst, und dann hinterher auf eine weitere Exception läufst wenn du auslesen() aufrufst. Die wird dann zwar auch abgefangen, aber dann ist dein ResultSet nicht definiert und du bekommst beim result.next() eine NullPointerException.