Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−1
Statement stmt = null;
ResultSet rs = null;
try{
stmt = connection.createStatement();
ResultSet rs = stmt.executeQuery("select * from VENDORS");
while(rs.next())
{
//тело цикла....
}
}
finally
{
rs.close(); stmt.close();
}Тимлид сказал что за такое будет руки отрывать. Что тут не так?
Запостил: 
mozga,
12 Декабря 2019
Follow us!
Понял, они же закрываются, так что после исполнения цикла они не нужны.
Должно быть как-то так:
В блоке try..catch отлавливается исключение, если вызов фабричных методов "connection.createStatement();" и "ResultSet rs = stmt.executeQuery("select * from VENDORS"); " был неудачен. В противном случае, инструкция входит в блок try..finally, где работает с полученными экземплярами объектов и затем освобождает их.