Exception

[evil]

Hallo ihr lieben!

Ich habe eine Frage zu Exception und zwar, soll in der Kommandozeile eine int Zahl übergeben werden, dann gibt es 3 Möglichkeiten (Richtige Eingabe, Zahl zu klein, keine Eingabe). 2 catch-Anweisungen habe ich hinbekommen, nun weiß ich aber nicht, was für eine Exception ich nutzen muss, damit ich die richtige Ausgabe bekomme, wenn KEINE Eingabe gemacht wurde. Google und Su-fu haben mir auch nicht weiter geholfen.

Danke im Vorraus

 

[Schumi]

Du kannst doch den gleichen Exceptiontyp benutzen und den Fehlertext differneziert setzen. Oder soll sich das anschließende Verhalten auch Abhängig des Fehlers ändern?

 

[Sergeant_Pepper]

Hi,
es wäre erstens hilfreich, wenn du den vorhandenen Java-Code hier posten würdest (mit [noparse]

[/noparse] package ... usw. [noparse]

[/noparse]).

Zweitens vorab: du kannst eine eigene Exception anlegen (die von einer der Klassen Exception oder RuntimeException erbt), und diese dann mit throw in der Methode werfen, in der du auch die bisherigen Exceptions wirfst.

 

[evil]

Es soll je nach Typ eine andere Fehlermeldung ausgegeben werden.
Ich hatte es bisher so:

catch (EOFException e) {
      System.out.println("Kommandozeilenparameter fehlt");
      System.out.println("Korrekter Aufruf: C <zahl>");
    }
    catch (NumberFormatException e) {
      System.out.println("Kommandozeilenparameter muss eine Zahl sein");
      System.out.println("Korrekter Aufruf: C <zahl>");
    
    }
    catch (Exception e) {
      System.out.print("Die Zahl muss groesser sein");
    }

Aber das schent nicht ganz richtig zu sein, geb ich nämlich nichts ein, wird "Die Zahl muss groesser sein" ausgegeben.

 

[Sergeant_Pepper]

zeig bitte mal den gesamten Code

 

[evil]

import java.io.*;

class C {
  public static void main(String[] args) throws IOException{
  BufferedOutputStream bos = new BufferedOutputStream(
                               new FileOutputStream("a.dat"));
    try {    	   
      byte[] b = new byte[Integer.parseInt(args[0])];
      for (int i = 0; i < b.length; ++i) {
    	b[i] = (byte) (Math.random() * 10);
      }
      bos.write(b, 5, 20);
    }
    catch (EOFException e) {
      System.out.println("Kommandozeilenparameter fehlt");
      System.out.println("Korrekter Aufruf: C <zahl>");
    }
    catch (NumberFormatException e) {
      System.out.println("Kommandozeilenparameter muss eine Zahl sein");
      System.out.println("Korrekter Aufruf: C <zahl>");
    
    }
    catch (Exception e) {
      System.out.print("Die Zahl muss groesser sein");
    }
 
    	    
    bos.close();
  }
}

 

[Sergeant_Pepper]

Ich habe eine Frage zu Exception und zwar, soll in der Kommandozeile eine int Zahl übergeben werden, dann gibt es 3 Möglichkeiten (Richtige Eingabe, Zahl zu klein, keine Eingabe).

vielleicht stehe ich gerade auf dem Schlauch, aber ich verstehe nicht, was dein Programm mit dieser Beschreibung zu tun hat.

Also: du legst ein [c]byte[][/c] an, dessen Größe vom Komandozeilen-Parameter abhängt. OK.
Damit das Sinn macht sollte die Zahl schon größer als 0 sein. Wo wird das geprüft? Nirgends ...

 

[evil]

Mh klingt logisch, aber leider weiß ich nicht so recht, wo ich die Prüfung machen soll.. Weil in der EOFException hat das ja wenig Sinn. Und davor wird das ja für alle Anweisungen geprüft

Das Programm war teilweise vorgegeben, man soll halt verschiedene Eingaben in die Kommandozeile machen und die verschiedenen Fehlermeldungen ausgeben (Übung für Exceptions --> Java Anfänger)

 

[Sergeant_Pepper]

Du willst also prüfen, ob die Eingabe größer ist als 25? Dann tu das doch ...

Statt
[c]byte[] b = new byte[Integer.parseInt(args[0])];[/c]
schreibst du

int groesse=Integer.parseInt(args[0]);
if (groesse<25)
{
	System.out.println("zu klein");
        System.exit(1);
}
byte[] b = new byte[groesse];

Und an Stelle von [c]System.exit(1);[/c] erzeugst du eine neue Exception

EDIT: du hast die EOFException nicht richtig verstanden. Jedenfalls passen die Sysout-Meldungen inhaltlich nicht zu dieser Exception.

 

[evil]

byte[] b = new byte[Integer.parseInt(args[0])];

leider ist gerade diese Zeile vorgegeben. Daher bringt mir das nicht viel :bahnhof:

Kann ich da irgendwie sagen

if (b[i] < 0) { ... }

 

[Sergeant_Pepper]

byte[] b = new byte[Integer.parseInt(args[0])];

leider ist gerade diese Zeile vorgegeben. Daher bringt mir das nicht viel :bahnhof:

bist du sicher dass die so bleiben MUSS ?

Wenn ja, dann schreibst du eben:

if (b.length<25)

Kann ich da irgendwie sagen

if (b[i] < 0) { ... }

Damit prüfst du doch nur den Inhalt eines einzelnen Elements und nicht die Größe des byte[]

 

[Sergeant_Pepper]

Vielleicht mal etwas allgemeiner: bevor du Exceptions erzeugen kannst, abhängig von bestimmten Zuständen, musst du verstanden haben, was das Programm macht.

Weißt du, was diese Zeilen machen sollen?

byte[] b = new byte[Integer.parseInt(args[0])];
for (int i = 0; i < b.length; ++i)
{
        b[i] = (byte) (Math.random() * 10);
}
bos.write(b, 5, 20);

 

[evil]

Achso, stimmt.

Jetzt bekomm ich aber eine Fehlermeldung. Ich habe nun die if-Schleife so gesetzt:

catch (EOFException e) {
      if (b.length<25) {
        System.out.println("Kommandozeilenparameter fehlt");
    	System.out.println("Korrekter Aufruf: C <zahl>");
      }
    }

Und bekomme die Fehlermeldung: Exception in thread “main” java.lang.ArrayIndexOutOfBoundsExceptions: 0 at C.main(C.java:8)

Sorry, wenn es doofe Fragen sind.. Aber aus Fehlern lernt man


Dort wird doch ein byte erzeugt und als int-Zahl dargestellt, danach als Zufallszahl in eine Datei geschrieben?!

 

[Sergeant_Pepper]

Ich habe nun die if-Schleife so gesetzt

ein "if" ist keine Schleife. "for" , "while", "do ... while" sind Schleifen.

Die Prüfung [c]if (b.length<25)[/c] solltest du im try-Block machen, nicht im catch.

Hast du verstanden, was die Anweisungen im try-Block machen sollen?

 

[evil]

Da bekomm ich doch noch mehr Fehlermeldungen

Im try-Block wird doch die Anweisung ausgeführt und der Fehler wird dann an die "passende" catch-Anweisung zugewiesen. Also hier wird die Zahl "verarbeitet" und an die catch-Anweisung übergeben

 

[Sergeant_Pepper]

ich habe keine Ahnung, in welcher Weise du die Aufgabe lösen sollst...

Sollst du eine eigene Exception werfen, wenn die eingebene Zahl kleiner als 25 ist?

 

[evil]

Achso, ich dachte das wäre deutlich genug gewesen.

import java.io.*;

class A {

  public static void main(String[] args) {
    BufferedOutputStream bos = new BufferedOutputStream("a.dat");
    byte[] b = new byte[Integer.parseInt(args[0])];
    for (int i = 0; i < b.length; ++i) {
      b[i] = (byte) (Math.random() * 10);
    
    bos.write(b, 5, 20);
    }
    bos.close();
  }

}

Dieser Code ist vorgegeben. nun soll man diese Klasse so korrigieren, dass sie fehlerfrei läuft und bei verschiedenen Eingaben verschiedene Fehlermeldungen ausgegeben werden.

Zb: %java C
Kommandozeilenparameter fehlt

%java C ahf
Kommandozeilenparameter muss eine Zahl sein

NumberFormatException e

habe ich zum Beispiel gefunden (in der Java Dokumentation) und da dachte ich, es gibt sowas auch, wenn man keine Parameter in der kommandozeile übergibt

 

[Gast2]

Wie wäre es so:

import java.io.*;

class A {

	public static void main(String[] args) {
		BufferedOutputStream bos;
		try {
			bos = new BufferedOutputStream(new FileOutputStream("a.dat"));
			if (args.length != 1) {
				throw new IllegalArgumentException("Not enough/too many arguments given!");
			}
			try {
				int i = Integer.parseInt(args[0]);
			} catch (NumberFormatException e) {
				throw new IllegalArgumentException("Argument is not numeric!");
			}

			byte[] b = new byte[Integer.parseInt(args[0])];
			for (int i = 0; i < b.length; ++i) {
				b[i] = (byte) (Math.random() * 10);
				bos.write(b, 5, 20);
			}
			bos.close();

		} catch (FileNotFoundException e) {
			System.err.println("Cannot write to a.dat: " + e.getMessage());
		} catch (IOException e) {
			System.err.println("Failed to write to a.dat: " + e.getMessage());
		} catch (IllegalArgumentException e) {
			System.err.println("Illegal Usage: " + e.getMessage());
			System.err.println("This programm need to be called with: java A <Integer>");
		}
	}

}

Du kannst noch 25 andere Prüfungen machen und IllegalArgumentExceptions schmeissen die unten ausgegeben werden, z.B.

if(Integer.parseInt(args[0])< 25){
    throw new IllegalArgumentException("Argument need to be larger than 25!");
}

 

[Sergeant_Pepper]

Fällt mir gerade ein...
füge diesen catch-Block hinzu

catch (ArrayIndexOutOfBoundsException e)
		{
			System.out.println("keine Parameter");
		}

der fängt den Fehler ab, der durch [c]Integer.parseInt(args[0])[/c] ausgelöst wird, wenn kein Paramter angegeben wird.

Stand ja auch schon in einem Posting von dir ... hätte eher drauf kommen müssen.

Bitte lies dir dazu auch die Javadoc durch, und auch gleich die für EOFException

 

[Gelöschtes Mitglied 16929]

Hier eine Lösung die funktionieren sollte ^^ viel spaß wenn du fragen hast dann meld dich!

import java.io.*;
 
public class A {
 
  public static void main(String[] args) {
	try {
		BufferedOutputStream bos = new BufferedOutputStream(new FileOutputStream("a.dat"));
		byte[] b = new byte[Integer.parseInt(args[0])];
	    if(b.length < 25)
	    	throw new IllegalArgumentException("The parameter must be greater than 24.");
	    for (int i = 0; i < b.length; i++) {
		 b[i] = (byte) (Math.random() * 10);
		 bos.write(b, 5, 20);
	    }
	    bos.close();
	} catch (NumberFormatException e) {
		System.out.println("Parameter must be a number.");
	} catch (ArrayIndexOutOfBoundsException e) {
		System.out.println("You must enter a parameter.");
	} catch (FileNotFoundException e) {
		System.out.println("Unable to create file."); 
	} catch (IOException e) {
		System.out.println("Unable to write file.");
	} catch (IllegalArgumentException e) {
		System.out.println(e.getMessage());
	}
  }
 
}

 

[Suinos]

wenn man keine Parameter in der kommandozeile übergibt

Und was hindert dich daran, solche Dinge einfach in einer if-Abfrage zu prüfen?

if (args.lenght == 0)
{
    System.out.println(":(");
}

else
{
    System.out.println(":)");
}

Gerade solche [c]IndexOutOfBoundsException[/c]s sind eigentlich dafür da, um dem Entwickler zu sagen, dass er was falsch macht.

 

[evil]

Hallo hemeroc,

danke für die Lösung Aber ein Frage hätte ich noch, woher kommt denn die Methode

System.out.println(e.getMessage())

? Die ist ja nirgends definiert, wenn ich das richtig sehe ???:L

Und @ Suinos, ich hatte das ja auch so, nur leider fehlte mir die else-Anweisung, weil ich noch eine weitere (unnötige) exception eingebaut hatte.

Vielen Dank an alle für eure Hilfe

 

[Landei]

Du gibst der Exception ja im catch einen Namen, z.B. e oder meineBlaException. Innerhalb des catch-Blocks kannst du sie dann damit ansprechen:

try {
 ...
} catch (BlaException meineBlaException) {
   System.out.println(meineBlaException.getMessage());
}

 

[Gast2]

Hallo hemeroc,

danke für die Lösung Aber ein Frage hätte ich noch, woher kommt denn die Methode

System.out.println(e.getMessage())

? Die ist ja nirgends definiert, wenn ich das richtig sehe ???:L

Und @ Suinos, ich hatte das ja auch so, nur leider fehlte mir die else-Anweisung, weil ich noch eine weitere (unnötige) exception eingebaut hatte.

Vielen Dank an alle für eure Hilfe

Hast du dir meine Lösung angesehn? Da übergebe ich die Fehlermeldung in den Constructor - die bekommst du dann mit e.getMessage() wenn du die Exception im Catchblock gefangen hast.

 

[evil]

Ahh, vielen Dank, habs verstanden. :toll: