You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by me...@apache.org on 2001/03/23 22:55:55 UTC
cvs commit: jakarta-tomcat/src/share/org/apache/tomcat/util/res StringManager.java
melaquias 01/03/23 13:55:55
Modified: src/share/org/apache/tomcat/util/res StringManager.java
Log:
Changes getString(String key) to return null if the requested resource is not found. This is consistent with general java container pattern usage and enables calling code to detect that the requested string was not found via a null check.
Revision Changes Path
1.2 +33 -15 jakarta-tomcat/src/share/org/apache/tomcat/util/res/StringManager.java
Index: StringManager.java
===================================================================
RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/util/res/StringManager.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- StringManager.java 2001/02/20 03:12:46 1.1
+++ StringManager.java 2001/03/23 21:55:55 1.2
@@ -1,4 +1,5 @@
/*
+ * $Id: StringManager.java,v 1.2 2001/03/23 21:55:55 melaquias Exp $
* ====================================================================
*
* The Apache Software License, Version 1.1
@@ -81,8 +82,12 @@
* <p>Please see the documentation for java.util.ResourceBundle for
* more information.
*
+ * @version $Revision: 1.2 $ $Date: 2001/03/23 21:55:55 $
+ *
* @author James Duncan Davidson [duncan@eng.sun.com]
* @author James Todd [gonzo@eng.sun.com]
+ * @author Mel Martinez [mmartinez@g1440.com]
+ * @see java.util.ResourceBundle
*/
public class StringManager {
@@ -114,32 +119,45 @@
private StringManager(String packageName,Locale loc) {
String bundleName = packageName + ".LocalStrings";
try {
- bundle = ResourceBundle.getBundle(bundleName,loc);
- } catch( MissingResourceException ex ) {
- bundle= ResourceBundle.getBundle( bundleName, Locale.US);
- }
+ bundle = ResourceBundle.getBundle(bundleName,loc);
+ } catch( MissingResourceException ex ) {
+ bundle= ResourceBundle.getBundle( bundleName, Locale.US);
+ }
}
/**
- * Get a string from the underlying resource bundle.
- *
- * @param key
+ Get a string from the underlying resource bundle or return
+ null if the String is not found.
+
+ @param key to desired resource String
+ @return resource String matching <i>key</i> from underlying
+ bundle or null if not found.
+ @throws IllegalArgumentException if <i>key</i> is null.
*/
public String getString(String key) {
- if (key == null) {
- String msg = "key is null";
+ if(key == null){
+ String msg = "key may not have a null value";
- throw new NullPointerException(msg);
+ throw new IllegalArgumentException(msg);
}
String str = null;
- try {
- str = bundle.getString(key);
- } catch (MissingResourceException mre) {
- str = "[cannot find message associated with key '" + key + "' due to " + mre + "]";
- // mre. print Stack Trace();
+ try{
+ str = bundle.getString(key);
+ }catch(MissingResourceException mre){
+ //bad: shouldn't mask an exception the following way:
+ // str = "[cannot find message associated with key '" + key + "' due to " + mre + "]";
+ // because it hides the fact that the String was missing
+ // from the calling code.
+ //good: could just throw the exception (or wrap it in another)
+ // but that would probably cause much havoc on existing
+ // code.
+ //better: consistent with container pattern to
+ // simply return null. Calling code can then do
+ // a null check.
+ str = null;
}
return str;