Creating a default DAO interface

Question

I'm wondering if it's a good practise to create default DAO interface instead of creating own interface for each class.

public interface DAO {

    public void addItem();
    public void updateItem();
    public void removeItem();
    public Object getItem(int id);
    public Object[] getAll();
}

Now we can implement this interface by multiple classes. Of course this sollution has it's cons like downcasting during retrieving data but I think it's still more efficient and code clear. Is this a good way to go?

Answer 1

It is a good way, but at least one improvement can be done using generics:

public interface DAO<T> {
    public void addItem(T item);
    public void updateItem(T item);
    public void removeItem(T item);
    public T getItem(int id);
    public List<T> getAll();
}

So now you will not need any casting. But anyway, you must make sure that all the DAOs will at least have those methods. Otherwise that will lead to more complexities. Moreover, if there are some DAOs that will have just exactly those methods, you will end up with pretty much compact clean code, e.g.:

public interface UserDAO extends DAO<User> {
    // And that is basically it :)
}

public class UserDAOImpl implements UserDAO {
    // Your implementations here
    // ...
}

Note: I have replaced the Object[] with List<T> and not T[] as you cannot do that in case of generics. But that is not a cons, it is better to use built-in containers.