敏捷开发中要慎用继承
ParticipantsInDB继承了来自DBTable的方法，比如clear和getCount。对于ParticipantsInDB来讲，clear这个方法的确是有用的：清空所有的参会者。但getCount就造成了一点点小意外了：通过ParticipantsInDB调用getCount这个方法时，是取得participants这个表里面所有的记录，不管删除标记是true还是false的。而实际上，没人想知道这个数据。即使有人想知道，这个方法也不应该叫做getCount，因为这名字很容易就会跟“计算所有的（有效）参会者数量”联系在一起。
因此，ParticipantsInDB是不是真的应该继承这个方法getCount呢？或者我们应该怎么做比较恰当呢？
它们之间是否真的有继承关系？
当我们继承了一些我们不想要的东西，我们应该再三的想想：它们之间是不是真的有继承关系？ParticipantsInDB必须是一个DBTable吗？ParticipantsInDB希不希望别人知道它是一个DBTable？
实际上，ParticipantsInDB描述的是系统中所有的参会者的集合，该系统可以是个单数据库的，也可以是多数据库的，也就是说，这个类可以代表一个数据库里的一个Participants表，也可以代表两个数据库各自的两个Participants表的总和。
如果还不清楚的话，我们就这样举例吧，比如，现在我们已经有了2000个参会者，在两个数据库中存放，其中数据库A的participants表里面存放了1000个参会者，数据库B的participants这个表存放了1000个参会者。DBTable顶多只能描述一个数据库里面的一张表，也就是1000个参会者，而participants则可以完全的描述这2000年参会者的信息。前面可以当作数据库的数据表在系统中的代表，而后者表示的应该包含更多业务逻辑的一个域对象。（原谅这边我只能用域对象这样的词来断开这样的混淆。）
因此，我们可以判断，ParticipantsInDB跟DBTable之间不应该有什么继承的关系。ParticipantsInDB不能继承DBTable这个类了。于是，现在ParticipantsInDB也没有getCount这个方法了。可是ParticipantsInDB还需要DBTable类里面的其他方法啊，那怎么办？所以现在我们让ParticipantsInDB里面引用了一个DBTable：
public class DBTable {
private Connection conn;
private String tableName; public DBTable(String tableName) {
this.tableName = tableName; this.conn = ...; } public void clear() { PreparedStatement st = conn.prepareStatement("DELETE FROM "+tableName);
try { st.executeUpdate(); }finally{ st.close(); } } public int getCount() { PreparedStatement st = conn.prepareStatement("SELECT COUNT(*) FROM "+tableName);
try { ResultSet rs = st.executeQuery();
rs.next(); return rs.getInt(1);
}finally{
st.close();
}
}
public String getTableName() {
return tableName;
}
public Connection getConn() {
return conn;
}
}
public class ParticipantsInDB {
private DBTable table;
public ParticipantsInDB() {
table = new DBTable("participants");
}
public void addParticipant(Participant part) {
}
public void deleteParticipant(String participantId) {
setDeleteFlag(participantId, true);
}
public void restoreParticipant(String participantId) {
setDeleteFlag(participantId, false);
}
private void setDeleteFlag(String participantId, boolean b) {
}
public void reallyDelete() {
PreparedStatement st = table.getConn().prepareStatement(
"DELETE FROM "+table.getTableName()+ " WHERE deleteFlag=true");
try {
st.executeUpdate();
}finally{
st.close();
}
}
public void clear() {
table.clear();
}
public int countParticipants() {
PreparedStatement st = table.getConn().prepareStatement(
"SELECT COUNT(*) FROM "+table.getTableName()+ " WHERE deleteFlag=false");
try {
ResultSet rs = st.executeQuery();
rs.next();
return rs.getInt(1);
}finally{
st.close();
}
}
}
唐月华
