- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
public void removeFromBase()
throws SQLException{
for(ServiceInstance serv: getServiceInstances())
serv.removeFromBase();
for(Pay pay: pays)
pay.removeFromBase();
String query;
PreparedStatement ps = null;
Connection connection = null;
try{
connection = TIDatasource.getInstance().getConnection();
query = "DELETE FROM orders WHERE id=?";
ps = connection.prepareStatement(query);
ps.setLong(1, getId());
int res = ps.executeUpdate();
if(res != 1)
throw new SQLException("Unexpected number of orders have deleted: " + res);
}catch(SQLException sqlex){
throw new SQLException("Can't remove order from base because of error: " + sqlex);
}finally{
ps.close();
connection.close();
}
}
Follow us!
выглядит как обычное удаление одной записи, все даже правильно, по-моему, по крайней мере, после 7ой строчки.
ключ
подбрасывание исключения
Многие сошлись на том, что это ужасно плохо.
Если комментарий PRIMARY KEY(`id`) намек на строчки 15-16, то проверка в 16-й строчке может выявить ошибку в бизнес логике (удаление уже удаленного объекта). Хотя конечно можно было бы прописать и явно if(res == 0) {...}