From 692dc7be9791f131bfb253c13363b75bc41b8467 Mon Sep 17 00:00:00 2001 From: ulrich Date: Sun, 01 Dec 2024 16:56:40 +0000 Subject: [PATCH] JarScanner umbenannt, das Laden von Klassen ohne Jar ebenfalls auf Scanner umgestellt --- src/de/uhilger/neon/Factory.java | 267 +++++++++++++++++++++++------------------------------ 1 files changed, 117 insertions(+), 150 deletions(-) diff --git a/src/de/uhilger/neon/Factory.java b/src/de/uhilger/neon/Factory.java index ba46730..7cb2a86 100644 --- a/src/de/uhilger/neon/Factory.java +++ b/src/de/uhilger/neon/Factory.java @@ -23,7 +23,6 @@ import com.sun.net.httpserver.HttpContext; import com.sun.net.httpserver.HttpHandler; import com.sun.net.httpserver.HttpServer; -import de.uhilger.neon.JarScanner.JarScannerListener; import de.uhilger.neon.entity.ContextDescriptor; import de.uhilger.neon.entity.NeonDescriptor; import de.uhilger.neon.entity.ServerDescriptor; @@ -48,6 +47,7 @@ import java.util.concurrent.Executors; import java.util.logging.Level; import java.util.logging.Logger; +import de.uhilger.neon.Scanner.ScannerListener; /** * Einen Neon-Server aus einer Beschreibungsdatei herstellen @@ -65,7 +65,7 @@ * @author Ulrich Hilger * @version 1, 6.2.2024 */ -public class Factory implements JarScannerListener { +public class Factory implements ScannerListener { public Factory() { listeners = new ArrayList<>(); @@ -94,21 +94,23 @@ Gson gson = new Gson(); return gson.fromJson(sb.toString(), NeonDescriptor.class); } - - public void runInstance(Class c, NeonDescriptor d) - throws ClassNotFoundException, NoSuchMethodException, InstantiationException, + + public void runInstance(Class c, NeonDescriptor d) + throws ClassNotFoundException, NoSuchMethodException, InstantiationException, IllegalAccessException, IllegalArgumentException, InvocationTargetException, IOException { this.runInstance(c, d, null, new ArrayList<>()); } - public void runInstance(Class c, NeonDescriptor d, List<String> packageNames) - throws ClassNotFoundException, NoSuchMethodException, InstantiationException, + public void runInstance(Class c, NeonDescriptor d, List<String> packageNames) + throws ClassNotFoundException, NoSuchMethodException, InstantiationException, IllegalAccessException, IllegalArgumentException, InvocationTargetException, IOException { this.runInstance(c, d, packageNames, new ArrayList<>()); } + /** * Einen Neon-Server gemaess einem Serverbeschreibungsobjekt herstellen und starten * + * @param starter die Klasse, mit der Neon durch Aufruf dieser Methode gestartet wird * @param d das Object mit der Serverbeschreibung * @param packageNames Namen der Packages, aus der rekursiv vorgefundene Actors eingebaut werden * sollen @@ -121,20 +123,12 @@ * @throws InvocationTargetException * @throws IOException */ - public void runInstance(Class c, NeonDescriptor d, List<String> packageNames, List<DataProvider> sdp) - throws ClassNotFoundException, NoSuchMethodException, InstantiationException, + public void runInstance(Class starter, NeonDescriptor d, List<String> packageNames, List<DataProvider> sdp) + throws ClassNotFoundException, NoSuchMethodException, InstantiationException, IllegalAccessException, IllegalArgumentException, InvocationTargetException, IOException { Logger.getLogger(Factory.class.getName()).log(Level.FINER, System.getProperty("java.class.path")); - Class pingClass = c.getClassLoader().loadClass("de.uhilger.neonbaselokal.actor.Ping"); - if(pingClass != null) { - Logger.getLogger(Factory.class.getName()).log(Level.FINER, - pingClass.getName() + " " + pingClass.getPackageName() + - " " + pingClass.getSimpleName()); - } else { - Logger.getLogger(Factory.class.getName()).log(Level.FINER, "pingClass not found"); - } - + List serverList = d.server; Iterator<ServerDescriptor> serverIterator = serverList.iterator(); while (serverIterator.hasNext()) { @@ -142,40 +136,40 @@ HttpServer server = HttpServer.create(new InetSocketAddress(sd.port), 0); fireServerCreated(server); - if(packageNames == null) { + if (packageNames == null) { packageNames = d.actorPackages; - } - addContexts(c, d, server, sd.contexts, packageNames, sdp); + } + addContexts(new Scanner(starter, Actor.class), d, server, sd.contexts, packageNames, sdp); server.setExecutor(Executors.newFixedThreadPool(10)); server.start(); } fireInstanceStarted(); } - + private Authenticator createAuthenticator(NeonDescriptor d) { Authenticator auth = null; - if(d.authenticator != null) { + if (d.authenticator != null) { try { Object authObj = Class.forName(d.authenticator.className) .getDeclaredConstructor().newInstance(); - if(authObj instanceof Authenticator) { + if (authObj instanceof Authenticator) { auth = (Authenticator) authObj; return auth; } - } catch (ClassNotFoundException | NoSuchMethodException | SecurityException | - InstantiationException | IllegalAccessException | IllegalArgumentException | - InvocationTargetException ex) { + } catch (ClassNotFoundException | NoSuchMethodException | SecurityException + | InstantiationException | IllegalAccessException | IllegalArgumentException + | InvocationTargetException ex) { // Klasse nicht gefunden. Muss das geloggt oder sonstwie behandel werden? return null; - } - } + } + } return auth; } - private void addContexts(Class c, NeonDescriptor d, HttpServer server, List contextList, List<String> packageNames, - List<DataProvider> sdp) - throws ClassNotFoundException, NoSuchMethodException, InstantiationException, + private void addContexts(Scanner scn, NeonDescriptor d, HttpServer server, List contextList, List<String> packageNames, + List<DataProvider> sdp) + throws ClassNotFoundException, NoSuchMethodException, InstantiationException, IllegalAccessException, IllegalArgumentException, InvocationTargetException, IOException { Map<String, HttpHandler> sharedHandlers = new HashMap(); Iterator<ContextDescriptor> contextIterator = contextList.iterator(); @@ -184,45 +178,45 @@ ContextDescriptor cd = contextIterator.next(); HttpHandler h = buildHandler(cd, sharedHandlers); if (h != null) { - HttpContext ctx = server.createContext(cd.contextPath, h); + HttpContext ctx = server.createContext(cd.contextPath, h); Map<String, Object> ctxAttrs = ctx.getAttributes(); /* Achtung: Wenn verschiedene Elemente dasselbe Attribut deklarieren, ueberschreiben sie sich die Attribute gegenseitig. */ - ctxAttrs.putAll(cd.attributes); - if (h instanceof Handler) { + ctxAttrs.putAll(cd.attributes); + if (h instanceof Handler) { for (String packageName : packageNames) { - wireActors(c, - packageName, Actor.class, (Handler) h, + wireActors(scn, + packageName, (Handler) h, cd.attributes.get("contextName")); - ctx.getAttributes().put("serverDataProviderList", sdp); + ctx.getAttributes().put("serverDataProviderList", sdp); } - } - if(cd.authenticator instanceof String) { - if(!(auth instanceof Authenticator)) { + } + if (cd.authenticator instanceof String) { + if (!(auth instanceof Authenticator)) { auth = createAuthenticator(d); } - if(auth instanceof Authenticator) { - ctx.setAuthenticator(auth); + if (auth instanceof Authenticator) { + ctx.setAuthenticator(auth); ctx.getAttributes().putAll(d.authenticator.attributes); fireAuthenticatorCreated(ctx, auth); // event umbenennen in etwas wie authAdded oder so } - + } - + //Authenticator auth = createAuthenticator(d); //if (auth instanceof Authenticator && cd.authenticator instanceof String) { // ctx.setAuthenticator(auth); // ctx.getAttributes().putAll(d.authenticator.attributes); // fireAuthenticatorCreated(ctx, auth); //} - if(cd.filter != null) { - for(String filterClassName : cd.filter) { + if (cd.filter != null) { + for (String filterClassName : cd.filter) { // Object filterObj = Class.forName(filterClassName) - .getDeclaredConstructor().newInstance(); - if(filterObj instanceof Filter) { + .getDeclaredConstructor().newInstance(); + if (filterObj instanceof Filter) { Filter filter = (Filter) filterObj; ctx.getFilters().add(filter); } @@ -235,10 +229,10 @@ } } } - - private HttpHandler buildHandler(ContextDescriptor cd, Map<String, HttpHandler> sharedHandlers) throws ClassNotFoundException, NoSuchMethodException, InstantiationException, + + private HttpHandler buildHandler(ContextDescriptor cd, Map<String, HttpHandler> sharedHandlers) throws ClassNotFoundException, NoSuchMethodException, InstantiationException, IllegalAccessException, IllegalArgumentException, InvocationTargetException, IOException { - HttpHandler h; + HttpHandler h; if (!cd.sharedHandler) { h = getHandlerInstance(cd); } else { @@ -252,7 +246,7 @@ } return h; } - + private HttpHandler getHandlerInstance(ContextDescriptor cd) { try { Object handlerObj = Class.forName(cd.className) @@ -263,112 +257,81 @@ // kein HttpHandler aus newInstance return null; } - } catch (ClassNotFoundException | NoSuchMethodException | SecurityException | - InstantiationException | IllegalAccessException | IllegalArgumentException | - InvocationTargetException ex) { + } catch (ClassNotFoundException | NoSuchMethodException | SecurityException + | InstantiationException | IllegalAccessException | IllegalArgumentException + | InvocationTargetException ex) { // Klasse nicht gefunden. Muss das geloggt oder sonstwie behandel werden? return null; } } - private void wireActors(Class c, String packageName, Class annotation, Handler h, String contextName) { - JarScanner js = new JarScanner(); - URI path; - try { - path = js.getPath(c); - if(path.toString().endsWith(".class")) { - ClassLoader cl = c.getClassLoader(); - InputStream stream = cl - .getResourceAsStream(packageName.replaceAll("[.]", "/")); - BufferedReader reader = new BufferedReader(new InputStreamReader(stream)); - Iterator i = reader.lines().iterator(); - while (i.hasNext()) { - String line = i.next().toString(); - if (line.endsWith(".class")) { - try { - Class actorClass = cl.loadClass(packageName + "." - + line.substring(0, line.lastIndexOf('.'))); - if (actorClass != null && actorClass.isAnnotationPresent(annotation)) { - wire(h, actorClass, contextName); - } - } catch (ClassNotFoundException ex) { - // Klasse nicht gefunden. Muss das geloggt oder sonstwie behandel werden? - } - } else { - wireActors(c, packageName + "." + line, annotation, h, contextName); - } - } - } else { - ClassLoader cl = js.getUrlClassLoader(c); - js.processZipContent(cl, new File(path), packageName, this, h, contextName); - } - //listClasses(c, packageName); - } catch (URISyntaxException ex) { - Logger.getLogger(Factory.class.getName()).log(Level.SEVERE, ex.getMessage(), ex); + @SuppressWarnings("unchecked") + private void wireActors(Scanner scn, String packageName, /*Class annotation, */Handler h, String contextName) { + if (!scn.isJar()) { + scn.processClasses(this, packageName, h, contextName); + } else { + scn.processZipContent(packageName, this, h, contextName); } } - + /** - * Diese Testmethode zeigt, dass die Methode getResourceAsStream nicht funktioniert wie - * dokumentiert. - * - * 1. Sie liefert den Inhalt einer gegebenen Package mitsamt Unterpackages als - * Stream, wenn sie auf eine Packagestruktur angewendet wird, die unverpackt in einem - * Verzeichnis des Dateisystems liegt. - * - * 2. Sie liefert - faelschlicherweise - null bzw. einen leeren Stream, wenn die Packagestruktur + * Diese Testmethode zeigt, dass die Methode getResourceAsStream nicht funktioniert wie + * dokumentiert. + * + * 1. Sie liefert den Inhalt einer gegebenen Package mitsamt Unterpackages als Stream, wenn sie + * auf eine Packagestruktur angewendet wird, die unverpackt in einem Verzeichnis des Dateisystems + * liegt. + * + * 2. Sie liefert - faelschlicherweise - null bzw. einen leeren Stream, wenn die Packagestruktur * in einem Jar verpackt ist. - * - * Saemtliche Versuche, ueber den ClassPath oder die Pfadangabe der Package das Verhalten zu - * aendern, gehen bislang fehl (z.B. / oder . als Separator, - * / oder . zu Beginn enthalten oder nicht, realative oder absolute packagepfadangabe). - * Es ist auch unerheblich, ob + * + * Saemtliche Versuche, ueber den ClassPath oder die Pfadangabe der Package das Verhalten zu + * aendern, gehen bislang fehl (z.B. / oder . als Separator, / oder . zu Beginn enthalten oder + * nicht, realative oder absolute packagepfadangabe). Es ist auch unerheblich, ob * Class.getResourceAsStream oder Class.getClassLoader().getResourceAsStream verwendet wird. - * - * Unabhaengig davon, ob und wie letztlich im Fall 2. oben die Methode getResourceAsStream - * dazu zu bringen waere, eine Inhaltsliste fuer eine Package zu liefern ist allein die - * Tatsache, dass sich die Methode unterschiedlich verhaelt bereits ein - * schwerer Bug. - * + * + * Unabhaengig davon, ob und wie letztlich im Fall 2. oben die Methode getResourceAsStream dazu zu + * bringen waere, eine Inhaltsliste fuer eine Package zu liefern ist allein die Tatsache, dass + * sich die Methode unterschiedlich verhaelt bereits ein schwerer Bug. + * * @param c - * @param packageName + * @param packageName */ private void listClasses(Class c, String packageName) { - Logger.getLogger(Factory.class.getName()).log(Level.FINER, "packageName: " + packageName); - //ClassLoader cl = c.getClassLoader(); - String newPackageName = packageName.replaceAll("[.]", "/"); - Logger.getLogger(Factory.class.getName()).log(Level.FINER, "newPackageName: " + newPackageName); - InputStream stream = c - .getResourceAsStream(newPackageName); - if(stream != null) { - BufferedReader reader = new BufferedReader(new InputStreamReader(stream)); - Iterator i = reader.lines().iterator(); - Logger.getLogger(Factory.class.getName()).log(Level.FINER, Long.toString(reader.lines().count())); - while (i.hasNext()) { - String line = i.next().toString(); - Logger.getLogger(Factory.class.getName()).log(Level.FINER, "class to inspect: " + line); - if (line.endsWith(".class")) { - try { - Class actorClass = c.getClassLoader().loadClass(packageName + "." - + line.substring(0, line.lastIndexOf('.'))); - if (actorClass != null && actorClass.isAnnotationPresent(Actor.class)) { - Logger.getLogger(Factory.class.getName()).log(Level.FINER, "ACTOR"); - } else { - Logger.getLogger(Factory.class.getName()).log(Level.FINER, "no actor"); - } - } catch (ClassNotFoundException ex) { - Logger.getLogger(Factory.class.getName()).log(Level.FINER, "Klasse nicht gefunden"); + Logger.getLogger(Factory.class.getName()).log(Level.FINER, "packageName: " + packageName); + //ClassLoader cl = c.getClassLoader(); + String newPackageName = packageName.replaceAll("[.]", "/"); + Logger.getLogger(Factory.class.getName()).log(Level.FINER, "newPackageName: " + newPackageName); + InputStream stream = c + .getResourceAsStream(newPackageName); + if (stream != null) { + BufferedReader reader = new BufferedReader(new InputStreamReader(stream)); + Iterator i = reader.lines().iterator(); + Logger.getLogger(Factory.class.getName()).log(Level.FINER, Long.toString(reader.lines().count())); + while (i.hasNext()) { + String line = i.next().toString(); + Logger.getLogger(Factory.class.getName()).log(Level.FINER, "class to inspect: " + line); + if (line.endsWith(".class")) { + try { + Class actorClass = c.getClassLoader().loadClass(packageName + "." + + line.substring(0, line.lastIndexOf('.'))); + if (actorClass != null && actorClass.isAnnotationPresent(Actor.class)) { + Logger.getLogger(Factory.class.getName()).log(Level.FINER, "ACTOR"); + } else { + Logger.getLogger(Factory.class.getName()).log(Level.FINER, "no actor"); } - } else { - listClasses(c, packageName + "." + line); + } catch (ClassNotFoundException ex) { + Logger.getLogger(Factory.class.getName()).log(Level.FINER, "Klasse nicht gefunden"); } - } } else { - Logger.getLogger(Factory.class.getName()).log(Level.FINER, "stream ist null"); + listClasses(c, packageName + "." + line); } + } + } else { + Logger.getLogger(Factory.class.getName()).log(Level.FINER, "stream ist null"); + } } - - + /* Eine Action-Annotation enthaelt gewoehnlich die Route, die 'unterhalb' des Kontextpfades als 'Ausloeser' zur @@ -377,7 +340,13 @@ Wenn die Action fuer alle Routen 'unterhalb' des Kontextpfades ausgefuehrt werden soll, muss die Action als Route '/' angeben. - */ + */ + /** + * + * @param h Handler, dem der Actor hinzugefuegt wird, falls der Kontext uebereinstimmt + * @param c hinzuzufuegende Actor-Klasse + * @param contextName Name des Kontext, dem der Actor hinzugefuegt wird + */ private void wire(Handler h, Class c, String contextName) { Method[] methods = c.getMethods(); for (Method method : methods) { @@ -392,7 +361,6 @@ } /* -------------- FactoryListener Implementierung --------------- */ - private List<FactoryListener> listeners; public void addListener(FactoryListener l) { @@ -413,7 +381,7 @@ l.serverCreated(server); } } - + private void fireHandlerCreated(HttpContext ctx, HttpHandler h) { for (FactoryListener l : listeners) { l.handlerCreated(ctx, h); @@ -425,7 +393,7 @@ l.contextCreated(context); } } - + private void fireAuthenticatorCreated(HttpContext context, Authenticator auth) { for (FactoryListener l : listeners) { l.authenticatorCreated(context, auth); @@ -438,10 +406,9 @@ } } - /* -------------- JarScannerListener Implementierung --------------- */ - + /* -------------- ScannerListener Implementierung --------------- */ @Override - public void actorFound(Class actorClass, Handler h, String contextName) { - wire(h, actorClass, contextName); + public void annotationFound(Class foundClass, Handler h, String contextName) { + wire(h, foundClass, contextName); } -} +} \ No newline at end of file -- Gitblit v1.9.3