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; } /**