if (x == false) vs. if (x = 'false')

Och vilket språk är detta?
Funkar det både om du sätter både x till false och true innan if-satsen? Jag skulle tro att det är skillnad. Rent generellt är = en tilldelning och == en jämförelse i de flesta språk. Dvs i ditt andra kodexempel så har x värdet 'false' efter din kod har körts. I Javascript finns även === för att göra en jämförelse.

if (x == false) { // Gör nåt }

if (x = 'false') { // Gör nåt }

x = 'false' är en tilldeldning, inte en jämförelse, och eftersom 'false' är ett "giltigt värde" (eller vad man ska kalla det, dvs det är inte 0, undefined eller en tom sträng) så kommer javascript i praktiken tolka det som true, och koden du har i if-blocket kommer att exekveras.

Men det blir ordentligt fel: du kan lika gärna skriva if (true) i ditt fall, för koden i if-blocket kommer alltid att köras. Dessutom får du följdfelet att x får värdet 'false', vilket knappast är önskvärt.

Det där är ett vanligt misstag i alla språk som inte kräver att värdet som if kontrollerar är en boolean. I språk som Java och C# är det inte tillåtet, och kompilatorn totalvägrar, vilket är bra.

@phz: Jag håller med, men lustigt nog har jag ofta sett rekommendationer att inte använda !, för att det anses mer svårläst/svårtolkat.

var nua = navigator.userAgent;
var is_android = ((nua.indexOf('Mozilla/5.0') > -1 && nua.indexOf('Android ') > -1 && nua.indexOf('AppleWebKit') > -1) && !(nua.indexOf('Chrome') > -1));

Det är en enorm skillnad på ett, två eller tre likamedtecken. = används för tilldelning, medan == och === används för jämförelse exkl. respektive inkl. jämförelse av datatypen.

Allt du skriver inom enkla eller dubbla ctitationstecken blir strängar, dvs. 'false' och "false" är strängar och false är ett booleskt värde.

Det här är anledningen till att folk inte ska använda Javascript för att lära sig programmera. Det är inte ditt fel OP utan det är språket som är så jävla sämst.

Precis. Det är inte bara en gång jag hittat fel i javascript orsakade av att === inte använts.

Ja, vad som är svårläst eller ej är en filosofisk fråga . I detta fall så är det till och med så lurigt att if (x == false) och if (!x) (och för den delen if (x === false)) inte beter sig på exakt samma sätt, visat i exempelvis den värdetabell som jag länkade ovan.

Personligen så ser jag generellt "lättläst" som ett uttryck för att kod är skriven på ett sätt som man kan förvänta sig i det givna språket, vilket inkluderar att använda språkets "best practices". Eftersom JavaScript har !-operatorn så skulle jag inte förvånas av att se den; snarare skulle jag fundera på varför den inte användes om jag såg en jämförelse med false.

Lägger man därtill att man i det närmaste bör kunna utgå ifrån att den som läser har någon form av syntaxuppmärkning i sin editor så låter argument som "svårläst" mest som argument i stil med "jag har inte riktigt lärt mig detta mer uppenbara sätt än, så jag är inte van vid att se det". Jag håller med dig (som höll med mig ) om att man bör använda ! där det är tänkt att vara applicerbart.

Det blir lite grumligare vatten när man ser personer använda exempelvis !!x som en "genväg" för att kasta x till en boolean enligt JavaScripts "truthy"/"falsy"-regler; i någon mån så förstår man vad som händer efter att ha sett det någon gång, men hade jag sett det i kod så hade jag tänkt både en och två gånger på hur situationen hade kunnat uppstå där en programmerare tänkte "detta är en bra lösning!" och sedan skrev så. Jag har sett personer som hävdat att det gör det tydligare att man medvetet använder en icke-boolean xsom en boolean om man skriver !!x, men jag känner mig inte övertygad .

Skriver mycket hellre JavaScript än t ex PHP: där har vi ett riktigt dåligt språk! (och standardbibliotek)

if(this.canSave) {
}

if(!this.invalid) {
}

Det är inte alltid man vill gå in i en sats bara för att en variabel är "sann", så man klarar sig inte utan ! bara genom att undvika "negativt döpta" variabler (invalid, incomplete, etc. — sedan finns också exempel på gränsfall: är empty positivt eller negativt uttryckt?)), om man inte förespråkar att skriva saker som:

if (valid) {
} else {
  // Gör något
}

// …men nog är det lättare att skriva/läsa:
if (!valid) {
  // Gör något
}

Eller du menar kanske att man i detta fall skulle låtit variabeln heta invalid (och tilldelats värde därefter) så att man hela tiden uttrycker det "positiva" villkor för den förgrening man önskar? Även då kan man tänka sig situationer då man skulle vilja testa sanningshalten åt ena hållet en gång och sedan åt andra hållet en annan gång. Dessutom kommer man aldrig ifrån inbyggda/externa funktioner som levererar en boolean där man inte själv nödvändigen har kontroll på definitionen, så ! är nog oundvikbar.

Som sidonotis så har Python löst det på ett rätt trevligt sätt med nyckelordet not, så att man skulle skriva:

if not valid:
    # Gör något

där ingen borde ha problem med tydligheten, kan tyckas .