Verschachtelte Exceptions - Übersicht verbessern

[SirBaros]

Hallo,
ich habe ein Programmcode indem viel verschachtelte Exceptions drinnen sind.
Hat jemand eine IDee wie ich die Übersicht verbessern kann. ALso die verschachtelung vereinfache?

public static Properties loadConfigFromFile(FileWriter logFile, String configFile){
		
		Properties properties = null;
		
		try{
			properties = new Properties();
			InputStream configstream = new FileInputStream(configFile);
			properties.load(configstream);
			
		}catch(Exception e){
			
			try {
				logFile.write(new Date()+" Error: Loading Configuration\r\n");
				logFile.flush();
				
			} catch (Exception ex) {
				System.out.println("Error: Writing ErrorlogOpportunity.txt");
			}
		}
		return properties;
	}

 

[SlaterB]

das catch fürs Log brauchst du doch sicher ständig, auch das Datum, das flush() usw.,
schreibe eine Methode log(xy);, statisch oder an einem Logger-Objekt,
dann ist das alles hier denkbar kurz und für tausend Fälle fertig

bleibt nur noch ein catch mit Log-Aufruf drin, besser gehts kaum

allgemein kann man überlegen, ob an dieser Stelle Exception abgefangen oder eher nach oben weitergeleitet/ gar nicht erst gefangen wird,
damit der Aufrufer dieser Methode abbricht und nicht mit leeren Properties weiterarbeitet

 

[KSG9|sebastian]

Zum einen wäre es sinnvoller auf eine entsprechende Log-API umzustellen, z.B. log4j.

Zum zweiten verschluckst du die eigentliche Exception (erster catch-Block).

Zum dritten könntest du deinen eigenen Logger auslagern, um nicht in jedem catch-Block Exception-Handling, Dateiverarbeitung u.s.w. haben