- 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();
}
}
ingenuus 13.06.2011 20:32 # 0
volatile 13.06.2011 20:39 # 0
Lure Of Chaos 13.06.2011 22:09 # 0
выглядит как обычное удаление одной записи, все даже правильно, по-моему, по крайней мере, после 7ой строчки.
3.14159265 13.06.2011 22:13 # −1
bugmenot 13.06.2011 22:38 # 0
ключ
Lure Of Chaos 14.06.2011 21:22 # 0
bugmenot 14.06.2011 21:29 # 0
подбрасывание исключения
Lure Of Chaos 14.06.2011 21:31 # 0
bugmenot 14.06.2011 21:38 # 0
Lure Of Chaos 14.06.2011 21:41 # 0
volatile 14.06.2011 22:11 # 0
Многие сошлись на том, что это ужасно плохо.
Lure Of Chaos 14.06.2011 22:15 # +2
tir 14.06.2011 08:01 # 0
Если комментарий PRIMARY KEY(`id`) намек на строчки 15-16, то проверка в 16-й строчке может выявить ошибку в бизнес логике (удаление уже удаленного объекта). Хотя конечно можно было бы прописать и явно if(res == 0) {...}
volatile 14.06.2011 10:50 # 0
guest8 09.04.2019 10:57 # −999