Compare two values with jquery

Question

Compare two values with jquery

Asked 7 years, 2 months ago

Viewed 1,086 times

1

Like I do to compare two variables. I have a variable that brings the neighborhood id, called bairrocadastrado, it returns the neighborhood ID equal 1093. Soon I have a foreach that returns all the neighborhoods registered in the system, type bairro_id 995, 1093 etc. Only that the problem is as follows. It finds the ID. It makes the comparison, but always returns the neighborhoods that are not equal, ie the second IF, and are equal. What could be wrong?

  function areasAtendimento()
  {
    // inicia as variaveis
    var id_atendimento = "";
    var id_bairro = "";
    var nomebairro = "";
    var nomecidade = "";
    var nomeestado = "";
    var valorfrete = "";
    var parseResult = "";
    var cidadecadastrada = "";
    var bairrocadastrado = "";
    var resultBairro = "";

    // pegar os itens do session storage
    var getEnderecoLogado = sessionStorage.getItem("dadosendereco");

    $.ajax({
      url: urlBase + "areaatendimento",
      method: 'GET',
      success: function (retorno)
      {

        parseResult = JSON.parse(getEnderecoLogado);
        bairrocadastrado = parseResult[parseResult.length-1].bairroendereco;

        retorno.data.forEach(function (item)
        {
          id_atendimento = item.area_atendimento_id;
          id_bairro = item.bairro.bairro_id;
          nomecidade = item.bairro.municipio.nome;
          nomebairro = item.bairro.nome;
          nomeestado = item.bairro.municipio.estado.nome;
          valorfrete = item.valor_frete;
          valorfrete = valorfrete.replace('.', ',');
          // bairro igual ao que está cadastrado
          if (id_bairro == bairrocadastrado)
          {
            $('.entregatotal').html("R$" + valorfrete);
            $('.bt-finalizar-01 .finalizar-compra-carrinho').css('display', 'block');
          }
          // bairro diferete, não exibe o botão
          if (id_bairro != bairrocadastrado)
          {
            $('.entregatotal').html("R$ 0,00");
            $('.bt-finalizar-01 .finalizar-compra-carrinho').css('display', 'none');
            $('.nao-atendido').css('display','block');
          }
        })
      },
      error: function (XMLHttpRequest, textStatus, errorThrown)
      {
        console.log('Erro');
      }
    });
  }

1

Javascript operators need to be more specific for ifs, use exactly the same '==' and '!=='

– AnthraxisBR

2017/09/19 at 14:21
What gives console.log(typeof id_bairro, id_bairro, typeof bairrocadastrado, bairrocadastrado);?

– Sergio

2017/09/19 at 14:22
Felipe, did you see my question ? if you don’t know use the console make Alert 1 to 1 and answer here what gives you.

– Sergio

2017/09/19 at 14:53
Yes, the problem is that it returns all the registered neighborhood ids and at the time of comparing makes the two ifs. Compare what exists and then the ones that don’t exist saying it’s different. More details in the footer here of the discussion.

– Felipe Michael da Fonseca

2017/09/19 at 14:58
Which of the variables is an array?

– Sergio

2017/09/19 at 15:00
bairrocadastrado is a sessionstorage array that takes the id of the neighborhood registered by the client that is 10813. Logo id_neighborhood are all registered neighborhoods that come within ajax, ie within the foreach. In this variable also comes the id of the neighborhood 10813

– Felipe Michael da Fonseca

2017/09/19 at 15:02
And you want to know ifid_bairro exists in bairrocadastrado, right? What is the structure of bairrocadastrado? Is an array of strings/numbers?

– Sergio

2017/09/19 at 15:08
registered neighborhood comes from this sessionStorage being as number: Takes the last of the array [{"idcliente":"ecca2c6b-cd15-443b-b4db-5a15d39196c5","cliente_endereco_id":"69b8626d-c8d0-44a0-933a-e38e1a2737cd","cependereco":"89400000","logradouroendereco":"Rua Jaime Correia Pereira","numeroendereco":"90","complementoendereco":"Casa","referencia":"Centro Comunitário","estadoendereco":"Santa Catarina","cidadeendereco":"Porto União","bairroendereco":10813}]

– Felipe Michael da Fonseca

2017/09/19 at 16:42
Ok, and in that array the value you want to compare is bairroendereco? is to check whether id_bairro is equal to "bairroendereco":10813?

– Sergio

2017/09/19 at 17:40
yes that’s right. Only I got into the loop and it does not come out of it. It takes all who are and those who do not and compares.kkk

– Felipe Michael da Fonseca

2017/09/19 at 17:46
Felipe, usa @ before the name for me to receive a notification, otherwise it will be hours before you see your comment :) Okay, then one last question: this array bairrocadastrado may have more than one object inside? or always comes with 1 object inside the array?

– Sergio

2017/09/19 at 19:36
@Sergio, only 1 object the neighborhood id

– Felipe Michael da Fonseca

2017/09/19 at 20:10
@Felipemichaeldafonseca ok, it would be case then you have bairrocadastrado = parseResult[parseResult.length-1].bairroendereco[0].bairroendereco? It’s quite long but it seems to me the minimum change to do in the code.

– Sergio

2017/09/19 at 20:17
yes @Sergio, but that would solve the IF problem?

– Felipe Michael da Fonseca

2017/09/19 at 20:20
Yes, if is right. The problem is the variable bairrocadastrado not getting what you expected. If bairrocadastrado is right the if will work. You could have if (id_bairro == bairrocadastrado) and below else in time of if (id_bairro != bairrocadastrado). It was a little better for reading the code, but it’s not wrong and the problem is not the != == in this case.

– Sergio

2017/09/19 at 20:22
sorry @Sergio, but returns the following error: Cannot read Property 'bairroendereco' of Undefined

– Felipe Michael da Fonseca

2017/09/19 at 20:27
Okay, going back a step. Put here the value of console.log(typeof getEnderecoLogado, getEnderecoLogado); without JSON.parse, right after var getEnderecoLogado =.

– Sergio

2017/09/19 at 20:29
actually @Sergio if it works only I wanted it to show if the shipping of the bairroendereco is equal to the foreach id. But then he returns all those who are different too, because he is different understood? I believe the error is there, we just do not know how to make it return the correct values and to stop it in the first if it is equal and do not execute the second.

– Felipe Michael da Fonseca

2017/09/19 at 20:30
@Sergio he returns: string [{"idcliente":"ecca2c6b-cd15-443b-b4db-5a15d39196c5","cliente_endereco_id":"69b8626d-c8d0-44a0-933a-e38e1a2737cd","cependereco":"89400000","logradouroendereco":"Rua Jaime Correia Pereira","numeroendereco":"90","complementoendereco":"Casa","referencia":"Centro Comunitário","estadoendereco":"Santa Catarina","cidadeendereco":"Porto União","bairroendereco":10813}]

– Felipe Michael da Fonseca

2017/09/19 at 20:32
As for what comes from getEnderecoLogado the correct way is bairrocadastrado = parseResult[0].bairroendereco;, since it is only 1. In relation to "it returns all that are different too" and "we do not know how to make it return the correct values" I still don’t understand what you mean. Refer to retorno ajax or sessionStorage?

– Sergio

2017/09/19 at 20:40

Show 15 more comments

2 answers

Browser other questions tagged javascript jquery

You are not signed in. Login or sign up in order to post.

by Lucas Souza • **170** points · Answer 1 · 2017-09-19T14:24:18+00:00

1

The second IF is replacing the first. Try changing to a ELSE or ELSE IF, like this:

// bairro igual ao que está cadastrado
      if (id_bairro == bairrocadastrado)
      {
        $('.entregatotal').html("R$" + valorfrete);
        $('.bt-finalizar-01 .finalizar-compra-carrinho').css('display', 'block');
      } 
      else if (id_bairro != bairrocadastrado)
      {
        $('.entregatotal').html("R$ 0,00");
        $('.bt-finalizar-01 .finalizar-compra-carrinho').css('display', 'none');
        $('.nao-atendido').css('display','block');
      }

"The second IF is replacing the first" ? What do you mean? You can give an example how this can happen?

– Sergio

2017/09/19 at 14:25
What he said makes sense, because first, he sees 'if neighborhood equal' and then he has run 'if different neighborhood' within the same loop, the second if will be the one that will be run last, it may be that even fill wrong.

– AnthraxisBR

2017/09/19 at 14:34
He always makes the second IF. That is, he finds everyone and if it is different as he finds everyone who they are, he already makes the second. The case is that you are finding all the same and all the different. But I wanted only the same.

– Felipe Michael da Fonseca

2017/09/19 at 14:35
You have the same class for both Ifs, so if he enters the first and then the second.. The second will prevail, you putting the ELSE or ELSE IF, it will get into one or the other...

– Lucas Souza

2017/09/19 at 14:36
How would you handle it then? Have any suggestions?

– Felipe Michael da Fonseca

2017/09/19 at 14:37
@Felipemichaeldafonseca , you have nothing telling the loop to stop the time you find the same, or you use an Else indicating exception, or tell the loop to stop the time you find the same. a Return 0 is enough

– AnthraxisBR

2017/09/19 at 14:37
I don’t understand. Could you explain me? I’m a beginner in Javascript and jquery

– Felipe Michael da Fonseca

2017/09/19 at 14:39
I tried with Return false and came up with nothing in this situation. It would not be better to make an accountant?

– Felipe Michael da Fonseca

2017/09/19 at 14:48
@Felipemichaeldafonseca puts the name of the different classes, as if to create two div even, maybe already solved... Type $('.total delivery'). html("R$" + value); and in ELSE $('.total deliveryNaoCad'). html("R$" + value);... But change from the other fields tbm.. And then you deal with show(show) or Hidden(hide) you want to display... I think it would be easier

– Lucas Souza

2017/09/19 at 14:48
No valid numbers in format String or Number which give the same result in != and ==. Hence my -1.

– Sergio

2017/09/19 at 14:59

Show 5 more comments

by AnthraxisBR • **4,361** points · Answer 2 · 2017-09-19T15:02:37+00:00

Try to modify this part:

if (id_bairro === bairrocadastrado)
{
    $('.entregatotal').html("R$" + valorfrete);
    $('.bt-finalizar-01 .finalizar-compra-carrinho').css('display', 'block');
    return false;
} else
{
    $('.entregatotal').html("R$ 0,00");
    $('.bt-finalizar-01 .finalizar-compra-carrinho').css('display', 'none');
    $('.nao-atendido').css('display','block');
}