Question

我正在使用JDBC，我的许多类都有一个内部RowMapper类，如下所示：

public class Foo {
  class AppleRows implements RowMapper<Apple> {
    public Apple mapRow(ResultSet rs, int rowNum) throws SQLException {
      Apple a = new Apple();
      a.setName(rs.getString("Name"));
    }
  }

  class AppleRowsJoinedWithSomethingElse implements RowMapper<Apple> {
    public Apple mapRow(ResultSet rs, int rowNum) throws SQLException {
      Apple a = new Apple();   
      a.setName(rs.getString("Name"));
      a.setSomethingElse(rs.getString("SomethingElse"));
    } 
  }
}

在上面的示例中，行a.setName(rs.getString("Name"))正在重复。这只是一个例子，但在我的实际代码中有超过10个这样的字段。我想知道是否有更好的方法来做到这一点？

注意：我需要不同的映射器，因为我在一些地方使用它们，我正在将结果与另一个表（获取更多字段）相关联。

Answer 1

你可以extend +使用super.mapRow() ...

public class Foo {
  class AppleRows implements RowMapper<Apple> {
    public Apple mapRow(ResultSet rs, int rowNum) throws SQLException {
      Apple a = new Apple();
      a.setName(rs.getString("Name"));
      return a;
    }
  }

  class AppleRowsJoinedWithSomethingElse extends AppleRows {
    public Apple mapRow(ResultSet rs, int rowNum) throws SQLException {
      Apple a = super.mapRow(rs, rowNum);
      a.setSomethingElse(rs.getString("SomethingElse"));
      return a;
    } 
  }
}

或者只是委托，如果你不喜欢使用继承作为代码重用的机制：

public class Foo {
  class AppleRows implements RowMapper<Apple> {
    public Apple mapRow(ResultSet rs, int rowNum) throws SQLException {
      Apple a = new Apple();
      a.setName(rs.getString("Name"));
      return a;
    }
  }

  class AppleRowsJoinedWithSomethingElse implements RowMapper<Apple> {
    public Apple mapRow(ResultSet rs, int rowNum) throws SQLException {
      Apple a = new AppleRows().mapRow(rs, rowNum);
      a.setSomethingElse(rs.getString("SomethingElse"));
      return a;
    } 
  }
}

Answer 2

一种可能的方法是使用继承，特别是如果你可以将内部类变成static嵌套类，如下所示：

public class Foo {
  static class AppleRows implements RowMapper<Apple> {
    public Apple mapRow(ResultSet rs, int rowNum) throws SQLException {
      Apple a = new Apple();
      a.setName(rs.getString("Name"));
      return a;
    }
  }

  static class AppleRowsJoinedWithSomethingElse extends AppleRows {
    public Apple mapRow(ResultSet rs, int rowNum) throws SQLException {
      Apple a = super.mapRow(rs, rowNum);
      a.setSomethingElse(rs.getString("SomethingElse"));
      return a;
    } 
  }
}

我在这里使用static嵌套类的原因是因为存在对非static嵌套类中包含类的实例的隐式引用（至少在当前实现中）

Answer 3

每个映射器映射单个行定义。我不会称这是对DRY的违反。

在AppleRowsJoinedWithSomethingElse的基础视图中更改单个列名称时，您只需在一个位置更改它。

如果您使用公共代码重构此内容，那么如果提供AppleRowsJoinedWithSomethingElse的视图发生更改，但AppleRows下方的视图没有更改，则您将处于一个奇怪的位置。

Answer 4

没有必要扩展以共享通用功能。我只是输入这个，所以它可能实际上没有编译。

public class CommonBlammy
{
    private CommonBlammy() { } // private constructor to prevent instantiation.

    public static Apple mapRow(final ResultSet resultSet)
    throws SQLException
    {
        Apple returnValue = new Apple();

        returnValue.setName(resultSet.getString("Name"));

        return returnValue;
    }
}

public class AppleRows implements RowMapper
{
    public Apple mapRow(
        final ResultSet resultSet,
        final int rowNumber)
        throws SQLException
    {
      return CommonBlammy.mapRow(ResultSet resultSet);
    }
}

public class AppleRowsJoinedWithSomethingElse implements RowMapper
{
    public Apple mapRow(
        final ResultSet resultSet,
        final int rowNumber)
    throws SQLException
    {
        Apple returnValue;

        returnValue = CommonBlammy.mapRow(ResultSet resultSet);
        returnValue.setSomethingElse(rs.getString("SomethingElse"));

        return returnValue;
    } 
}

有没有一种干法来整合RowMappers的相同代码？

4 个答案: