Changeset 4075

Timestamp:

Dec 16, 2010, 8:41:07 AM (12 years ago)

Author:

Gregory Vincic

Message:

Rewrote class documentation of ActionFactory?

File:

• trunk/client/servlet/src/org/proteios/action/ActionFactory.java (modified) (5 diffs)

trunk/client/servlet/src/org/proteios/action/ActionFactory.java

@@ -103 +103 @@
 
 /**
-    This factory creates one instance of each action during it's initialization.
-    Each action is mapped by an id in a HashMap<String, AbstractAction>. The id
-    is generated using {@link #getId(Class)}.
+    This factory implements the {@link ActionFactoryInterface}. It's purpose is to initialize each new action
+    with additional factories e.g. {@link ItemFactory} and localizer. You should never instantiate your own 
+    actions.
                  
     @author gregory
@@ -445 +445 @@
             log.info("Loading actions from jar-files");
             FileLocator fileLocator = new FileLocator();
-            File libDir = new File(context
-            .getRealPath("WEB-INF" + File.separator + "lib"));
-            List<File> jarFiles = fileLocator.find(libDir, false, Pattern
-            .compile("proteios_client\\.jar$"));
+            File libDir = new File(context.getRealPath("WEB-INF" + File.separator + "lib"));
+            List<File> jarFiles = fileLocator.find(libDir, false, Pattern.compile("proteios_client\\.jar$"));
             for (File jarFile : jarFiles)
             {
-                List<JarEntry> classesInJar = fileLocator.find(jarFile, Pattern
-                .compile(".*\\.class$"));
+                List<JarEntry> classesInJar = fileLocator.find(jarFile, Pattern.compile(".*\\.class$"));
                 for (JarEntry entry : classesInJar)
                 {
                     try
                     {
-                        String className = fileLocator.asClassName(entry
-                        .getName());
+                        String className = fileLocator.asClassName(entry.getName());
                         // Load classes that extend ProteiosAction
                         Class<?> classInJar = actionLoader.loadClass(className);
@@ -467 +463 @@
                             if (classInJar != ProteiosAction.class)
                             {
-                                loadAction((ProteiosAction<?>) classInJar
-                                .newInstance());
+                                loadAction((ProteiosAction<?>) classInJar.newInstance());
                             }
                         }
@@ -489 +484 @@
             for (File extJarFile : extJarFiles)
             {
-                List<JarEntry> classesInJar = local.find(extJarFile, Pattern
-                .compile(".*\\.class$"));
+                List<JarEntry> classesInJar = local.find(extJarFile, Pattern.compile(".*\\.class$"));
                 for (JarEntry entry : classesInJar)
                 {
@@ -496 +490 @@
                     {
                         String className = local.asClassName(entry.getName());
-                        log.debug("Extension classname for entry '" + entry
-                        .getName() + "' is '" + className);
+                        log.debug("Extension classname for entry '" + entry.getName() + "' is '" + className);
                         // Load classes that extend ProteiosAction
                         Class<?> classInJar = actionLoader.loadClass(className);
                         if (ProteiosAction.class.isAssignableFrom(classInJar))
                         {
-                            loadAction((ProteiosAction) classInJar
-                            .newInstance());
+                            loadAction((ProteiosAction) classInJar.newInstance());
                         }
                     }