[jboss-cvs] JBossAS SVN: r81975 - projects/demos/microcontainer/trunk/ioc/src/main/java/org/jboss/demos/ioc/access.

[jboss-cvs-commits at lists.jboss.org]

Author: alesj
Date: 2008-12-02 12:32:37 -0500 (Tue, 02 Dec 2008)
New Revision: 81975

Modified:
   projects/demos/microcontainer/trunk/ioc/src/main/java/org/jboss/demos/ioc/access/ConstantsProvider.java
Log:
Add checks.

Modified: projects/demos/microcontainer/trunk/ioc/src/main/java/org/jboss/demos/ioc/access/ConstantsProvider.java
===================================================================
--- projects/demos/microcontainer/trunk/ioc/src/main/java/org/jboss/demos/ioc/access/ConstantsProvider.java	2008-12-02 17:29:46 UTC (rev 81974)
+++ projects/demos/microcontainer/trunk/ioc/src/main/java/org/jboss/demos/ioc/access/ConstantsProvider.java	2008-12-02 17:32:37 UTC (rev 81975)
@@ -36,6 +36,10 @@
    private String className;
    private ClassInfo classInfo;
 
+   public ConstantsProvider()
+   {
+   }
+
    public ConstantsProvider(String className)
    {
       this.className = className;
@@ -58,6 +62,9 @@
 
    public Object getConstant(String constantName) throws Throwable
    {
+      if (classInfo == null)
+         throw new IllegalArgumentException("Cannot execute constant lookup on null class info.");
+
       return getConstant(classInfo, constantName);
    }
 
@@ -69,7 +76,13 @@
 
    protected Object getConstant(ClassInfo classInfo, String constantName) throws Throwable
    {
+      if (constantName == null)
+         throw new IllegalArgumentException("Null constant name");
+
       FieldInfo field = classInfo.getDeclaredField(constantName);
+      if (field == null)
+         throw new IllegalArgumentException("No such constant: " + constantName + ", class info: " + classInfo);
+
       return field.get(null);
    }
 }
\ No newline at end of file