Opa, segue minha avaliação.
- Não vejo necessidade de o README ser um arquivo RTF, já que o mesmo só possui texto puro.
- Tive que caçar o arquivo sql, pois o README não dizia onde ele estava.
- Copiei a pasta, como descrito nas instruções, mas o procedimento não deu certo. Tentei fazer algumas coisas, mas não consegui rodar. O ideal seria um arquivo war.
- Além disso, em nenhum lugar é dito qual usuário/senha é usado para entrar no bd... seria preciso caçar nos arquivos de configuração do seu projeto.
Não consegui ver a interface...mas vamos ao código:
- O uso de Struts e Spring tira muita importância do que foi feito. Acredito que o professor queria que usassemos as tecnologias mais básicas, pra irmos entendendo como as coisas foram evoluindo. Com o uso dessas tecnologias a coisa ficou muito fácil...mas, na minha opinião, fugiu totalmente do escopo.
- As páginas não seguem o padrão xhtml.
- Não entendi porque cada DAO implementa uma interface diferente. Nesse caso não vejo a necessidade da criação das interfaces.
- Os arquivos com mais de uma classe dentro ficam muito complicados. Talvez seja melhor diminuir ou até eliminar essa prática.
O fato de não conseguir rodar e ver a interface acabou atrapalhando um pouco a avaliação. Mas acho que é isso...
Tentei ser bem conciso, mas se algo não ficou claro, pode perguntar sem problemas!
abraço