4

Tenho uma função para buscar um json localmente e alterar os dados de algumas fields. porém tenhoq ue fazer o teste se o campo está com o codigo de status e dependendo eu altero com o valor correto para visualização no frontend.

segue exemplo:

  $http.get("http://localteste:18888/tess").success(function (dados) {


         local_locations = $.map(dados, function (dev) {
            //Verifica Status da Bateria

            if (dev.status) {
                if (dev.status <= 10)
                    dev.status= "Muito Baixo";
               else if (dev.status<= 25)
                    dev.status = "Baixo";;
               else if ((dev.status== 50) || (dev.status>= 25))
                    dev.status= "Medio";;
              else  if ((dev.status== 75) || (dev.status>= 50))
                    dev.status= "Alto";;
               else if ((dev.status== 100) || (dev.status> 75))
                    dev.batUseBattery = "Muito alto";
       }
 });

Seria mais ou menos isso, porém há muitos testes como esse. Como eu poderia melhorar?

1
  • Qual o objetivo de reduzir o número de if? É apenas por causa da leitura? Em relação a complexidade algorítmica não existe qualquer problema.
    – Nbento
    23/09/2014 às 16:07

3 Respostas 3

11

Uma das coisas que eu não entendi no seu código é o motivo de estar usando duas comparações por if. Supostamente basta um teste em cada, pois se qualquer uma das comparações for verdadeira, você não chegará no else if seguinte (como já dito na resposta do @Wakim):

// O if externo pode ser removido se você tem certeza de ser numerico
   if (dev.status <= 10) {
      dev.batUseBattery = "Muito Baixo";
   } else if (dev.status <= 25) {
      dev.batUseBattery = "Baixo";
   } else if (dev.status <= 50) {
      dev.batUseBattery = "Medio";
   } else if (dev.status <= 75) {
      dev.batUseBattery = "Alto";
   } else { // não precisa comparar nada aqui, afinal ja é maior que 75
      dev.batUseBattery = "Muito alto";
   } 

Outra alternativa, caso tenha muitos valores, é o uso de um loop e um array para testar as faixas (mas na quantidade de opções do seu exemplo, eu ficaria com if...else mesmo).

dev.batUseBattery = "Muito baixo";
var niveis = [
    [10,'Baixo'], // Maior que 10 é "Baixo"
    [25,'Medio'], // Maior que 25 é "Medio" etc...
    [50,'Alto'],
    [75,'Muito alto'],
];
for (var i = 0; i < niveis.length; i++) {
   if( dev.Status > niveis[i][0] ) {
       dev.batUseBattery = niveis[i][1];
   }
}

Faça seus testes no JS Fiddle.

11
  • 1
    Bem legal essa segunda forma, +1.
    – Wakim
    23/09/2014 às 16:25
  • @Wakim a primeira, que é "quase um clone" da sua (devidamente "upada"), eu ainda acho melhor pra poucas opções, a segunda eu deixei pensando em um caso que a pessoa tivesse uma situação com bem mais opções.
    – Largato
    23/09/2014 às 16:27
  • 1
    eu cheguei a pensar em fazer um mapa para indexar com o status (se não tivesse o intervalo de 10 era simples), mas o seu é bem mais legível.
    – Wakim
    23/09/2014 às 16:29
  • Caraca. Mandou bem! A segunda opção ficaria inviável implantar agora. Mas muito obrigado pela observação! :D
    – peterq
    23/09/2014 às 16:37
  • 1
    Só por curiosidade, qual a relação de custo benefício entre as duas opções? O que seria menos "custoso" para o browser realizar, os IFs ou o FOR com IF interno + ARRAY ?
    – Fernando
    23/09/2014 às 16:43
4

Você pode reduzir esses if's verificando apenas o limite superior do intervalo, e fazendo proveito da ordem em que as condições são avaliadas para encaixar o valor no intervalo correto.

Cuidado porque se dev.status for 0, ele não vai classificar corretamente.

if ((typeof dev.status) !== 'undefined') {
    if (dev.status <= 10)
        dev.batUseBattery= "Muito Baixo";
    else if (dev.status <= 25)
        dev.batUseBattery = "Baixo";
    else if (dev.status <= 50)
        dev.batUseBattery= "Medio";
    else  if (dev.status <= 75)
        dev.batUseBattery= "Alto";
    else if (dev.status <= 100)
        dev.batUseBattery = "Muito alto";
}

Fiz um JSFiddle para que possa ver o resultado.

2

Acredito que não tem muito o que fazer. Pode retirar os ELSEs e fazer intervalos nos IFs mas sinceramente não vejo nenhum ganho de organização ou otimização.

if ((typeof dev.status) !== 'undefined') {
  if (dev.status <= 10)
    dev.batUseBattery= "Muito Baixo";
  if (dev.status > 10 && dev.status <= 25)
    dev.batUseBattery = "Baixo";
  if (dev.status > 25 && dev.status <= 50)
    dev.batUseBattery= "Medio";
  if (dev.status > 50 && dev.status <= 75)
    dev.batUseBattery= "Alto";
  if (dev.status > 75)
    dev.batUseBattery = "Muito alto";
}

Com switch não dá porque são intervalos e não valores exatos.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .