[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
jboss-cvs-commits at lists.jboss.org
Tue Dec 2 12:32:37 EST 2008
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
More information about the jboss-cvs-commits
mailing list