De fato, isto aqui não está fazendo muito sentido:
@Autowired
private AtividadeRepository atividadeRepository;
private AtividadeDAO atividadeRepository = new AtividadeDAO();
Aliás, nem passaria na compilação, visto que você está declarando dois objetos diferentes com o mesmo nome dentro do escopo de uma mesma classe.
Outra coisa é que provavelmente você esteja fazendo alguma confusão de conceitos. Ou você usa um repositório, ou usa uma DAO. Repositórios são entidades que trabalham a nível de coleção. DAOs são entidades que trabalham a nível de tecnologia de banco de dados.
Além disso, a diferença entre DAO (Data Access Object) e DAL (Data Access Layer) não é apenas de uma letra: DAO é o objeto que faz parte de uma DAL. Normalmente, tanto a DAO quanto um repositório devolvem objetos do tipo DTO (Data Transfer Object).
Tendo isto entendido, podemos passar para o @Autowired
.
@Autowired
é a anotação para Injeção de Dependência. Ela indica ao framework que o objeto marcado vai ser gerenciado pelo framework (seu ciclo de vida inteiro). Faz sentido para um Service
, mas não faz muito sentido para uma entidade, visto que normalmente uma entidade (na maioria das vezes) é um objeto anêmico que não requer gerenciamento do ciclo de vida (definição de DTO).
Portanto, para este caso, acho que seria interessante remover a declaração da DAO.