Se a intenção é realmente verificar se a string é vazia, então, sim, prejudica a leitura do código e, possivelmente, a execução do mesmo. O motivo é que a comparação feita em:
if (!value) {
console.log("String vazia");
}
não é uma verificação se a string é vazia, mas sim se o valor de value
é falsy, como comentado na própria pergunta. Isso implica que value
, além de não poder ser uma string vazia, não poderá ser o inteiro zero, falso, nulo, indefinido, NaN (not a number), etc. Porém, todos estes valores seriam esperados que passassem no teste "não é uma string vazia", pois não são, de fato, uma string vazia (exceto a string vazia propriamente dita). Quando faço if (!value)
, entenda-se que value
pode ser qualquer valor truthy e não somente uma string não vazia.
const testes = ["", false, null, undefined, NaN, [], 0, "Foo"];
for (let teste of testes) {
if (!teste) {
console.log("String vazia: " + teste);
} else {
console.log("Não é uma string vazia: " + teste);
}
}
Para, de fato, verificar se value
é uma string vazia, você terá que verificar o tipo da variável, seja utilizando o typeof
:
const testes = ["", false, null, undefined, NaN, [], 0, "Foo"];
for (let teste of testes) {
if (typeof teste === "string" && teste.length == 0) {
console.log("String vazia: " + teste);
} else {
console.log("Não é uma string vazia: " + teste);
}
}
Perceba que ao verificar o tipo fica explícito que o desejado é realmente buscar por strings vazias, logo, todos os outros valores são válidos.
Ou seja utilizando a comparação rígida, com o operador ===
, uma vez que ==
efetua uma comparação frouxa.
O que é uma comparação frouxa?
const testes = ["", false, null, undefined, NaN, [], 0, "Foo"];
for (let teste of testes) {
if (teste === "") {
console.log("String vazia: " + teste);
} else {
console.log("Não é uma string vazia: " + teste);
}
}
Novamente a verificação fica explícita sobre o seu objetivo.
É muito importante ressaltar que nulo é diferente de string vazia. A resposta do Maniero sobre isso mostra graficamente a diferença.
value
for 2, oif
será falso, mesmo que não seja uma string vazia. Lembrando que o JavaScript possui a comparação frouxa, a verificação de string vazia seriaif (value === "")
. Se prejudicar a leitura, é devido a condição não fazer exatamente o descrito. Faz sentido?