- added more checks on method/constructor parameter
- made more local variables final
// Invoke default constructor
this();
+ // Validate all parameter
+ if (null == factoryJndi) {
+ // Throw IAE
+ throw new NullPointerException("factoryJndi is null"); //NOI18N
+ } else if (factoryJndi.isEmpty()) {
+ // Throw IAE
+ throw new IllegalArgumentException("factoryJndi is empty"); //NOI18N
+ } else if (null == queueJndi) {
+ // Throw IAE
+ throw new NullPointerException("queueJndi is null"); //NOI18N
+ } else if (queueJndi.isEmpty()) {
+ // Throw IAE
+ throw new IllegalArgumentException("queueJndi is empty"); //NOI18N
+ }
+
// Try it out
try {
// Get initial context
* @return Property value
*/
protected int getIntegerContextParameter (final String parameterKey) throws NullPointerException, NumberFormatException {
+ // Validate parameter
+ if (null == parameterKey) {
+ // Throw IAE
+ throw new NullPointerException("parameterKey is null"); //NOI18N
+ } else if (parameterKey.isEmpty()) {
+ // Throw IAE
+ throw new IllegalArgumentException("parameterKey is empty"); //NOI18N
+ }
+
// Get context parameter
final Integer contextValue = Integer.parseInt(this.getStringContextParameter(parameterKey));