Refactoring des FontLoader

Das Öffnen von Resourcen Streams wurde in eine eigene Hilfsklasse ausgelagert. FontLoader nutzt nun die neue Logging API.

src/schule/ngb/zm/util/FontLoader.java

@@ -2,13 +2,10 @@ package schule.ngb.zm.util;
 
 import java.awt.Font;
 import java.awt.FontFormatException;
-import java.awt.GraphicsEnvironment;
-import java.io.File;
-import java.io.FileInputStream;
 import java.io.IOException;
 import java.io.InputStream;
-import java.net.URL;
 import java.util.Map;
+import java.util.Objects;
 import java.util.concurrent.ConcurrentHashMap;
 
 public class FontLoader {
@@ -16,10 +13,13 @@ public class FontLoader {
 	private static final Map<String, Font> fontCache = new ConcurrentHashMap<>();
 
 	public static Font loadFont( String source ) {
-		if( source == null || source.length() == 0 )
+		Objects.requireNonNull(source, "Font source may not be null");
-			throw new IllegalArgumentException("Font source may not be null or empty.");
+		if( source.length() == 0 ) {
+			throw new IllegalArgumentException("Font source may not be empty.");
+		}
 
 		if( fontCache.containsKey(source) ) {
+			LOG.trace("Retrieved font <%s> from font cache.", source);
 			return fontCache.get(source);
 		}
 
@@ -27,47 +27,33 @@ public class FontLoader {
 		Font font = Font.decode(source);
 		if( font != null && source.toLowerCase().contains(font.getFamily().toLowerCase()) ) {
 			fontCache.put(source, font);
+			LOG.debug("Loaded system font for <%s>.", source);
 			return font;
+		} else {
+			font = null;
 		}
 
 		// Load userfonts
-		try {
+		try( InputStream in = ResourceStreamProvider.getResourceStream(source) ) {
-			InputStream in;
+			font = Font.createFont(Font.TRUETYPE_FONT, in).deriveFont(Font.PLAIN);
-
-			File file = new File(source);
-			if( file.isFile() ) {
-				in = new FileInputStream(file);
-			} else {
-				// load ressource relative to .class-file
-				in = FontLoader.class.getResourceAsStream(source);
-
-				// relative to ClassLoader
-				if( in == null ) {
-					in = FontLoader.class.getClassLoader().getResourceAsStream(source);
-				}
-
-				// load form web or jar-file
-				if( in == null ) {
-					in = new URL(source).openStream();
-				}
-
-
-				font = Font.createFont(Font.TRUETYPE_FONT, in).deriveFont(Font.PLAIN);
-			}
 
 			if( font != null ) {
 				fontCache.put(source, font);
 				//ge.registerFont(font);
 			}
-
+			LOG.debug("Loaded custom font from <%s>.", source);
-			return font;
+		} catch( IOException ioex ) {
-		} catch( IOException | FontFormatException ex ) {
+			LOG.error(ioex, "Error loading custom font file from source <%s>.", source);
-			// TODO: Log error!
+		} catch( FontFormatException ffex ) {
-			return null;
+			LOG.error(ffex, "Error creating custom font from source <%s>.", source);
 		}
+
+		return font;
 	}
 
 	private FontLoader() {
 	}
 
+	private static final Log LOG = Log.getLogger(FontLoader.class);
+
 }

src/schule/ngb/zm/util/ResourceStreamProvider.java

@@ -0,0 +1,171 @@
+package schule.ngb.zm.util;
+
+import schule.ngb.zm.Zeichenmaschine;
+
+import java.io.*;
+import java.net.URL;
+import java.util.logging.Level;
+import java.util.logging.LogManager;
+import java.util.logging.Logger;
+
+/**
+ * Helferklasse, um {@link InputStream}s für Resourcen zu erhalten.
+ */
+public class ResourceStreamProvider {
+
+	/**
+	 * Sucht eine zur angegebenen Quelle passende Resource und öffnet einen
+	 * passenden {@link InputStream}. Die konkrete Art des Streams hängt davon
+	 * ab, welche Art an Resource gefunden wird:
+	 * <ol>
+	 * <li>Ist {@code source} eine existierende Datei
+	 *    {@code new File(source}.isFile() == true}, dann wird ein
+	 *    {@link FileInputStream} erstellt.</li>
+	 * <li>Ist {@code source} ein relativer Pfad im Projekt, wird ein passender
+	 * 	Stream mit {@link Class#getResourceAsStream(String)} geöffnet.</li>
+	 * <li>Ist {@code source} im Classpath enthalten, wird ein passender Stream
+	 * 	mit {@link ClassLoader#getResourceAsStream(String)} geöffnet.</li>
+	 * <li>Ist {@code source}  eine gültige {@link URL}, dann wird ein
+	 *    {@link URL#openStream() Netwerkstream} geöffnet.</li>
+	 * </ol>
+	 * <p>
+	 * Die Möglichen Resourcen werden in der gelisteten Reihenfolge durchprobiert
+	 * und der erste gefundene Stream zurückgegeben. Auftretende Exceptions
+	 * werden als {@link IOException} geworfen. Das bedeutet, falls für {@code source}
+	 * keine gültige Resource gefunden werden kann, wird am Ende die
+	 * von {@link URL#openStream()} erzeugte {@code IOException} geworfen.
+	 * <p>
+	 * Bei einer Exception werden die folgenden Quellen nicht mehr abgefragt.
+	 * Eine {@link SecurityException} beim Öffnen des {@code FileInputStream}
+	 * (zum Beispiel, weil auf eine existierende Datei keine Leserechte bestehen)
+	 * verhindert daher, dass noch im Classpath gesucht wird.
+	 *
+	 * @param source Eine Quelle für die Resource (Absoluter Dateipfad,
+	 * 	Dateipfad im Classpath oder Netzwerkresource)
+	 * @return Ein {@code InputStream} für die Resource
+	 * @throws NullPointerException     Falls {@code source} {@code null} ist.
+	 * @throws IllegalArgumentException Falls {@code source} ein leerer String
+	 *                                  ist.
+	 * @throws IOException              Geworfen beim öffnen des Streams zu
+	 *                                  einer bestehenden Resource oder falls
+	 *                                  keine passende Resource gefunden wurde.
+	 */
+	public static InputStream getResourceStream( String source ) throws NullPointerException, IllegalArgumentException, IOException {
+		if( source == null ) {
+			throw new NullPointerException("Resource source may not be null");
+		}
+		if( source.length() == 0 ) {
+			throw new IllegalArgumentException("Resource source may not be empty.");
+		}
+
+		InputStream in;
+
+		// See if source is a readable file
+		File file = new File(source);
+		try {
+			if( file.isFile() ) {
+				in = new FileInputStream(file);
+			}
+		} catch( FileNotFoundException fnfex ) {
+			// Somehow an exception occurred, but we still try other sources
+			in = null;
+		}
+		// File does not exist, try other means
+		// load ressource relative to .class-file
+		in = Zeichenmaschine.class.getResourceAsStream(source);
+
+		// relative to ClassLoader
+		if( in == null ) {
+			in = Zeichenmaschine.class.getClassLoader().getResourceAsStream(source);
+		}
+
+		// load form web or jar-file
+		if( in == null ) {
+			in = new URL(source).openStream();
+		}
+
+		// One of the above got a valid Stream,
+		// otherwise an Exception was thrown
+		return in;
+	}
+
+	/**
+	 * Ermittelt zur angegebenen Quelle einen passenden {@link URL} (<em>Unified
+	 * Resource Locator</em>). Eine passende Datei-Resource wird wie folgt
+	 * ermittelt:
+	 * <ol>
+	 * <li>Ist {@code source} eine existierende Datei
+	 *    ({@code new File(source}.isFile() == true})?</li>
+	 * <li>Ist {@code source} ein relativer Pfad im Projekt ({@code getResource(source) != null})?.</li>
+	 * <li>Ist {@code source} im Classpath enthalten ({@code getClassLoader().getResource(source) != null})?</li>
+	 * <li>Ansonten erstellt ein {@link URL}-Objekt.</li>
+	 * </ol>
+	 * <p>
+	 * Ein {@code URL} für die erste gefundene Resource wird zurückgegeben.
+	 * Auftretende Exceptions
+	 * werden als {@link IOException} geworfen.
+	 * <p>
+	 * Bei einer Exception werden die folgenden Quellen nicht mehr abgefragt.
+	 * Eine {@link java.net.MalformedURLException} beim Konstruieren des {@code URL}
+	 * zu einer Datei verhindert daher, dass noch im Classpath gesucht wird.
+	 *
+	 * @param source Eine Quelle für die Resource (Absoluter Dateipfad,
+	 * 	Dateipfad im Classpath oder Netzwerkresource)
+	 * @return Ein {@code InputStream} für die Resource
+	 * @throws NullPointerException     Falls {@code source} {@code null} ist.
+	 * @throws IllegalArgumentException Falls {@code source} ein leerer String
+	 *                                  ist.
+	 * @throws IOException              Geworfen beim erzeugen eines URL zu
+	 *                                  einer bestehenden Resource.
+	 */
+	public static URL getResourceURL( String source ) throws NullPointerException, IllegalArgumentException, IOException {
+		if( source == null ) {
+			throw new NullPointerException("Resource source may not be null");
+		}
+		if( source.length() == 0 ) {
+			throw new IllegalArgumentException("Resource source may not be empty.");
+		}
+
+		File file = new File(source);
+		if( file.isFile() ) {
+			return file.toURI().toURL();
+		}
+
+		URL url;
+
+		url = Zeichenmaschine.class.getResource(source);
+		if( url != null ) {
+			return url;
+		}
+
+		url = Zeichenmaschine.class.getClassLoader().getResource(source);
+		if( url != null ) {
+			return url;
+		}
+
+		return new URL(source);
+	}
+
+	public static InputStream getResourceStream( File file ) throws FileNotFoundException, SecurityException {
+		if( file == null ) {
+			throw new NullPointerException("Provided file can't be null.");
+		}
+
+		return new FileInputStream(file);
+	}
+
+	public static InputStream getResourceStream( URL url ) throws IOException {
+		if( url == null ) {
+			throw new NullPointerException("Provided URL can't be null.");
+		}
+
+		return url.openStream();
+	}
+
+	private ResourceStreamProvider() {
+	}
+
+
+	private static final Log LOG = Log.getLogger(ResourceStreamProvider.class);
+
+}