Ricardo Terra
Doutorando em Ciência da Computação pela UFMG/UWaterloo
==================================================================================
module Entities: br.com.linkcom.sgm.beans.*
module MAController: br.com.linkcom.neo.controller.MultiActionController+
module GService: br.com.linkcom.neo.service.GenericService+
module GDAO: br.com.linkcom.neo.persistence.GenericDAO+
module QBuilder: br.com.linkcom.neo.persistence.QueryBuilder
==================================================================================
Regra DCL: Entities must-annotate javax.persistence.Entity
Violações (2):
- ControleCadastro
- ControleCadastroItem
Recomendação: Colocar anotação.
==================================================================================
Regra DCL: Entities must-annotate javax.persistence.Id
Violações (2):
- ControleCadastro
- ControleCadastroItem
Recomendação: Colocar anotação.
==================================================================================
Regra DCL: Entities must-annotate javax.persistence.GeneratedValue
Violações (2):
- ControleCadastro
- ControleCadastroItem
Recomendação: Colocar anotação
==================================================================================
Regra DCL: Entities must-annotate br.com.linkcom.neo.bean.annotation.DescriptionProperty
Violações (32):
- AcompanhamentoIndicador
- AnexoAnomalia
- AnexoIndicador
- Atividade
- AuditoriaGestaoIndicador
- AuditoriaGestaoIndicadorItem
- AuditoriaInterna
- CausaEfeito
- Comentario
- ComentarioItem
- Competencia
- ControleCadastro
- ControleCadastroItem
- EmailHistorico
- EmailHistoricoUsuario
- ItemAuditoriaInterna
- LogProcesso
- MapaCompetencia
- MapaEstrategico
- MapaNegocio
- MatrizFCS
- MatrizFCSFator
- MatrizFCSIniciativaFator
- PainelIndicadorFiltro
- ParametrosSistema
- Permissao
- SolicitacaoCancelamentoIndicador
- SolicitacaoRepactuacaoIndicador
- Tela
- UsuarioAuditoriaInterna
- UsuarioPapel
- UsuarioUnidadeGerencial
Recomendação: Colocar anotação.
==================================================================================
Regra DCL: $system cannot-create GService, GDAO, MAController
Violação (1):
- UsuarioPapelService: 37 (linha)
Recomendação: Remover instanciação
==================================================================================
Regra DCL: GDAO cannot-create QBuilder
Descrição: Classes que estendem GenericDAO devem utilizar o método query()
ao invés de instanciar o QueryBuilder.
Violações (12):
- br.com.linkcom.sgm.dao.UnidadeGerencialDAO: 172
- br.com.linkcom.sgm.dao.UnidadeGerencialDAO: 360
- br.com.linkcom.sgm.dao.UnidadeGerencialDAO: 378
- br.com.linkcom.sgm.dao.UnidadeGerencialDAO: 397
- br.com.linkcom.sgm.dao.SolicitacaoRepactuacaoIndicadorDAO: 48
- br.com.linkcom.sgm.dao.SolicitacaoCancelamentoIndicadorDAO: 148
- br.com.linkcom.sgm.dao.SolicitacaoCancelamentoIndicadorDAO: 183
- br.com.linkcom.sgm.dao.MapaNegocioDAO: 52
- br.com.linkcom.sgm.dao.IndicadorDAO: 235
- br.com.linkcom.sgm.dao.AuditoriaGestaoDAO: 79
- br.com.linkcom.sgm.dao.ArquivoDAO: 47
- br.com.linkcom.sgm.dao.AnomaliaDAO: 323
Recomendação: Minha abordagem não foi capaz de dar recomendações.
No entanto, seria interessante, de acordo com o conversado com o Rógel, que o
QueryBuilder não seja instanciado diretamente, mas sim, instanciado
sempre pelo mesmo método, como o método newQueryBuilder. Isso facilitaria
personalizações de queries que afetam todo o sistema. Isso só acontece
com tipos Wrappers (Integer, Long, etc) e Java (String and Object).
==================================================================================
Autor: Ricardo Terra
88 comentários
--
Ricardo Terra
--
Ricardo Terra
--
Ricardo Terra
Doutorando em Ciência da Computação pela UFMG/UWaterloo