NullPointer bei if anweisung

[Guest]

Hallo,
ich habe eine Klasse in NetBeans, die u.a. Temperaturdaten (temp1) an eine MySQL Datenbank sendet. Ist kein Temperatursensor vorhanden, so wird 0 angezeigt. Dies führt zu Problemen, da 0 sowohl "kein Sensor vorhanden" als auch "0°C" heißen könnte. Deshalb möchte ich diesen Fall mit einer if-Anweisung ausschließen, indem geprüft wird, ob ein Sensor vorhanden ist oder nicht und entsprechend 0 oder "null" schreiben. Dabei bekomme ich jedoch eine NullPointerException:

mainLoop(): java.lang.NullPointerException
java.lang.NullPointerException
        at etws.MySQL.savedata(MySQL.java:19)
        at etws.ETWS.mainLoop(ETWS.java:422)
        at etws.ETWS.main(ETWS.java:560)

package etws;
import java.sql.*;



public class MySQL{

private static Prefs prefs;
private static String temp1, temp2, taupunkt, wrichtung, wgeschw, baro, hum, regen;

public static void savedata( WeatherCruncher cruncher )
    {
    
    temp1 = "null";
    if (prefs.isTemp1DeviceAvailable()) {temp1 = cruncher.temp1Str;}
    
    System.out.println("Schreibe Daten in MySQL Datenbank");

    String treiber=null, DbUrl=null ;
    treiber = "com.mysql.jdbc.Driver" ;
    DbUrl = "jdbc:mysql://localhost:3306/wetterstation" ;

     try {

         //*** Treiber laden ***********************************
         Class.forName( treiber ).newInstance();
          //*** Verbindung aufnehmen:    ************************
          //*** Der User peter mit Kennwort mysql möcht was wissen
         Connection cn = DriverManager.getConnection( DbUrl, "root", "root" );
         PreparedStatement pstmt = cn.prepareStatement("INSERT INTO wetterdaten " +
                 "(datum, temp1, temp2, taupunkt, wrichtung, wgeschw, baro, hum, regen)" +
                 "VALUES (now(),"+ temp1 + "," + cruncher.temp2Str+ "," +
                 cruncher.dpStr + "," + cruncher.windDirection + "," +
                 cruncher.windSpdStr + "," + cruncher.pressStr + "," + cruncher.humStr +
                 "," + cruncher.rainStr + ")");

          
         pstmt.executeUpdate();
        cn.close();
        } catch( Exception ex ) {   System.out.println( ex );   }
   }

}

 

[Ebenius]

Die Variable prefs wird nirgends initialisiert. Sie ist null. Und daher kann man darüber auch keine Methoden aufrufen. Das ist natürlich schwer zu verstehen, wenn man selbst alles static schreibt. ;-)

 

[hdi]

Man sollte mal eine Liste erstellen mit den gängigsten Compile-Fehlern und einer Erklärung

cannot find symbol
outoufbounds
null-pointer

... diese Dinge kommen immer und immer wieder. Wär was für den FAQ Bereich!

Kann ja mal jemand machen der sich mit der JVM auskennt und das auch korrekt erklären kann.

 

[Ariol]

Verteilte Systeme?

 

[Guest]

Oh, dachte ich hab das schon ausprobiert. Lösung:

 prefs = ETWS.prefs;

Trotzdem neue Frage:
Ich hab folgende Funktion in der Klasse WeatherStation:

public void initDevices() {
    System.out.println("InitDevices");

    // temperature device
    if (prefs.isTemp1DeviceAvailable()) {
      temp1Device = new OneWireContainer10(adapter, prefs.getTemp1Sensor());

      if (temp1Device == null) {
        Logger.log("No DS1820 Temperature1 Sensor found - Disabling Device");
      }
    } else {
      System.out.println("Invalid or No Temp1 Device Specified");
    }

Wie kann ich nun aus der Klasse MySQL herraus prüfen, ob

temp1Device == null

 

[Ebenius]

Das verstehe ich überhaupt nicht. Die Bedingung in Zeile 9 ist immer false. Ein Konstruktur (Zeile 7) liefert immer einen Wert der nicht null ist.

Nur für Pädanten: Die einzige Ausnahme, dass ein anderer Thread die Variable zwischen Zeile 7 und 9 auf null setzen würde, schließe ich hier mal aus.

 

[Guest]

Sorry
drüber steht noch

private OneWireContainer10 temp1Device = null;

 

[Ebenius]

Denk nochmal darüber nach!

// Hier wird der Variablen temp1Device ein Wert zugewiesen der nie null ist!
temp1Device = new OneWireContainer10(adapter, prefs.getTemp1Sensor());

// Hier wird direkt danach geprüft, ob die Variable nicht vielleicht doch null ist?
if (temp1Device == null) {

Hast Du Angst, dass sich die Maschine ab und an irrt und die erste Zeile vergisst auszuführen? ;-)

 

[hdi]

In der nächsten Version von Java soll es auch eine neue if-Konstruktion geben, die solche lästigen Abfragen ablöst:

int i = 5
if ( i == 5 && word? ){
          ...
}

:bae:

 

[Ebenius]

Ich verschenke ein Semikolon!

 

[0x7F800000]

Hast Du Angst, dass sich die Maschine ab und an irrt und die erste Zeile vergisst auszuführen? ;-)

Java2K!!! :lol:

 

[Guest]

Mal andersrum:
Wenn ich die Meldung "No DS1820 Temperature1 Sensor found - Disabling Device" bekomme, wenn kein Sensor eingesteckt ist, ist das doch der Beweis dafür, dass es eben doch null sein kann?!

 

[Ebenius]

Füge mal vor die "if"-Zeile das da ein:

try {
  temp1Device.toString();
} catch (NullPointerException ex) {
  ex.printStackTrace();
}

Kommt jemals die NullPointerException? Und kommt jetzt noch "No DS1820 ......"?

 

[Guest]

Okay, du hattest recht, die Fehlermeldung die ich gelesen habe hieß nur so ähnlich. Hab ich verwechselt.
Trotzdem komme ich mit meinem Problem nicht weiter.
Wie kann ich denn jetzt aus der MySQL Klasse herraus prüfen, ob temp1Device.isPresent() aus der Klasse WeatherStation -> getTemperature1 wahr ist? Weiß leider auch nicht was ihr für code benötigt, um diese Aufgabe lösen zu können. Deshalb habe ich mal das gesamte Programm angehängt, würde mich freuen wenn sich einer erbarmt. Schwer ist es bestimmt nicht - wenn man weiß wie...
Und bevor jetzt ein "wenn du keine Ahnung hast, warum machst du nicht einfachere Sachen" kommt: Ich muss das für die Schule machen (sozusagen eine selbstauferlegte Pflicht). Das Programm stammt im Original auch nicht von mir.

Also, wäre super wenn sichs mal wer anschaun könnte:
http://uploaded.to/?id=rl1h0a

 

[Guest]

Keiner eine Idee?