jeśli instrukcja w javascript zawsze jest prawdziwa

Question

Tak, mam kod, nie jest zrobiony, ale wszystko, co chcę zrobić, to wyświetlenie jednego pola alertu, jeśli napiszę słowo "pomoc", i powiedz coś innego, jeśli cokolwiek innego zostanie wprowadzone .

function prompter() { 
var reply = prompt("This script is made to help you learn about new bands, to view more info, type help, otherwise, just click OK") 
if (reply === 'help' || 'Help') 
    { 
    alert("This script helps you find new bands. It was originally written in Python 3.0.1, using Komodo IDE, but was then manually translated into Javascript. Please answer the questions honestly. If you have no opinion on a question, merely press OK without typing anything.") 
    } 
else 
    { 
    alert("Press OK to continue") 
    } 
}; 

, ale co się dzieje, nie ważne co, pojawi się pierwsze okno alertu, nawet jeśli naciśniesz przycisk anuluj! Jak to naprawić?

Answer 1

if (reply === 'help' || 'Help') 

powinno być:

if (reply === 'help' || reply === 'Help') 

od 'Help' jest "truthy", a więc pierwsza część if zawsze będą wprowadzone.

Oczywiście jeszcze lepiej byłoby zrobić bez uwzględniania wielkości liter porównania:

if (reply.toLowerCase() === 'help') 

Przykład:http://jsfiddle.net/qvEPe/

Answer 2

Powodem dlaczego to zawsze wyskakuje, że reply === 'help' || 'Help' ocenia jako (reply === 'Help') || ('Help'). Ciąg literalny Help jest zawsze zgodny z prawdą w JavaScript, więc zawsze ocenia to jako prawdę.

Aby to naprawić trzeba porównać reply obu wartości

if (reply === 'help' || reply === 'Help') { 
    ... 
} 

Albo jeśli chcesz żadnym wypadku wariant wykorzystania pomocy regex

if (reply.match(/^help$/i)) { 
    ... 
} 

Answer 3

Problemem jest tutaj:

if (reply === 'help' || 'Help') // <-- 'Help' evaluates to TRUE 
           //  so condition is always TRUE 

Operator równości nie "rozpowszechnia", spróbuj

if (reply === 'help' || reply === 'Help') 

Answer 4

Wystarczy zmienić: if (reply === 'help' || 'Help')

do tego: if (reply === 'help' || reply === 'Help')

Oświadczenie or nie porównując zmienną.

Answer 5

Problemem jest to linia:

if (reply === 'help' || 'Help') 

Ponieważ w JavaScript, obiekty i niepuste ciągi ocenić wartość true, gdy stosowany jako wartość logiczną. Istnieje kilka wyjątków od tego przy użyciu ==

if("0") // true 
if("0" == true) // false 

W ogóle, to nie jest dobry pomysł, aby użyć == lub surowych zmiennych w instrukcji if.

Jak inni zwrócili uwagę, należy

if (reply === 'help' || reply === 'Help') 

albo lepiej:

if (typeof reply === 'string' && reply.toLowerCase() === 'help') 

zamiast.