Exception wird nicht aufgefangen

[7bkahnt]

Hallo,
ich habe eine webbasierte Anwendung, welche ein Tabelle aus einer DB ausliest und dem Benutzer anzeigt. Dieser kann dann Datensätze z.B. bearbeiten.
Er klickt auf bearbeiten und bekommt den gewählten DS zu Gesicht. Dann nimmt er Änderungen vor und klickt auf Ändern. Somit wird der DS geupdatet.

Nun zu meinem Problem.
Ich möchte dem Problem, dass zwei oder mehrere Nutzer den gleichen DS bearbeiten können, aus dem Weg gehen.

Ersteinmal der Quellcode:

CrudAction.java:

public String edit()
	{
		 Integer userid=(Integer)session.get("userObject");   
		 HttpServletRequest request = (HttpServletRequest) ActionContext.getContext().get(ServletActionContext.HTTP_REQUEST);
		 crud = crudDAO.listCrudById(Long.parseLong(request.getParameter("id")));
		try {
			lockDAO.lockCrud(Long.parseLong(request.getParameter("id")),userid); 
		} catch (Exception e) {

			System.out.println("FEEHLER");
			
		}
		   
		return SUCCESS;
	}

lockdao ist bei mir so deklariert:

private LockDAO lockDAO = new LockDAOImpl();

während LockDAO ein Inteface ist.
Er geht also mit lockDAO.lockCrud() in die Klasse LockDAOImpl und führt dort die lockCrud() aus.



CrudDAOImpl.java:

@Override
	public void lockCrud(Long auswahlid,Integer userid) throws Exception {
		Lock lock = new Lock();
		System.out.println("lockCrud()");
		Calendar cal = Calendar.getInstance();
		DateFormat df = DateFormat.getDateTimeInstance();
		String zeit=df.format(cal.getTime());
		System.out.println(zeit);
		
		lock.setId(auswahlid);
		lock.setZeit(zeit);
		lock.setId_sb(userid);
		session.save(lock);
	}

Wenn Nutzer A also Bearbeiten klickt, geht er in die edit()-Methode, welche lockCrud() aufruft.
Dort wird die ID des Nutzers, sowie die ID des ausgewählten Datensatzes in eine Lock-Tabelle geschrieben. Die Spalte "ID", in welche die ID´s der Datensätze kommen ist Unique.
Wenn jetzt also Nutzer B ebenfalls den gleichen DS bearbeiten will, klickt er auf Bearbeiten und geht wiederum in die lockCrud().
Dort versucht er wieder in die Tabelle einen Eintrag zu machen. Da aber die ID des Datensatzes schon vorhanden ist kommt es nun zur Exception.
Diese soll geworfen werden und die edit()-Methode soll die Exception auffangen. Im catch-Block soll dann eine Methode aufgerufen werden, welche dem Nutzer den Hinweis gibt, dass der DS gerade bearbeitet wird.

Aber irgendwie geht er nie in den catch-Block rein und schreibt mir somit auch nie FFEEHHLER aus
Habt ihr vielleicht einen Tipp woran das liegen könnte?.

Der Fehler der in der console ausgeschrieben wird:

Hibernate: insert into Lock_t (ID_SB, ID) values (?, ?)
Error! Please, check your JDBC/JDNI Configurations and Database Server avaliability. 
 Could not commit the Hibernate Transaction: Could not execute JDBC batch update
org.hibernate.exception.ConstraintViolationException: Could not execute JDBC batch update

Caused by: java.sql.BatchUpdateException: ORA-00001: Unique Constraint (SYSTEM.LOCK_T_UK2) verletzt
org.apache.catalina.core.StandardWrapperValve invoke
SCHWERWIEGEND: Servlet.service() for servlet default threw exception
java.lang.IllegalStateException

Ich hatte es auch mit mehreren Exceptions probiert, wie IOException oder auch die IllegalStateException, die ja laut Fehlermeldung (s. oben) sogar geworfen wird.
Aber irgendwie lässt sich nichts abfangen.

 

[SlaterB]

du hast keine weiteren Informationen wie StackTrace?

versuche an höheren Stellen, z.B. direkt im Servlet ein try/catch, falls wirklich ein Servlet beteiligt ist, wie die Fehlermeldung andeutet,
vielleicht tritt die Exception an anderen Aufruf-Stellen auf

ganz schick wäre auch ein Test in einem normalen Java-Programm mit main-Methode, statt komplizierter Web-Umgebung

 

[FArt]

Die Lösung ist einfach: die Methode wirft keine Exception oder wie wird bereits vorher verabeitet.

Tipp: throws Exception ist ganz schlechter Stil, genau so ein catch auf Exception. Hier wäre sogar eine eigene Exception (z.B. AlreadyLockedException) sinnvoll, die noch spezielle Informationen enthält, wie z.B. der Datensatz (ID) um den es geht usw.
Eigentlich wäre es noch sinnvoller, gar keine Exception zu werfen, sondern einen Status zurückzugeben, denn dass ein Datensatz gesperrt ist, ist keine Ausnahme, sondern ein möglicher Status des Locks.

 

[SlaterB]

ach ja, jetzt fällt mir wieder ein, dass das SQL, was hier doch zur Exception führt, nicht unbedingt beim save()-Aufruf ausgeführt wird,
sondern erst verzögert, z.B. bei flush() + commit() auf die Session oder auch bei einer nächsten DB-Abfrage,

vielleicht kannst du mit flush() dafür sorgen, dass die BatchUpdateException innerhalb von lockCrud() auftritt

 

[7bkahnt]

ich habe es mal probiert
CrudDAOImpl.java:

@Override
	public boolean lockCrud(Integer auswahlid,Integer userid) throws Exception,BatchUpdateException,SQLException
	{
		System.out.println("lockCrud()");
		
		Lock lock = new Lock();
		boolean checkeintrag=false;
		
		Calendar cal = Calendar.getInstance();
		DateFormat df = DateFormat.getDateTimeInstance();
		String zeit=df.format(cal.getTime());
		
		
			String statement = "select lock.id_sb from Lock lock where lock.id=?";
		    List list = (List) session.createQuery(statement).setInteger(0, auswahlid).list();
		
		 if(list.size()==0){
			lock.setId(auswahlid);
			lock.setZeit(zeit);
			lock.setId_sb(userid);
			session.flush();
			session.save(lock);	
			checkeintrag=true;
		}
		else{
			buser=(Integer) list.get(0);
			checkeintrag=false;
		}
		return checkeintrag;
	}

CrudAction.java:

public String edit()
	{
		Integer userid=(Integer)session.get("userObject");   
		HttpServletRequest request = (HttpServletRequest) ActionContext.getContext().get(ServletActionContext.HTTP_REQUEST);
		Integer auswahlid=Integer.parseInt(request.getParameter("id"));
		session.put("auswahlid", auswahlid);
		
		//gewählten DS holen
		crud = crudDAO.listCrudById(Long.parseLong(request.getParameter("id")));
		
		//DS sperren
		boolean checkeintrag=false;
		try {
			checkeintrag=lockDAO.lockCrud(Integer.parseInt(request.getParameter("id")),userid); 
		} catch (BatchUpdateException e) {
			System.out.println("FEHLER durch BatchUpdateException");
			return "Fehler";
			}
		catch (SQLException e) {
			System.out.println("Fehler durch SQLException");
                        return "Fehler";
		}
		catch (Exception e) {
			System.out.println("Feher durch Exception");
                        return "Fehler";
		}
		
		//wenn DS nicht gelockt
		if(checkeintrag==true){
			return SUCCESS;
		}
		else{
			return "noedit";
		}
	}

Dasselbe in Grün. Irgendwie wird durch die Methode lockCrud keine Exception ausgelöst. Warum versteh ich ehrlich gesagt nicht.
Ich hatte bisher mit Exceptions noch nicht soviel am Hut, aber es muss doch wenigstens eine Exception ausgelöst werden od nicht?

Das mit dem Status hab ich ja schon hinbekommen, wenn Nutzer A "bearbeiten" klickt und danach Nutzer B auf den selben DS, dann bekommt er angezeigt, dass er gerade bearbeitet wird.
Das Problem ist eben nur wenn beide gleichzeitig bearbeiten klicken, weil dann wollen beide
insert into lock(....)..... ausführen mit denselben DS-ID´s und die Spalte ist ja unique.
Somit bekomm ich unter anderem einen Fehler ORA-00001: Unique Constraint (SYSTEM.LOCK_T_UK2) verletzt, was ja klar ist.
Nur ich dachte ich kann sowas evtl abfangen, dass der Erste durchkommt und der Zweite dann einen Hinweis bekommt.

 

[SlaterB]

flush() == durchspülen, saubermachen, alles wegschreiben natürlich NACH dem save()

wenn es Transaktionen gibt und jeder für sich arbeitet bis committed wird,
dann kann es immer noch sein, dass die Exception erst sehr spät beim commit stattfindet,

 

[7bkahnt]

Stimmt klar. Super danke!
Jetzt schreibt er mir aus "Fehler durch Exception".
Jetzt gibt es nur noch ein Problem.
Eclipse geht bei mir immer nach dem gleichzeitigen klicken in den Debug Mode.
Somit muss ich immer solange step over klicken, bis er endlich den Fehler catched.
Gibt es da eine Möglichkeit das zu umgehen, dass er nicht in den Debug-Modus geht?
Sorry klingt ein wenig nach Anfängerfragen, aber hatte bisher nicht soviel mit Exceptions und Debuggen am Hut.

grüsse

 

[SlaterB]

wie startest du das Programm denn, Screenshot vom Menüeintrag/ Button den du wählst,
selber links, rechts, oben, unten davon nach normalen Run suchen,

evtl. hast du kleine grüne Kreise, Breakpoints am linken Rand im Java-Editor einer Klasse, die kann man auch wieder entfernen

 

[7bkahnt]

Nee ich hatte keine Breakpoints gesetzt. Bin immer in der MyEclipse Java Enterprise-Umgebung
Ich hab das Projekt einfach nochmal komplett rausgehauen, MyEclipse neugestartet und wieder importiert.
Jetzt funktioniert es wunderbar.
Die Exception wird sofort ausgeschrieben und der zweite Nutzer bekommt sofort den Hinweis, dass der DS bearbeitet wird.
Danke euch für die super Hilfe!

grüsse