Is it OK to have singleton DAO objects?

Question

Consider the following classes' structure:

1. BaseDAO with methods to crest PreparedStatement and get connection from pool
2. AccountDAO extends BaseDAO to work with Account table via JDBC. This class is singleton
3. AccountService witch calls methods of AccountDAO like this: AccountDAO.getInstance().login(name, password).

AccountDAO is a Spring bean with @Transactional annotations to methods that insert some data.

Is this OK? I think singleton DAO classes can cause performance problems. May be it is better to use some spring injections into service layer classes? (I'm new to Spring, so any advice will be appriciated)

Answer 1

The recommended approach in the Spring documentation is to write your DAOs as normal classes and use the singleton scope. This will work fine if your DAOs maintain no state.

http://static.springsource.org/spring/docs/2.0.x/reference/beans.html#beans-factory-scopes-prototype

section 3.4.2.

If you are using Spring, you shouldn't need to deal with prepared statements and whatnot, unless you are doing something wonky. Look at JdbcTemplate or HibnerateTemplate. Yes, you should wire Spring to inject your DAOs into your services or wherever you need to use them.