Eine Exception bei false

[frau-u]

Hi,

Ich habe hier etwas programmiert, was bestimmte Objekte mit Hilfe von Konstruktoren erstellt. Dabei wird ein Wert mit einer Hilfsmethode überprüft, die einen Boolean-Wert zurückgibt.
Dabei sollte, wenn der Wert falsch ist, eine selbstgeschriebene Exception greifen, die dann eine Ausgabe a la "Wert ungültig" auf den Bildschirm bringt, aber ich habe keine Ahnung, wie ich die einbringen soll.

Die Methode ist die folgende:

static public boolean testISBN(String isbn) {
		boolean isISBN = false;
		boolean lengthOK = false;
		if(isbn.length() == 13) {
			isbn = isbn.replaceAll(" ", "");
			if(isbn.length() ==10)
				lengthOK = true;
			else if(isbn.length() == 13)
			{
				isbn = isbn.replaceAll("-","");
				if(isbn.length() ==10)
					lengthOK = true;
			}
		} 
        		
		if(lengthOK == true){
			int zahl = Integer.parseInt(isbn.substring(0,1)) *10;
			zahl += Integer.parseInt(isbn.substring(1,2)) *9; 
			zahl += Integer.parseInt(isbn.substring(2,3)) *8; 
			zahl += Integer.parseInt(isbn.substring(3,4)) *7; 
			zahl += Integer.parseInt(isbn.substring(4,5)) *6; 
			zahl += Integer.parseInt(isbn.substring(5,6)) *5; 
			zahl += Integer.parseInt(isbn.substring(6,7)) *4; 
			zahl += Integer.parseInt(isbn.substring(7,8)) *3; 
			zahl += Integer.parseInt(isbn.substring(8,9)) *2; 
			if(isbn.endsWith("X"))
				zahl += 10;
			else
				zahl += Integer.parseInt(isbn.substring(9,10)) *1;
			if(zahl % 11 ==0)
				isISBN = true;
		}    
        
		return isISBN;
	}

Wenn nun lengthOK oder isISBN false sind, soll es die Exception werfen.
Ich dachte, dass ich das vielleicht mit:

if (isISBN = false) {
            throw new FalscheIsbnException("falscher Wert");
        }

Doch alles, was ich bisher versucht habe, funktionierte leider nicht.
Kann mir jemand einen (oder ein paar :lol Hinweis(e) geben, wie ich das am besten mache?
Danke.

 

[akira]

if (isISBN = false) {
            throw new FalscheIsbnException("falscher Wert");
        }

Wenn du Werte auf Gleichheit überprüfst, must Du das mit == machen, ein einfaches = ist eine Zuweisung.
Da es sich sowieso um einen boolean handelt, kannst Du auch schreiben:

if(!isISBN) {
    throw new DeineException();
}

 

[foobar]

if (isISBN = false) {

Das ist eine Zuweisung und keine Abfrage ;-)

Du mußt den == Vergleichsoperator verwenden:

if (false == isISBN) {

[Edit: Mist zu spät :cry:]

 

[frau-u]

Ah, super, das sieht doch schon gleich viel besser aus.

Allerdings habe ich noch das Problem, dass damit die Exception geworfen wird und das Programm damit abbricht, ich würde es aber gerne weiterführen.
Ich kenne eigentlich nur die Möglichkeit, die try-catch-Anweisung in eine Schleife zu schreiben, aber wenn ich nur die Exception aufgrund des falschen Boolean-Wertes werfen will, geht das in die Hose, denn nach

if(!isISBN) {
    throw new FalscheIsbnException();
}

wird es ja gleich beendet.

Habt ihr noch einen guten Tipp für mich?
Danke.

 

[akira]

Naja, ist die Frage, ob eine Exception überhaupt Sinn macht.

Eine Exception ist per Definition eine "Ausnahme", in der Regel ein Fehlerfall.

Wenn man Exceptions wirft, kann man auf diese reagieren indem man sie fängt, d.h. das entsprechende Codestück in einen try - catch - Block packt. In der catch-Klausel kann man dann den Fehler so gut es geht behandeln, das Programm endet dann nicht unbedingt.

 

[frau-u]

Naja, ist die Frage, ob eine Exception überhaupt Sinn macht.

Eine Exception ist per Definition eine "Ausnahme", in der Regel ein Fehlerfall.

Wenn man Exceptions wirft, kann man auf diese reagieren indem man sie fängt, d.h. das entsprechende Codestück in einen try - catch - Block packt. In der catch-Klausel kann man dann den Fehler so gut es geht behandeln, das Programm endet dann nicht unbedingt.

Aber es sind doch meine Hausaufgaben.... ;-)
Ne, im Ernst: es ist wirklich eine Übungsaufgabe von einer Uni. Zwar keine Hausaufgabe, aber dort zum üben erstellt worden.

So ganz habe ich den Sinn auch noch nicht verstanden - man könnte ja auch eine ganz einfach Abfrage schreiben und dann schlicht per System.out.print ausgeben. Mal schauen, ob ich vielleicht irgendwo eine Lösung bekommen kann, wie die sich das vorstellen.
Danke zumindest.