Index: src/java/org/apache/commons/dbutils/handlers/ArrayHandler.java
===================================================================
--- src/java/org/apache/commons/dbutils/handlers/ArrayHandler.java (revision 752322)
+++ src/java/org/apache/commons/dbutils/handlers/ArrayHandler.java (working copy)
@@ -43,14 +43,14 @@
* The RowProcessor implementation to use when converting rows
* into arrays.
*/
- private RowProcessor convert = ROW_PROCESSOR;
+ private final RowProcessor convert;
/**
* Creates a new instance of ArrayHandler using a
* BasicRowProcessor for conversion.
*/
public ArrayHandler() {
- super();
+ this(ROW_PROCESSOR);
}
/**
Index: src/java/org/apache/commons/dbutils/handlers/ArrayListHandler.java
===================================================================
--- src/java/org/apache/commons/dbutils/handlers/ArrayListHandler.java (revision 752322)
+++ src/java/org/apache/commons/dbutils/handlers/ArrayListHandler.java (working copy)
@@ -34,14 +34,14 @@
* The RowProcessor implementation to use when converting rows
* into Object[]s.
*/
- private RowProcessor convert = ArrayHandler.ROW_PROCESSOR;
+ private final RowProcessor convert;
/**
* Creates a new instance of ArrayListHandler using a
* BasicRowProcessor for conversions.
*/
public ArrayListHandler() {
- super();
+ this(ArrayHandler.ROW_PROCESSOR);
}
/**
Index: src/java/org/apache/commons/dbutils/handlers/BeanHandler.java
===================================================================
--- src/java/org/apache/commons/dbutils/handlers/BeanHandler.java (revision 752322)
+++ src/java/org/apache/commons/dbutils/handlers/BeanHandler.java (working copy)
@@ -33,13 +33,13 @@
/**
* The Class of beans produced by this handler.
*/
- private Class type = null;
+ private final Class type;
/**
* The RowProcessor implementation to use when converting rows
* into beans.
*/
- private RowProcessor convert = ArrayHandler.ROW_PROCESSOR;
+ private final RowProcessor convert;
/**
* Creates a new instance of BeanHandler.
@@ -48,7 +48,7 @@
* are created from.
*/
public BeanHandler(Class type) {
- this.type = type;
+ this(type, ArrayHandler.ROW_PROCESSOR);
}
/**
Index: src/java/org/apache/commons/dbutils/handlers/BeanListHandler.java
===================================================================
--- src/java/org/apache/commons/dbutils/handlers/BeanListHandler.java (revision 752322)
+++ src/java/org/apache/commons/dbutils/handlers/BeanListHandler.java (working copy)
@@ -34,13 +34,13 @@
/**
* The Class of beans produced by this handler.
*/
- private Class type = null;
+ private final Class type;
/**
* The RowProcessor implementation to use when converting rows
* into beans.
*/
- private RowProcessor convert = ArrayHandler.ROW_PROCESSOR;
+ private final RowProcessor convert;
/**
* Creates a new instance of BeanListHandler.
@@ -49,7 +49,7 @@
* are created from.
*/
public BeanListHandler(Class type) {
- this.type = type;
+ this(type, ArrayHandler.ROW_PROCESSOR);
}
/**
Index: src/java/org/apache/commons/dbutils/handlers/ColumnListHandler.java
===================================================================
--- src/java/org/apache/commons/dbutils/handlers/ColumnListHandler.java (revision 752322)
+++ src/java/org/apache/commons/dbutils/handlers/ColumnListHandler.java (working copy)
@@ -32,20 +32,20 @@
/**
* The column number to retrieve.
*/
- private int columnIndex = 1;
+ private final int columnIndex;
/**
* The column name to retrieve. Either columnName or columnIndex
* will be used but never both.
*/
- private String columnName = null;
+ private final String columnName;
/**
* Creates a new instance of ColumnListHandler. The first column of each
* row will be returned from handle().
*/
public ColumnListHandler() {
- super();
+ this(1, null);
}
/**
@@ -55,7 +55,7 @@
* ResultSet.
*/
public ColumnListHandler(int columnIndex) {
- this.columnIndex = columnIndex;
+ this(columnIndex, null);
}
/**
@@ -65,9 +65,16 @@
* ResultSet.
*/
public ColumnListHandler(String columnName) {
- this.columnName = columnName;
+ this(1, columnName);
}
+ // Helper
+ private ColumnListHandler(int columnIndex, String columnName) {
+ super();
+ this.columnIndex = columnIndex;
+ this.columnName = columnName;
+ }
+
/**
* Returns one ResultSet column value as Object.
*
Index: src/java/org/apache/commons/dbutils/handlers/MapHandler.java
===================================================================
--- src/java/org/apache/commons/dbutils/handlers/MapHandler.java (revision 752322)
+++ src/java/org/apache/commons/dbutils/handlers/MapHandler.java (working copy)
@@ -35,14 +35,14 @@
* The RowProcessor implementation to use when converting rows
* into Maps.
*/
- private RowProcessor convert = ArrayHandler.ROW_PROCESSOR;
+ private final RowProcessor convert;
/**
* Creates a new instance of MapHandler using a
* BasicRowProcessor for conversion.
*/
public MapHandler() {
- super();
+ this(ArrayHandler.ROW_PROCESSOR);
}
/**
Index: src/java/org/apache/commons/dbutils/handlers/MapListHandler.java
===================================================================
--- src/java/org/apache/commons/dbutils/handlers/MapListHandler.java (revision 752322)
+++ src/java/org/apache/commons/dbutils/handlers/MapListHandler.java (working copy)
@@ -34,14 +34,14 @@
* The RowProcessor implementation to use when converting rows
* into Maps.
*/
- private RowProcessor convert = ArrayHandler.ROW_PROCESSOR;
+ private final RowProcessor convert;
/**
* Creates a new instance of MapListHandler using a
* BasicRowProcessor for conversion.
*/
public MapListHandler() {
- super();
+ this(ArrayHandler.ROW_PROCESSOR);
}
/**
Index: src/java/org/apache/commons/dbutils/handlers/ScalarHandler.java
===================================================================
--- src/java/org/apache/commons/dbutils/handlers/ScalarHandler.java (revision 752322)
+++ src/java/org/apache/commons/dbutils/handlers/ScalarHandler.java (working copy)
@@ -32,20 +32,20 @@
/**
* The column number to retrieve.
*/
- private int columnIndex = 1;
+ private final int columnIndex;
/**
* The column name to retrieve. Either columnName or columnIndex
* will be used but never both.
*/
- private String columnName = null;
+ private final String columnName;
/**
* Creates a new instance of ScalarHandler. The first column will
* be returned from handle().
*/
public ScalarHandler() {
- super();
+ this(1, null);
}
/**
@@ -55,7 +55,7 @@
* ResultSet.
*/
public ScalarHandler(int columnIndex) {
- this.columnIndex = columnIndex;
+ this(columnIndex, null);
}
/**
@@ -65,7 +65,13 @@
* ResultSet.
*/
public ScalarHandler(String columnName) {
- this.columnName = columnName;
+ this(1, columnName);
+ }
+
+ // Helper constructor
+ private ScalarHandler(int columnIndex, String columnName){
+ this.columnIndex = columnIndex;
+ this.columnName = columnName;
}
/**