我在静态方法中使用通用类型时遇到了一些麻烦。
欢迎所有对源代码的评论,尤其是那些显着改进代码的评论。我目前也不打算使用任何外部框架,除了 JDBC,以保持它仍然简单,请不要过分强调这一点。
我对不使用外部框架的看法也得到了以下事实的支持:我将在数据库上使用的操作非常少:
- 插入数据
- 更新数据
- 检索所有字段。(只需输入不同的 SQL 查询,您就可以选择要检索的字段
我不打算制作一个完整的框架,所以我知道它不会支持一切。检索所有字段的速度并不是一个真正的问题,因为这几乎只在服务器启动时完成,如果在任何其他时间使用,它将在后台任务中完成,我并不关心它何时完成.
实体.java:
abstract public class Entity<KeyType, DataType> {
protected KeyType key;
protected List<Object> data;
public Entity() {
data = new ArrayList<>();
}
//abstract public static Map<KeyType, DataType> getAll();
protected List<Object> createData(final DataAction dataAction) {
List<Object> list = new ArrayList<>();
if (dataAction == DataAction.INSERT) {
list.add(key);
}
list.addAll(data);
if (dataAction == DataAction.UPDATE) {
list.add(key);
}
return list;
}
abstract public void insert();
abstract public void update();
protected static <KeyType, DataType> Map<KeyType, DataType> getData(final Class<DataType> dataTypeClass, final String query) {
Map<KeyType, DataType> map = new HashMap<>();
try {
PreparedStatement preparedStatement = DatabaseConnection.getConnection().prepareStatement(query);
ResultSet resultSet = preparedStatement.executeQuery();
while (resultSet.next()) {
KeyType key = (KeyType)resultSet.getObject(1);
int index = 2;
List<Object> dataList = new ArrayList<>();
while (resultSet.getObject(index) != null) {
dataList.add(resultSet.getObject(index));
index++;
}
DataType dataObject = null;
try {
dataObject = dataTypeClass.getConstructor(List.class).newInstance(dataList);
} catch (InstantiationException | IllegalAccessException | IllegalArgumentException | InvocationTargetException | NoSuchMethodException | SecurityException ex) {
Logger.getLogger(Entity.class.getName()).log(Level.SEVERE, null, ex);
}
map.put(key, dataObject);
}
} catch (SQLException ex) {
Logger.getLogger(Entity.class.getName()).log(Level.SEVERE, null, ex);
}
return map;
}
protected void executeQuery(final String query, final List<Object> data) {
try {
PreparedStatement preparedStatement = DatabaseConnection.getConnection().prepareStatement(query);
int dataIndex = 0;
for (Object dataObject : data) {
preparedStatement.setObject(dataIndex, dataObject);
dataIndex++;
}
preparedStatement.execute();
preparedStatement.close();
} catch (SQLException ex) {
Logger.getLogger(Entity.class.getName()).log(Level.SEVERE, null, ex);
}
}
}
一个具体的实现,Account.java:
public class Account extends Entity<String, Account> {
private final static String SELECT_ALL_QUERY = "SELECT * FROM accounts";
private final static String INSERT_QUERY = "INSERT INTO accounts (username, password) VALUES(?, ?)";
private final static String UPDATE_QUERY = "UPDATE accounts SET password=? WHERE username=?";
private String username;
private String password;
public Account(final String username, final String password) {
this.username = username;
this.password = password;
key = username;
data.add(password);
}
public Account(final List<Object> data) {
this((String)data.get(0), (String)data.get(1));
}
public String getUsername() {
return username;
}
public void setUsername(final String username) {
this.username = username;
}
public String getPassword() {
return password;
}
public void setPassword(final String password) {
this.password = password;
}
public static Map<String, Account> selectAll() {
return getData(Account.class, SELECT_ALL_QUERY);
}
@Override
public void insert() {
executeQuery(INSERT_QUERY, createData(DataAction.INSERT));
}
@Override
public void update() {
executeQuery(UPDATE_QUERY, createData(DataAction.UPDATE));
}
}
我对具体的实现总体上很满意,似乎我已经设法将它降低到最低限度,除了public Account(final List<Object> data)
看起来不太好,但我可以忍受它。
但是,正如猜测的那样,getData()
fromEntity
肯定不好,如果可能的话,我想改进它。
我想使用的是类似的东西DataType dataObject = new DataType(dataList)
,但似乎无法实例化通用类型参数。
那么有什么方法可以在我当前的视图中优化我当前的代码吗?是否可以进一步解耦具体类和抽象类?
编辑:
添加了一个相关问题(我认为我不应该为这件事提出一个全新的问题,对吧?):
有没有办法将静态字符串(SQL 查询)和 Account 类中的insert()
和update()
移出到 Entity 类中?