[Javascript] Condition van if statement doet stom

[Verwijderd]

Het moest er toch een keer van komen, starten met Javascript. En dan stuit onvermijdelijk op de meest stompzinnige foutjes, hieronder waarschijnlijk eentje daarvan.

Het probleem is dat het gewoon niet werkt.. er gebeurt niet wanneer ik de <span> beklik.

Ik weet bijna zeker dat het aan de conditie van de if ligt, aangezien hij met twee aparte functies (dan ook twee <span>'s met onclick natuurlijk) wel gewoon werkt.

JavaScript:

function wissel() 
if ( document.getElementById('nav').style.display = "";){
document.getElementById('nav').style.display = "none";
document.getElementById('wv').style.backgroundImage = 'url(../img1.jpg)';
}
else {
document.getElementById('nav').style.display = "";
document.getElementById('wv').style.backgroundImage = 'url(../img..jpg)';
}

HTML:

<span id="wv" onclick="wissel()">verdwijn/verschijn</span>
<div id="nav">Navigatie</div>

Nu heb ik dus geen idee waar ik in godsnaam op zou kunnen zoeken.. op Google, op GoT, dus dan maar een topic. Bvd.

[Maxxi]

Ik programmeer wel in Java nooit in java script, maar dit viel me op:
if ( document.getElementById('nav').style.display = ""->;){

Hier een ";" na de "" =

Hoort die daar wel?

[ Voor 15% gewijzigd door Maxxi op 22-04-2006 00:05 ]

[Verwijderd]

Was ook mijn twijfel, maar had ook geen resultaat.

[Maxxi]

if (time < 10)
{
document.write("<b>Good morning</b>")
}

Werkt wel,

if (time < 10;) // de ;
{
document.write("<b>Good morning</b>")
}

werkt niet
http://www.w3schools.com/js/tryit.asp?filename=tryjs_ifthen
http://www.w3schools.com/js/js_if_else.asp


EDIT:

En hier een foutje
document.getElementById('wv').style.backgroundImage = 'url(../img..jpg)';
(../img..jpg) // 2x .. ipv 1

[ Voor 44% gewijzigd door Maxxi op 22-04-2006 00:09 ]

[Erkens]

== ipv =

en die ; natuurlijk weg

[ Voor 56% gewijzigd door Erkens op 22-04-2006 00:07 ]

[plakbandrol]

inderdaad die eerste puntkomma hoort er niet

[Arjan]

die ; hoeft daar niet, maar is strikt genomen volgens mij niet fout.
Display zou echter nooit "" mogen zijn, maar slechts 'none', ' block', 'inline' of zelfs nog 'inline-block';
gebruik voor te debuggen ook firefox + javascript console, scheelt ontzettend veel tijd bij foutopsporen

[Maxxi]

Opgelost?

[twanvl]

Vergelijken moet met ==, niet met =. Die laatste is een toekening, en dat lijkt mij niet de bedoeling.

[Skaah]

Erkens schreef op zaterdag 22 april 2006 @ 00:07:
== ipv =

en die ; natuurlijk weg

Jup, want nu doe je een assigment, en een lege string geeft false; dus wordt het else {} blok uitgevoerd.

[Victor]

Atgast schreef op zaterdag 22 april 2006 @ 00:09:
die ; hoeft daar niet, maar is strikt genomen volgens mij niet fout.
Display zou echter nooit "" mogen zijn, maar slechts 'none', ' block', 'inline' of zelfs nog 'inline-block';
gebruik voor te debuggen ook firefox + javascript console, scheelt ontzettend veel tijd bij foutopsporen

Display kan in dit geval wel degelijk leeg zijn, als het nog niet als rule in een inline style is gedefinieerd. De TS kan dan ook beter checken op de computedstyle van het element, dan op basis van inline styles. Zie ook Quirksmode.org

[Verwijderd]

Maxxi schreef op zaterdag 22 april 2006 @ 00:10:
Opgelost?

Nee. Ik heb alles toegepast wat jullie zeiden, maar werken doet het niet..

Dit heb ik nu:

JavaScript:

function wissel() 
if ( document.getElementById('nav').style.display == "block"){
document.getElementById('nav').style.display = "none";
document.getElementById('wv').style.backgroundImage = 'url(../img1.jpg)';
}
else {
document.getElementById('nav').style.display = "block";
document.getElementById('wv').style.backgroundImage = 'url(../img..jpg)';
}

De Javascript Console van Firefox zegt overigens:

Fout
missing { before function body
Bronbestand http://127.0.0.1/
Regel 22
Broncode
if (document.getElementById('nav').style.display == "block"){

[ Voor 10% gewijzigd door Verwijderd op 22-04-2006 00:18 ]

[Victor]

Verwijderd schreef op zaterdag 22 april 2006 @ 00:16:
[...]

Nee. Ik heb alles toegepast wat jullie zeiden, maar werken doet het niet..

Dit heb ik nu:

JavaScript:

function wissel() 
if ( document.getElementById('nav').style.display == "block"){
document.getElementById('nav').style.display == "none";
document.getElementById('wv').style.backgroundImage == 'url(../img1.jpg)';
}
else {
document.getElementById('nav').style.display == "block";
document.getElementById('wv').style.backgroundImage == 'url(../img..jpg)';
}

De Javascript Console van Firefox zegt overigens:

Fout
missing { before function body
Bronbestand http://127.0.0.1/
Regel 22
Broncode
if (document.getElementById('nav').style.display == "block"){

Binnen je if conditie controleer je de waarde met ==, en als uit te voeren code gebruik je = om de waarde mee te zetten.

[Verwijderd]

Um, ja, dat had ik ook eigenlijk.. Foutje!

[Victor]

Verwijderd schreef op zaterdag 22 april 2006 @ 00:18:
Um, ja, dat had ik ook eigenlijk.. Foutje!

En even accolades om je functie zetten

[Verwijderd]

Nét, 1 seconde geleden kom ik daar ook achter. Wát een koe ben ik. Erg bedankt allemaal..

Ik vond het al allemaal zo onlogisch dat het niet werkte..

[Chester]

Je vergeet een accolade na function wissel(). Moet zijn

code:

Function wissel(){
  if ( document.getElementById('nav').style.display == "block"){
    document.getElementById('nav').style.display = "none";
    document.getElementById('wv').style.backgroundImage = 'url(../img1.jpg)';
  }
  else {
    document.getElementById('nav').style.display = "block";
    document.getElementById('wv').style.backgroundImage = 'url(../img..jpg)';
  }
}

spuit 11

[ Voor 30% gewijzigd door Chester op 22-04-2006 00:24 . Reden: Te langzaam met lezen/typen :P ]

[Verwijderd]

Wrijf het er maar in..

[XangadiX]

function (arguments) { statements }

* XangadiX wrijft het er ook even in