Identificação de campos obrigatgorios e validação.

git-svn-id: https://svn.bento.ifrs.edu.br/default/ASES/e-selo@9826 c2178572-b5ca-4887-91d2-9e3a90c7d55b

Identificação de campos obrigatgorios e validação.
git-svn-id: https://svn.bento.ifrs.edu.br/default/ASES/e-selo@9826 c2178572-b5ca-4887-91d2-9e3a90c7d55b
magno.oliveira
1 parent cb02b921
Showing 7 changed files with 196 additions and 191 deletions Show diff stats
Desenvolvimento/Codificacao/e-Selo/WebContent/WEB-INF/jsp/faixa/form.jsp
Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/component/messages.properties
Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/CriteriosController.java
Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/FaixaController.java
Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/ParametroCalculoController.java
Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/RecomendacaoController.java
Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/model/Faixa.java
@@ -25,43 +25,44 @@
 		<div class="row">
 			<div class="col-md-12">
-				<form name="faixasForm" class="form-horizontal" action="<c:url value="/conformidades"/>" method="post">
+				<form name="faixasForm" class="form-horizontal" action="<c:url value="/faixas"/>" method="post">
 					<fieldset>
 						<div class="form-group">
-							<label for="inputFaixa" class="col-lg-2 control-label">Faixa</label>
+							<label for="inputFaixa" class="col-lg-2 control-label required">Faixa</label>
 							<div class="col-xs-2">
 								<input id="inputFaixa" name="faixa.faixa" type="text" class="form-control required number" placeholder="Faixa" value="${faixa.faixa}" />
 							</div>
 						</div>
 						<div class="form-group">
-								<label for="inputDesc" class="col-lg-2 control-label">Descrição</label>
+								<label for="inputDesc" class="col-lg-2 control-label required">Descrição</label>
 								<div class="col-lg-10">
-									<input id="inputDesc" name="faixa.descricao" type="text" class="form-control" placeholder="Descrição"  value="${faixa.descricao}" >
+									<input id="inputDesc" name="faixa.descricao" type="text" class="form-control required" placeholder="Descrição"  value="${faixa.descricao}" >
 								</div>
 						</div>
 						<input type="hidden" name="faixa.diretriz.id" value="${diretrizId}" />
 						<div class="form-group">
-							<label for="inputFaixaInicio" class="col-lg-2 control-label">Faixa Inicial</label>
+							<label for="inputFaixaInicio" class="col-lg-2 control-label required">Faixa Inicial</label>
 							<div class="col-xs-2">
-								<input id="inputFaixaInicio" name="conformidade.faixaInicio" type="text" class="form-control required number" placeholder="Faixa Inicial" value="${faixa.faixaInicio}" min="0" />
+								<input id="inputFaixaInicio" name="faixa.faixaInicio" type="text" class="form-control required number" placeholder="Faixa Inicial" value="${faixa.faixaInicio}" min="0" />
 							</div>
 						</div>
 						<div class="form-group">
-							<label for="inputFaixaFim" class="col-lg-2 control-label">Faixa Final</label>
+							<label for="inputFaixaFim" class="col-lg-2 control-label required">Faixa Final</label>
 							<div class="col-xs-2">
-								<input id="inputFaixaFim" name="conformidade.faixaFim" type="text" class="form-control required number" placeholder="Faixa Final" value="${faixa.faixaFim}" min="0" />
+								<input id="inputFaixaFim" name="faixa.faixaFim" type="text" class="form-control required number" placeholder="Faixa Final" value="${faixa.faixaFim}" min="0" />
 							</div>
 						</div>
 						<div class="form-group">
-							<label for="inputPercentualConformidade" class="col-lg-2 control-label">Percentual de conformidade</label>
+							<label for="inputPercentualConformidade" class="col-lg-2 control-label required">Percentual de conformidade</label>
 							<div class="col-xs-2">
-								<input id="inputPercentualConformidade" name="conformidade.percentualConformidade" type="text" class="form-control required number" placeholder="Percentual de conformidade" value="${faixa.percentualConformidade}" min="0" />
+								<input id="inputPercentualConformidade" name="faixa.percentualConformidade" type="text" class="form-control required number" placeholder="Percentual de conformidade" value="${faixa.percentualConformidade}" min="0" />
 							</div>
 						</div>
 						<c:if test="${faixa.id != null}">
-							<input type="hidden" name="faixa.id" value="${faixa.id}" />							
+							<input type="hidden" name="faixa.id" value="${faixa.id}" />
+							<input type="hidden" name="faixa.criterio.id" value="${criterioId}" />
 							<input type="hidden" name="_method" value="put" />
 						</c:if>
-conformidades.mensagem.salva=Conformidade salva com sucesso.
-conformidades.mensagem.alterada=Conformidade alterada com sucesso.
-conformidades.validacao.diretriz=A Diretriz é obrigatória.
-conformidades.validacao.descricao=A descrição é obrigatória.
-conformidades.validacao.faixa=O número da Faixa é obrigatório.
-conformidades.validacao.faixa_final=A Faixa Final é obrigatória.
-conformidades.validacao.faixa_inicial=A Faixa Inicial é obrigatória.
-conformidades.validacao.tipo_conformidade=O Tipo de Conformidade é obrigatório.
-conformidades.validacao.range_faixas=A Faixa Inicial deve ser menor que a Faixa Inicial.
+faixas.mensagem.salva=Conformidade salva com sucesso.
+faixas.mensagem.alterada=Conformidade alterada com sucesso.
+faixas.validacao.diretriz=A Diretriz é obrigatória.
+faixas.validacao.descricao=A descrição é obrigatória.
+faixas.validacao.faixa=O número da Faixa é obrigatório.
+faixas.validacao.percentual_conformidade=O percentual de conformidade da Faixa é obrigatório.
+faixas.validacao.faixa_final=A Faixa Final é obrigatória.
+faixas.validacao.faixa_inicial=A Faixa Inicial é obrigatória.
+faixas.validacao.tipo_conformidade=O Tipo de Conformidade é obrigatório.
+faixas.validacao.range_faixas=A Faixa Inicial deve ser menor que a Faixa Inicial.
 parametro_calculo.validacao.descricao=A descrição é obrigatória.
 tipo_conformidade.mensagem.salva=Tipo de Diretriz salvo com sucesso.
 tipo_conformidade.mensagem.alterada=Tipo de Diretriz alterado com sucesso.
@@ -20,6 +20,7 @@ import br.com.eselo.dao.TipoTesteDAO;
 import br.com.eselo.model.Criterio;
 import br.com.eselo.model.ParametroCalculo;
 import br.com.eselo.model.Recomendacao;
+import br.com.eselo.model.TipoTeste;
 @Resource
 public class CriteriosController {
@@ -71,34 +72,9 @@ public class CriteriosController {
 	@Restrito(descricao = "CRITERIO_ADICIONAR")
 	public void adicionar(Criterio criterio) {
-		if (criterio.getNumeroCriterio() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o número do critério.",
-					"criterio.numeroCriterio"));
-		} else if (hasNumeroCriterioRecomendacaoAtual(criterio)) {
-			validator
-					.add(new ValidationMessage(
-							"Já existe um críterio ativo com esse numero. Critério não cadastrado.",
-							"criterio.numeroCriterio"));
-		} else if (criterio.getPeso() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o peso do critério.",
-					"criterio.peso"));
-		} else if (criterio.getDescricao() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar a descrição do critério.",
-					"criterio.descricao"));
-		}  else if (criterio.getPeso() <= 0) {
-			validator.add(new ValidationMessage(
-					"O peso do critério deve ser maior que zero.",
-					"criterio.peso"));
-		} else if (criterio.getTipoTeste().getId() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o Tipo de teste.",
-					"criterio.tipoTeste"));
-		}else{
-			criterio.setTipoTeste(new TipoTesteDAO(dao.getSession()).load(criterio.getTipoTeste().getId()));
-		}
+		criterio.setRecomendacao(new RecomendacaoDAO(dao.getSession()).load(criterio.getRecomendacao().getId()));
+		
+		validarCriterio(criterio);
 		result.include("recomendacaoId", criterio.getRecomendacao().getId());
 		result.include("parametroCalculo", criterio.getRecomendacao().getParametroCalculo());
@@ -107,7 +83,6 @@ public class CriteriosController {
 		validator.onErrorUsePageOf(CriteriosController.class).novo(criterio.getRecomendacao().getId());
-
 		this.criarNovoParametroAvaliacao(criterio, ParametroCalculoController.MODO_ADD);
 		result.include("mensagem", "Critério salvo com sucesso.");
@@ -117,40 +92,26 @@ public class CriteriosController {
 	@Put("/criterio")
 	@Restrito(descricao = "CRITERIO_ATUALIZAR")
 	public void atualizar(Criterio criterio) {
+		
+		criterio.setRecomendacao(new RecomendacaoDAO(dao.getSession()).load(criterio.getRecomendacao().getId()));
+
+		@SuppressWarnings("unchecked")
+		List<TipoTeste> tipoTestes = (List<TipoTeste>) dao.getAllTipoTeste();
 		Criterio criterioCadastrado = dao.load(criterio.getId());
 		criterioCadastrado.setNumeroCriterio(criterio.getNumeroCriterio());
 		criterioCadastrado.setDescricao(criterio.getDescricao());
 		criterioCadastrado.setPeso(criterio.getPeso());
 		criterioCadastrado.setTipoTeste(criterio.getTipoTeste());
-		if (criterio.getNumeroCriterio() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o número da recomendação.",
-					"criteiro.numeroCriterio"));
-		} else if (hasNumeroCriterioRecomendacaoAtual(criterioCadastrado)) {
-			validator
-					.add(new ValidationMessage(
-							"Já existe um criteiro ativo com esse numero. Criteiro não atualizado.",
-							"criteiro.numeroCriteiro"));
-		} else if (criterio.getPeso() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o peso do criteiro.",
-					"criteiro.peso"));
-		} else if (criterio.getDescricao() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar a descrição do criteiro.",
-					"criteiro.descricao"));
-		} else if (criterio.getTipoTeste() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o Tipo de teste.",
-					"criterio.tipoTeste"));
-		} 
+		
 		result.include("recomendacaoId", criterio.getRecomendacao().getId());
 		result.include("parametroCalculo", criterio.getRecomendacao().getParametroCalculo());
 		result.include("tipoTeste", criterio.getTipoTeste());
-		populateSelectLists();
-
+		result.include("tipoTesteList", tipoTestes);
+		
+		validarCriterio(criterio);
+		
 		validator.onErrorUsePageOf(CriteriosController.class).form(
 				criterioCadastrado.getId());
@@ -226,7 +187,37 @@ public class CriteriosController {
 		}
 	}
-	
+	private void validarCriterio(Criterio criterio){
+		
+		if (criterio.getNumeroCriterio() == null) {
+			validator.add(new ValidationMessage(
+					"É obrigatório informar o número da recomendação.",
+					"criteiro.numeroCriterio"));
+		} else if (	hasNumeroCriterioRecomendacaoAtual(criterio)) {
+			validator
+			.add(new ValidationMessage(
+					"Já existe um criteiro ativo com esse numero.",
+					"criteiro.numeroCriteiro"));
+		}else if (criterio.getPeso() == null) {
+			validator.add(new ValidationMessage(
+					"É obrigatório informar o peso do criteiro.",
+					"criteiro.peso"));
+		} else if (criterio.getPeso() <= 0 ) {
+			validator.add(new ValidationMessage(
+					"O peso do criteiro deve ser maior que 0.",
+					"criteiro.peso"));
+		} else if (criterio.getDescricao() == null) {
+			validator.add(new ValidationMessage(
+					"É obrigatório informar a descrição do criteiro.",
+					"criteiro.descricao"));
+		} else if (criterio.getTipoTeste().getId() == null) {
+			validator.add(new ValidationMessage(
+					"É obrigatório informar o Tipo de teste.",
+					"criterio.tipoTeste"));
+		}else{
+			criterio.setTipoTeste(new TipoTesteDAO(dao.getSession()).load(criterio.getTipoTeste().getId()));
+		}
+	}
 	/**
 	 * Recebe um critério antes de ser cadastrad0 e verifica se ja existe um
@@ -237,21 +228,32 @@ public class CriteriosController {
 	 */
 	private boolean hasNumeroCriterioRecomendacaoAtual(Criterio criterio) {
 		boolean retorno = false;
+		
+		if(	criterio.getDescricao() != null && 
+				criterio.getNumeroCriterio() != null && 
+				criterio.getPeso() != null && 
+				criterio.getRecomendacao() != null && 
+				criterio.getTipoTeste().getId() != null){
+			Recomendacao recomendacao = criterio.getRecomendacao();
-		List<Criterio> listaCadastrada = new RecomendacaoDAO(dao.getSession()).load(criterio.getRecomendacao().getId()).getCriterios();
-		for (Criterio criterioCadastrado : listaCadastrada) {
-			if (criterioCadastrado.getNumeroCriterio().longValue() == criterio
-					.getNumeroCriterio().longValue()) {
-				if (criterio.getId() != null) {
-					if (criterioCadastrado.getId().longValue() != criterio
-							.getId().longValue()) {
+			List<Criterio> listaCadastrada = dao.listAllAtivas(recomendacao);
+			
+			for (Criterio criterioCadastrado : listaCadastrada) {
+				if (criterioCadastrado.getNumeroCriterio().longValue() == criterio
+						.getNumeroCriterio().longValue()) {
+					if (criterio.getId() != null) {
+						if (criterioCadastrado.getId().longValue() != criterio
+								.getId().longValue()) {
+							return true;
+						}
+					} else {
 						return true;
 					}
-				} else {
-					return true;
 				}
 			}
+
 		}
+		
 		return retorno;
 	}
@@ -49,7 +49,7 @@ public class FaixaController {
 		return this.index(idCriterio, 1);
 	}
-	@Get("/conformidades")
+	@Get("/faixas")
 	@Restrito(descricao = "CONFORMIDADES_LISTA")
 	public List<Faixa> index(Long idCriterio, int pagina) {
 		CriterioDAO cDAO = new CriterioDAO(dao.getSession());
@@ -133,8 +133,8 @@ public class FaixaController {
 		for (Faixa faixa : faixas) {
 			faixa.setCriterio(criterio);
-			//criterioId = conformidade.getCriterio().getId();
-			validarConformidade(faixa);
+			//criterioId = faixa.getCriterio().getId();
+			validarFaixa(faixa);
 			validator.onErrorUsePageOf(FaixaController.class).add(
 					faixa.getCriterio().getId());
@@ -143,20 +143,21 @@ public class FaixaController {
 		criarNovoParametroAvaliacao(faixas, ParametroCalculoController.MODO_ADD);
-		result.include("mensagem", Messages.getString("conformidades.mensagem.salva"));
+		result.include("mensagem", Messages.getString("faixas.mensagem.salva"));
 		result.redirectTo(ParametroCalculoController.class).index(1);
 	}
-	@Put("/conformidades")
+	@Put("/faixas")
 	@Restrito(descricao = "CONFORMIDADES_ATUALIZAR")
 	public void atualizar(Faixa faixa) {
 		Faixa faixaCadastrada = dao.load(faixa.getId());
-		validarConformidade(faixa);
-
-		populateSelectLists();
+		Criterio criterio = daoCriterio.load(faixa.getCriterio().getId());
+		faixa.setCriterio(criterio);
+		
+		validarFaixa(faixa);
-		validator.onErrorUsePageOf(FaixaController.class).form(faixaCadastrada.getId());
+		validator.onErrorRedirectTo(FaixaController.class).form(faixaCadastrada.getId());
 		faixaCadastrada.setCriterio(faixa.getCriterio());
 		faixaCadastrada.setFaixaInicio(faixa.getFaixaInicio());
@@ -166,9 +167,8 @@ public class FaixaController {
 		faixas.add(faixaCadastrada);
 		criarNovoParametroAvaliacao(faixas, ParametroCalculoController.MODO_EDIT);
-			
-		result.include("mensagem",
-				Messages.getString("conformidades.mensagem.alterada"));
+		
+		result.include("mensagem",	Messages.getString("faixas.mensagem.alterada"));
 		result.redirectTo(ParametroCalculoController.class).index(1);
 	}
@@ -189,34 +189,38 @@ public class FaixaController {
 		result.include("faixaList", dao.getAllRecomendacao());
 	}
-	private void validarConformidade(Faixa conformidade) {
-		if (conformidade.getCriterio().getId() == null) {
+	private void validarFaixa(Faixa faixa) {
+		if (faixa.getCriterio().getId() == null) {
+			validator.add(new ValidationMessage(Messages
+					.getString("faixas.validacao.faixa"),
+					"faixa.diretriz.id"));
+		} else if (faixa.getFaixa() == null) {
 			validator.add(new ValidationMessage(Messages
-					.getString("conformidades.validacao.diretriz"),
-					"conformidade.diretriz.id"));
-		} else if (conformidade.getFaixa() == null) {
+					.getString("faixas.validacao.faixa"),
+					"faixa.faixa"));
+		} else if (faixa.getDescricao() == null) {
 			validator.add(new ValidationMessage(Messages
-					.getString("conformidades.validacao.faixa"),
-					"conformidade.faixa"));
-		} else if (conformidade.getDescricao() == null) {
+					.getString("faixas.validacao.descricao"),
+					"faixa.descricao"));
+		} else if (faixa.getFaixaInicio() == null) {
 			validator.add(new ValidationMessage(Messages
-					.getString("conformidades.validacao.descricao"),
-					"conformidade.descricao"));
-		} else if (conformidade.getFaixaInicio() == null) {
+					.getString("faixas.validacao.faixa_inicial"),
+					"faixa.faixaInicio"));
+		} else if (faixa.getFaixaFim() == null) {
 			validator.add(new ValidationMessage(Messages
-					.getString("conformidades.validacao.faixa_inicial"),
-					"conformidade.diretriz.id"));
-		} else if (conformidade.getFaixaFim() == null) {
+					.getString("faixas.validacao.faixa_final"),
+					"faixa.faixaFim"));
+		} else if (faixa.getFaixaInicio() > faixa.getFaixaFim()) {
 			validator.add(new ValidationMessage(Messages
-					.getString("conformidades.validacao.faixa_final"),
-					"conformidade.diretriz.id"));
-		} else if (conformidade.getFaixaInicio() > conformidade.getFaixaFim()) {
+					.getString("faixas.validacao.range_faixas"),
+					"faixa.faixaInicio"));
+		} else if (faixa.getPercentualConformidade() == null) {
 			validator.add(new ValidationMessage(Messages
-					.getString("conformidades.validacao.range_faixas"),
-					"conformidade.diretriz.id"));
+					.getString("faixas.validacao.percentual_conformidade"),
+					"faixa.percentualConformidade"));
 		}
 	}
-
+	
 	private void criarNovoParametroAvaliacao(List<Faixa> faixas, int tipo) {
 		ParametroCalculo ultimoParametroAvaliacao = paDAO.getLastParametrosCalculo();
 		boolean ok = false;
@@ -34,9 +34,11 @@ import br.com.eselo.component.Messages;
 import br.com.eselo.component.UsuarioWeb;
 import br.com.eselo.dao.ParametroCalculoDAO;
 import br.com.eselo.dao.RecomendacaoDAO;
+import br.com.eselo.dao.UsuarioDAO;
 import br.com.eselo.model.Criterio;
 import br.com.eselo.model.ParametroCalculo;
 import br.com.eselo.model.Recomendacao;
+import br.com.eselo.model.Usuario;
 @Resource
 public class ParametroCalculoController {
@@ -138,12 +140,13 @@ public class ParametroCalculoController {
 	@Restrito(descricao = "PARAMETRO_CALCULO_ATUALIZAR")
 	public void atualizar(ParametroCalculo parametroCalculo) {
-		ParametroCalculo parametroCalculoCadastrado = dao.load(parametroCalculo
-				.getId());
-		parametroCalculoCadastrado.setTsFim(parametroCalculo.getTsFim());
-		parametroCalculoCadastrado.setUsuario(parametroCalculo.getUsuario());
-		parametroCalculoCadastrado
-				.setDescricao(parametroCalculo.getDescricao());
+		List<Usuario> usuarios = dao.getAllUsuarios();
+		ParametroCalculo parametroCalculoCadastrado = dao.load(parametroCalculo.getId());
+		
+		parametroCalculo.setTsInicio(parametroCalculoCadastrado.getTsInicio());
+		parametroCalculoCadastrado.setUsuario(new UsuarioDAO(dao.getSession()).load(usuarioWeb.getId()));
+		parametroCalculo.setUsuario(parametroCalculoCadastrado.getUsuario());
+		parametroCalculoCadastrado.setDescricao(parametroCalculo.getDescricao());
 		if (parametroCalculo.getTsFim() != null) {
 			if (parametroCalculo.getTsFim().before(
@@ -159,14 +162,14 @@ public class ParametroCalculoController {
 					"parametroCalculo.descricao"));
 		}
-		// result.include("usuarioList", dao.getAllUsuarios());
-		// result.include("recomendacaoList", new
-		// RecomendacaoDAO(dao.getSession()).listAllAtivas(this.dao.load(parametroCalculo.getId())));
-		System.err.println(parametroCalculo.getId()
-				+ "Id do parametro de calculo");
-
-		validator.onErrorUsePageOf(ParametroCalculoController.class).form(
-				parametroCalculo.getId());
+		
+		
+		 result.include("usuarioList", usuarios);
+		 result.include("recomendacaoList", parametroCalculoCadastrado.getRecomendacoes());
+		 result.include("parametroCalculo", parametroCalculoCadastrado);
+		 
+				 
+		validator.onErrorUsePageOf(ParametroCalculoController.class).form(parametroCalculo.getId());
 		dao.saveOrUpdate(parametroCalculoCadastrado);
@@ -66,31 +66,8 @@ public class RecomendacaoController {
 	@Restrito(descricao = "RECOMENDACAO_ADICIONAR")
 	public void adicionar(Recomendacao recomendacao) {
-		if (recomendacao.getNumeroRecomendacao() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o número da recomendação.",
-					"recomendacao.numeroRecomendacao"));
-		} else if (hasNumeroRecomendacaoNoParametroAtual(recomendacao)) {
-			validator
-					.add(new ValidationMessage(
-							"Já existe uma recomendação ativa com esse numero. Recomendação não cadastrada.",
-							"recomendacao.numeroRecomendacao"));
-		} else if (recomendacao.getDescricao() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar a descrição da recomendação.",
-					"recomendacao.descricao"));
-		} else if (recomendacao.getPeso() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o peso da recomendação.",
-					"recomendacao.peso"));
-		} else if (recomendacao.getPeso() <= 0) {
-			validator.add(new ValidationMessage(
-					"O peso da recomendação deve ser maior que zero.",
-					"recomendacao.peso"));
-		}
+		validarRecomendacao(recomendacao);
-		result.include("parametroCalculoId", recomendacao.getParametroCalculo().getId());
-		result.include("parametroCalculo", recomendacao.getParametroCalculo());
 		validator.onErrorUsePageOf(RecomendacaoController.class).form(
 				recomendacao.getParametroCalculo().getId());
@@ -107,34 +84,12 @@ public class RecomendacaoController {
 		recomendacaoCadastrada.setNumeroRecomendacao(recomendacao.getNumeroRecomendacao());
 		recomendacaoCadastrada.setDescricao(recomendacao.getDescricao());
 		recomendacaoCadastrada.setPeso(recomendacao.getPeso());
+		recomendacao.setCriterios(recomendacaoCadastrada.getCriterios());
-		if (recomendacao.getNumeroRecomendacao() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o número da recomendação.",
-					"recomendacao.numeroRecomendacao"));
-		} else if (hasNumeroRecomendacaoNoParametroAtual(recomendacao)) {
-			validator
-					.add(new ValidationMessage(
-							"Já existe uma recomendação ativa com esse numero. Recomendação não atualizada.",
-							"recomendacao.numeroRecomendacao"));
-		} else if (recomendacao.getDescricao() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar a descrição da recomendação.",
-					"recomendacao.descricao"));
-		} else if (recomendacao.getPeso() == null) {
-			validator.add(new ValidationMessage(
-					"É obrigatório informar o peso da recomendação.",
-					"recomendacao.peso"));
-		} else if (recomendacao.getPeso() <= 0) {
-			validator.add(new ValidationMessage(
-					"O peso da recomendação deve ser maior que zero.",
-					"recomendacao.peso"));
-		}
-
-		result.include("parametroCalculoId", recomendacao.getParametroCalculo().getId());
-		result.include("parametroCalculo", recomendacao.getParametroCalculo());
-		validator.onErrorUsePageOf(ParametroCalculoController.class).form(
-				recomendacaoCadastrada.getParametroCalculo().getId());
+		validarRecomendacao(recomendacao);
+		
+		validator.onErrorUsePageOf(RecomendacaoController.class).form(
+				recomendacaoCadastrada.getId());
 		criarNovoParametroAvaliacao(recomendacao, ParametroCalculoController.MODO_EDIT);
@@ -194,6 +149,41 @@ public class RecomendacaoController {
 		new CopiadorParametroCalculo().criarNovoParametroCalculo(ultimoParametroAvaliacao, usuarioWeb, dao.getSession());
 		}
 	}
+	
+	
+	private void validarRecomendacao(Recomendacao recomendacao){
+		
+		
+		if (recomendacao.getNumeroRecomendacao() == null) {
+			validator.add(new ValidationMessage(
+					"É obrigatório informar o número da recomendação.",
+					"recomendacao.numeroRecomendacao"));
+		} else if (hasNumeroRecomendacaoNoParametroAtual(recomendacao)) {
+			validator
+					.add(new ValidationMessage(
+							"Já existe uma recomendação ativa com esse numero. Recomendação não atualizada.",
+							"recomendacao.numeroRecomendacao"));
+		} else if (recomendacao.getDescricao() == null) {
+			validator.add(new ValidationMessage(
+					"É obrigatório informar a descrição da recomendação.",
+					"recomendacao.descricao"));
+		} else if (recomendacao.getPeso() == null) {
+			validator.add(new ValidationMessage(
+					"É obrigatório informar o peso da recomendação.",
+					"recomendacao.peso"));
+		} else if (recomendacao.getPeso() <= 0) {
+			validator.add(new ValidationMessage(
+					"O peso da recomendação deve ser maior que zero.",
+					"recomendacao.peso"));
+		}
+		
+
+		result.include("parametroCalculoId", recomendacao.getParametroCalculo().getId());
+		result.include("parametroCalculo", recomendacao.getParametroCalculo());
+		
+	}
+	
+
 	/**
 	 * Recebe uma recomendação antes de ser cadastrada e verifica se ja existe uma
@@ -205,18 +195,22 @@ public class RecomendacaoController {
 	private boolean hasNumeroRecomendacaoNoParametroAtual(Recomendacao recomendacao) {
 		boolean retorno = false;
-		List<Recomendacao> listaCadastrada = dao.recomendacoesDoUltimoParametro(dao
-				.getLastParametrosCalculo());
-		for (Recomendacao recomendacaoCadastrada : listaCadastrada) {
-			if (recomendacaoCadastrada.getNumeroRecomendacao().longValue() == recomendacao
-					.getNumeroRecomendacao().longValue()) {
-				if (recomendacao.getId() != null) {
-					if (recomendacaoCadastrada.getId().longValue() != recomendacao
-							.getId().longValue()) {
+		if(recomendacao.getDescricao()!=null &&
+				recomendacao.getNumeroRecomendacao()!=null &&
+				recomendacao.getPeso()!=null ){
+			List<Recomendacao> listaCadastrada = dao.recomendacoesDoUltimoParametro(dao
+					.getLastParametrosCalculo());
+			for (Recomendacao recomendacaoCadastrada : listaCadastrada) {
+				if (recomendacaoCadastrada.getNumeroRecomendacao().longValue() == recomendacao
+						.getNumeroRecomendacao().longValue()) {
+					if (recomendacao.getId() != null) {
+						if (recomendacaoCadastrada.getId().longValue() != recomendacao
+								.getId().longValue()) {
+							return true;
+						}
+					} else {
 						return true;
 					}
-				} else {
-					return true;
 				}
 			}
 		}
@@ -91,7 +91,7 @@ public class Faixa {
 		this.criterio = criterio;
 	}
-	public double getPercentualConformidade() {
+	public Double getPercentualConformidade() {
 		return percentualConformidade;
 	}
	@@ -91,7 +91,7 @@ public class Faixa {		@@ -91,7 +91,7 @@ public class Faixa {
91	this.criterio = criterio;	91	this.criterio = criterio;
92	}	92	}
93		93
94	- public double getPercentualConformidade() {	94	+ public Double getPercentualConformidade() {
95	return percentualConformidade;	95	return percentualConformidade;
96	}	96	}
97		97