Erro ao inicializar cores nos dados de uma Datatable utilizando HashMap

Cara, existe sim:
Ao invés de fazer isso aqui: r.getSituacao().equalsIgnoreCase(“Cliente está no hotel”)
faça apenas
private static final String CLIENTE_NO_HOTEL = “CLIENTE_NO_HOTEL”; // salve assim no DB
r.getSituacao().equals(CLIENTE_NO_HOTEL);// a comparação fica objetiva, direta e com menos chance de erro.

No seu managedBean, crie métodos tipo

public boolean isClienteNoHotel(){

return r.getSituacao().equals(CLIENTE_NO_HOTEL);

}

Com isso, você teria um IF para cada caso, e ficaria mais fácil de refatorar depois.

jakefrog:

Cara, existe sim:
Ao invés de fazer isso aqui: r.getSituacao().equalsIgnoreCase(“Cliente está no hotel”)
faça apenas
private static final String CLIENTE_NO_HOTEL = “CLIENTE_NO_HOTEL”; // salve assim no DB
r.getSituacao().equals(CLIENTE_NO_HOTEL);// a comparação fica objetiva, direta e com menos chance de erro.

No seu managedBean, crie métodos tipo

public boolean isClienteNoHotel(){

return r.getSituacao().equals(CLIENTE_NO_HOTEL);

}

Com isso, você teria um IF para cada caso, e ficaria mais fácil de refatorar depois.</blockquote>

Bom segui seu conselho e andou bastante, porém ele está dando o seguitne erro, ele acaba repetindo todos os dados na mesma coluna e linha com cada valor de uma cor diferente. Quebrei o método inicicial em 5 pra cada situação.

Ele tem que mostrar algo assim +ou-: na linha 1 tem 4 dados cada um em uma coluna separada no meu caso os dias do mes, dependendo da situação esse dado pode aparecer azul, vermelho, verde ou preto segue a alteraão q fiz na pagina

<h:outputText value="#{orcamentoBean.valores[cat][mes]}" style="color:Green" rendered="#{orcamentoBean.clienteNoHotel == 0}" >  
                            <f:attribute name="cat" value="#{cat}"/>  
                            <f:attribute name="mes" value="#{mes}"/>  
                        </h:outputText>  
                        <h:outputText value="#{orcamentoBean.valores[cat][mes]}" style="color:#FFD700" rendered="#{orcamentoBean.reservaConfirmada == 1}" >  
                            <f:attribute name="cat" value="#{cat}"/>  
                            <f:attribute name="mes" value="#{mes}"/>  
                        </h:outputText>  
                        <h:outputText value="#{orcamentoBean.valores[cat][mes]}" style="color:Red" rendered="#{orcamentoBean.reservaNaoConfirmada == 2}" >  
                            <f:attribute name="cat" value="#{cat}"/>  
                            <f:attribute name="mes" value="#{mes}"/>  
                        </h:outputText>  
                        <h:outputText value="#{orcamentoBean.valores[cat][mes]}" style="color:Blue" rendered="#{orcamentoBean.checkOut == 3}" >  
                            <f:attribute name="cat" value="#{cat}"/>  
                            <f:attribute name="mes" value="#{mes}"/>  
                        </h:outputText>  
                        <h:outputText value="#{orcamentoBean.valores[cat][mes]}" rendered="#{orcamentoBean.noShow == 4}" >  
                            <f:attribute name="cat" value="#{cat}"/>  
                            <f:attribute name="mes" value="#{mes}"/>  
                        </h:outputText>

C não fez igual eu falei! =P

#{orcamentoBean.clienteNoHotel == 0}

eu fiz com o método retornando boolean, deveria ser apenas:
#{orcamentoBean.clienteNoHotel}

Você não precisa fazer == na sua página se seu método retornar um boolean.

jakefrog:

C não fez igual eu falei! =P

#{orcamentoBean.clienteNoHotel == 0}

eu fiz com o método retornando boolean, deveria ser apenas:
#{orcamentoBean.clienteNoHotel}

Você não precisa fazer == na sua página se seu método retornar um boolean.

Opa erro meu esqueci de alterar o metodo pra boolean, agora está alterado e mesmo assim ele continua repetindo os dados na mesma coluna, quando deixo apenas 1 output na página ele preenche certo com aquela cor porem quando coloco as outras ele repete os dados com as 4 cores

Segue o metodo que fiz separadamente:

@Transient
	public boolean isEstaNoHotel(){		
		for (Reservas r: listaReserva) {
			if(r.getSituacao().equals(CLIENTE_NO_HOTEL)){	
				return r.getSituacao().equals(CLIENTE_NO_HOTEL);
			}
		}		
	
		return false;
	}
	
	@Transient
	public boolean isReservaConfirmada(){
		for (Reservas r: listaReserva) {
			if(r.getSituacao().equals(RESERVA_CONFIRMADA)){
				return r.getSituacao().equals(RESERVA_CONFIRMADA);
			}
		}
		return false;
	}
	
	@Transient
	public boolean isReservaNaoConfirmada(){
		for (Reservas r: listaReserva) {
			if(r.getSituacao().equals(RESERVA_FALTANDO_DEPOSITO)){
				y = 3;
				return r.getSituacao().equals(RESERVA_FALTANDO_DEPOSITO);
			}
		}		
	
		return false;
	}
	
	@Transient
	public boolean isCheckOutFeito(){
		for (Reservas r: listaReserva) {
			if(reserva.getSituacao().equals(CHECKOUT)){
				return reserva.getSituacao().equals(CHECKOUT);
			}
		}		
	
		return false;
	}
	
	@Transient
	public boolean isNoShow(){		
		for (Reservas r: listaReserva) {
			if(r.getSituacao().equals(NO_SHOW)){
				return r.getSituacao().equals(NO_SHOW);
			}
		}		
	
		return false;
	}

Tentei alterar pra true o return e o erro persiste também.

Pq o @Transient?

O mesmo cara que você está persistindo no banco é o mesmo cara que você declarou como managedBean?

jakefrog:

Pq o @Transient?

O mesmo cara que você está persistindo no banco é o mesmo cara que você declarou como managedBean?

Sim é o mesmo cara. Desculpa a demora, to procurando alguma solução aqui pois creio que o erro ta sendo de lógica no método já que to notando que ele qnd vai pra página entra mais de uma vez no método