Du verwendest einen veralteten Browser. Es ist möglich, dass diese oder andere Websites nicht korrekt angezeigt werden. Du solltest ein Upgrade durchführen oder ein alternativer Browser verwenden.
mein Problem bezieht sich auf das Thema Exceptions.
Mein Programm wertet Datenbankeinträge aus und setzt dementsprechend eine Ampel auf Grün oder Rot.
Im Fall eines Verbindungsfehlers zur Datenbank wird eine SQL-Exception geworfen und die Ampel blinkt in Gelb. Soweit so gut, funktioniert wunderbar bis hier.
Allerdings weiß ich nicht, wie ich wieder den normalen Ablauf starte. Verzweifeltes Suchen in der Java-Insel und in Foren konnte mir leider auch nicht weiterhelfen....
Hier der Code:
Java:
public void run() {
while(!interrupted()) {
try {
maxWarenkorb = getMaxWarenkorb(); //maximalen Warenkorb aktualisieren
query = getQuery(); //SQL-Abfrage aktualisieren
db_url = getDbUrl(); //DB-Url aktualisieren
db_user = getDbUser(); //DB-Username aktualisieren
db_pw = getDbPassword(); //DB-Passwort aktualisieren
Connection con = DriverManager.getConnection(db_url,db_user,db_pw); //Verbindung zur DB herstellen
Statement stmt = con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_UPDATABLE); //SQL-Statement erstellen
ResultSet rs = stmt.executeQuery(query); //Ergebnisse der Abfrage in ein ResultSet schreiben
count = 0;
//Zählt die Anzahl der Elemente(Fahrzeuge) im ResultSet
while(rs.next()) {
String s = rs.getString(1);
System.out.println(s);
count++;
}
System.out.println("Fahrzeuge: " + count);
stmt.close(); //Statement schließen
con.close(); //Verbindung zur DB schließen
//Setzt den Zustand der Ampel von Grün auf Rot, sobald die maximale Kapazität des Warenkorbs überschritten wird
/*
if(count <= maxWarenkorb) {
zustand = Zustand.GRÜN;
}
else {
zustand = Zustand.ROT;
}
*/
zustand = getZustand();
adminAmpel.setFüllstand(count); //Füllstand im Ampelobjekt aktualisieren
adminAmpel.setZustand(zustand.toString()); //Zustand im Ampelobjekt aktualisieren
Thread.sleep(10000); //Wartezeit bis zur nächsten Datenbankabfrage
}
//Exceptions abfangen
catch(SQLException e) {
System.out.println("Fehler bei der SQL-Abfrage!");
zustand = Zustand.STÖRUNG;
while(zustand == Zustand.STÖRUNG) {
adminAmpel.setZustand(zustand.toString());
}
zustand = getZustand();
continue;
//break;
}
catch(Exception e ) {
e.printStackTrace();
break;
}
}
}
hast Du es eigentlich schon richtig gemacht. Nach dem Behandeln der SQLException setzt Du die while(!interrupted())-Schleife einfach mit continue fort. Ich glaube aber, dass Du dir mit diesen Zeilen:
eventuell eine Endlosschleife gebaut hast. Die Variable zustand wird nämlich im Body der Schleife nie geändert. D.h. die Bedingung "(zustand == Zustand.STÖRUNG)" ist immer wahr und die while Schleife läuft endlos.
aktuell schickst du deinen Thread in eine endlosschleife. Setz doch einfach den zustand auf Zustand.STOERUNG (Umlaute sollten vermieden werden) und versuch die Datenbankabfrage in 10 Sekunden nochmal.
Das macht natürlich nur Sinn wenn ne chance besteht dass das beim nächsten mal klappt.
while(zustand == Zustand.STÖRUNG) {
adminAmpel.setZustand(zustand.toString());
}
zustand = getZustand(); // Muss das nicht evtl. mit in den Schleifenblock REIN?
Ist Dir da die letzte Zeile evtl. einfach nur aus dem Klammernblock der while-Schleife gerutscht? Sieht jedenfalls fast so aus.
Das meiste wurde ja schon erwähnt aber du solltest auch mit finally block arbeiten um offene Verbindungen zu schliessen. Sprich Datenbankverbindungen oder anderem.
