several string concatenations in loops
close resources in finally and not in the catch block