Usamos cookies para medir audiência e melhorar sua experiência. Você pode aceitar ou recusar a qualquer momento. Veja sobre o iMasters.
Fala rapaziada, gostaria de saber se essa sintaxe aqui está correta:
var s = url.searchParams.get("utm_source");
var c = url.searchParams.get("utm_campaign");
var m = url.searchParams.get("utm_medium");
if(s && c && m != null){
alert("ok");
}
Mais especificamente esse trecho:
if(s && c && m != null){
alert("ok");
}
O objetivo é saber se s,c e m são diferentes de null.
Está correta? Mudariam algo?
Desde já agradeço rapaziada, abração e fiquem com Deus.Boa @Omar~ , valeu pela orientação, removi o !=null e ficou melhor.
>
2 horas atrás, BrunoBit disse:
Boa @Omar~ , valeu pela orientação, removi o !=null e ficou melhor.
Só que você deve pensar o que vai vim a ser inserido na variável, e qual é o objetivo.
Se por exemplo e uma variável receber algum dado que não é o que você espera ela passa a ter true.
Então ela é válida mesmo não sendo o que você queria.
Ou seja se você espera por exemplo receber uma string em url.searchParams.get("utm_source")
E por ventura não receber a string mas a var é definida ela é true e será aceita.
Não quer dizer que é certo ou errado, e sim o que realmente deseja verificar.
Sim, nesse contexto só preciso que retorne true mesmo, funcionou perfeitamente.
Se você puder dar uma olhada no meu outro tópico, to com um possível problema de callback/async/sync no ajax, esse código completo ta aqui:
Ao meu ver está correto. Porque você está checando se todas essas variáveis são true ao mesmo tempo. Só removeria a questão != null porque tudo tem que ser true para entrar na condição.
Veja bem se a variável S for null ela não entra na requisição if(s porque assim você verifica se ela é true.
Em outras palavras isso quer dizer if(s && c && m != null){
Se S for true e C também for true e M não pode ser null.
Especificando o valor nulo ou não tudo vai dar o mesmo resultado.