Доброго времени суток.
Допустим, у нас есть какой-то DAO класс, работающий через JDBC. Нужно добавить метод, обновляющий поле записи по его id (id - уникальный). Вот первый вариант метода:
1.
2.
3.
4.
public void upadateValue(String value, String id) {
String sql = "UPDATE some_table SET table_value=? WHERE item_id = ?";
jdbcTemplate.update(sql, value, id);
}
Вот второй вариант:
1.
2.
3.
4.
5.
6.
7.
8.
public Boolean upadateValue(String value, String id) {
String sql = "UPDATE some_table SET table_value=? WHERE item_id = ?";
int result = jdbcTemplate.update(sql, value, id);
if(result != 1){
return false;
}
return true;
}
Во втором варианте, после обновления делается дополнительная проверка кол-ва изменённых записей. Если было изменено больше одной записи, либо ни одной, то возвращаем из метода значение false, указывающее, на ошибку. Иначе true, говорящее, что всё прошло как ожидалось.
Как частный случай, можно вместо булевого значения возвращать void, а в случае возникновения ошибки (result != 1) бросать какой-нить Exception. Такой подход выглядит надёжнее, т.к. тогда код, в котором вызывается этот метод, уже не сможет проигнорировать случай с ошибкой.
А теперь вопросы:
1. Нужно ли вообще делать подобную проверку (будь то возвращаемое значение или выбрасываемый эксэпшн)? С одной стороны, это даёт лишнюю уверенность, что код отработал верно. С другой стороны, это сильно смахивает на паранойю.
2. Если по предыдущему пункту ответ - проверку делать нужно, то писать подобную конструкцию с проверкой в каждом методе, выглядит как-то однообразно. Возможно для этого есть какие-то конкретные типичные решения или подходы?
3. В данном случае, по условию задачи обновление должно затрагивать только одну запись (по уникальному id). Но уникальность id описана в схеме БД, а проверка кол-ва изменений реализована в методе. Т.е. если вдруг в какой-то момент, изменится ожидаемое поведение (например станет допустимым иметь несколько записей с одним id), то этот метод со своей проверкой становится не актуальным. Т.е. такая проверка усложняет поддержку?
4. Возможно стоит делать проверку правильности обновления/удаления уровнем выше? Скажем, где-то в сервис-слое.
Дополнительные сомнения на счёт этого подхода возникают на фоне того, что в сети, практически нигде в примерах не встречал подобного. А это явный признак, что что-то делается не так.
