Custom Exception ausgeben?

[ZeroQool]

Hallo,

anscheinend habe ich einen gravierenden Denkfehler. Folgender Code:

private void CreateUpdateImportUserInfo(UserInfo pUserInfo) throws Exception
{
  try
  {
     if (pUserInfo.getUsername() == null || pUserInfo.getUsername().equals(""))
				throw new RequiredFieldException("misssing username");
  }
  catch (Exception e)
  {
    e.printStackTrace();
    throw new Exception(e.getMessage());
  }

So sieht meine Custom Exceptionklasse aus:

public class RequiredFieldException extends Exception
{
	/**
	 * Property
	 */

	private static final long serialVersionUID = 8450122375285827321L;

	/**
	 * Constructor
	 */
	public RequiredFieldException()
	{
		super();
	}

	public RequiredFieldException(String message, Throwable cause)
	{
		super(message, cause);
	}

	public RequiredFieldException(String message)
	{
		super(message);
	}

	public RequiredFieldException(Throwable cause)
	{
		super(cause);
	}

	/**
	 * Getter/Setter
	 */
}

Mit Absicht ist der Username nicht gefüllt. Jetzt möchte das ich in meiner SOAP Exception die Message "missing username" erscheint, aber stattdessen kommt

<faultstring>java.lang.NullPointerException</faultstring>

Ich finde einfach nicht den Fehler. Im Debugger sehe ich das im

e.getMessage()

missing username steht

, aber keine Ahnung warum es nicht so ausgegeben wird wie ich es möchte. Bitte um eure Hilfe. Danke

 

[faetzminator]

Keine Ahnung, warum du nachträglich nicht eine [c]RuntimeException[/c] wirfst (oder ggf. [c]e[/c]). Einen Fehler kann ich ebenfalls nicht erkennen. Warum nicht einfach schnell debuggen?

 

[HoaX]

Wenn du dir den Stacktrace der Exception ausgeben lässt/anschaust, dann siehst du in welcher Zeile die Exception geworfen wird. Evtl ist pUserInfo null?

Warum nicht "throw e;" in Zeile 11?

 

[ZeroQool]

Wenn du dir den Stacktrace der Exception ausgeben lässt/anschaust, dann siehst du in welcher Zeile die Exception geworfen wird. Evtl ist pUserInfo null?

Warum nicht "throw e;" in Zeile 11?

pUserInfo ist NULL. Imgrunde versuche ich Felder zu validieren, in dem ich prüfe ob der User im Objekt pUserInfo Username gefüllt hat. Wenn zB LEER oder NULL, dann soll eine Exception geworfen in dem der User erkennen kann, "ah habe den Username vergessen..."

 

[faetzminator]

Dann musst du aber zumindest eine Instanz von [c]UserInfo[/c] übergeben oder [c]pUserInfo[/c] ebenfalls auf [c]null[/c] prüfen.

 

[ZeroQool]

Dann musst du aber zumindest eine Instanz von [c]UserInfo[/c] übergeben oder [c]pUserInfo[/c] ebenfalls auf [c]null[/c] prüfen.

Stimmt, da hast du vollkommend recht, aber hab jetzt glücklicherweise die Situation, dass ich das ganze gerade mit SOAP UI teste und er überall

?

reinschreibt und zum Test habe ich Username so übergeben:

<username></username>

 

[Eldorado]

Mal eine ganz andere Frage, warum überschreibst du eigentlich Exception, wenn du sowieso alles an die super-Klasse weitergibst? Um das Problem nochmal anders zu beschreiben: du rufst die Methode

getUsername()

von einem nicht vorhandenen Objekt(leere Variable) auf und so wird natürlich eine NullpointerException geworfen...(Wenn dein SOAP UI das egal ist, ok, kenne ich mich nicht aus)
mfg Eldorado

 

[Ebenius]

Mal eine ganz andere Frage, warum überschreibst du eigentlich Exception, wenn du sowieso alles an die super-Klasse weitergibst?

Find ich bei Exceptions normal, da der Exception-Typ ja Informationsbestandteil ist. Allerdings sollte im Konstruktor "CreateUpdateImportUserInfo" die [c]throws[/c]-Klausel auch den eigenen Typ mit aufführen, da sie eine besondere Bedeutung hat.

Ebenius

 

[Eldorado]

Find ich bei Exceptions normal, da der Exception-Typ ja Informationsbestandteil ist.

stimmt, hab grad nicht dran gedacht, dass der Exception-Typ natürlich von dem Klassennamen abhängt. Außerdem ist es ja noch wichtig um verschiedene Fehler, verschieden Behandeln zu können.
mgf Eldorado