Kan geen javascript functie aanroepen - Softwareontwikkeling

zaterdag 27 september 2008 14:17

Acties:

Verwijderd

Topicstarter

Ik kan geen Javascript functies meer aanroepen.
Ik ben al een uur aan het zoeken in mijn code & google maar nergens iets gevonden (ook firebug al gebruikt, geen errors)

checkComment roep ik beneden aan. Ziet er iemand mijn fout?

code:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" lang="en">
    <head>
        <title>Margot De Ridder</title>
        <link rel="stylesheet" type="text/css" href="style.css" media="screen" />
        
        <script type="text/javascript">AC_FL_RunContent = 0;</script>
        <script type="text/javascript" src="AC_RunActiveContent.js"></script>
        
        <script type="text/javascript">
        <!--
            preload_image_object = new Image();
    
            images = new Array;
            images[0] = "images/background1.jpg";
            images[1] = "images/background2.jpg";
            images[2] = "images/background3.jpg";
            images[3] = "images/background4.jpg";
            images[4] = "images/background5.jpg";
    
            var i = 0;
            for(i=0; i<=4; i++){
                preload_image_object.src = images[i];
                if(i==4){
                    setTimeout("changeImage()",40000);
                }
            }
    
            var x = 0;
    
            function changeImage() {
                x+=1;
                if(x==5){ x=0; }
                element = document.getElementById("container");
                element.style.backgroundImage = "url(" + images[x] + ")";
                setTimeout("changeImage()",50000);
            }

            function checkComment() {
                alert('test');
                var correct = true;
                document.getElementById("error").innerHTML = "er zijn 1 of meerdere fouten gevonden:<ul>";
                if (document.comment.naam.value == "") {
                    correct = false;
                    document.getElementById("error").innerHTML += "<li>U moet een naaminvullen<\/li>";
                }
                if (document.comment.bericht.value == "" || document.comment.bericht.value == "Berichtje") {
                    correct = false;
                    document.getElementById("error").innerHTML += "<li>U moet een bericht invullen<\/li>";
                }
                if (document.comment.bericht.length() > 350) {
                    correct = false;
                    document.getElementById("error").innerHTML += "<li>Het bericht mag maximaal 350 tekens bevatten<\/li>";
                }
                document.getElementById("error").innerHTML += "<\/ul>";
                if(correct== true){
                    document.getElementById("error").innerHTML = "";}
                    document.comment.submit();
                }
            }
            function checkGast(){
                var correct = true;
                document.getElementById("error").innerHTML = "er zijn 1 of meerdere fouten gevonden<ul>";
                if (document.gastenboek.naam.value == "") {
                    correct = false;
                    document.getElementById("error").innerHTML += "<li>U moet een naaminvullen<\/li>";
                }
                if(tekst.email.value.indexOf("@") == -1 || tekst.email.value.indexOf("(at)") == -1 || tekst.email.value.indexOf(" at ") == -1){ 
                    correct= false;
                    document.getElementById("error").innerHTML += "<li>Ongeldig E-mailadres (ter beveiliging mag je @ vervangen door (at)<\/li>";
                }
                if (document.comment.bericht.value == "" || document.comment.bericht.value == "Berichtje") {
                    correct = false;
                    document.getElementById("error").innerHTML += "<li>U moet een bericht invullen<\/li>";
                }
                document.getElementById("error").innerHTML += "<\/ul>";
                if(correct){document.getElementById("error").innerHTML = "";}
                alert('test4');
                return correct; 
            }
        -->
        </script>

    </head>
    <body>
        <div id="container">
            <div id="menu">
<script language="javascript" type="text/javascript">
    if (AC_FL_RunContent == 0) {
        alert("This page requires AC_RunActiveContent.js.");
    } else {
        AC_FL_RunContent(
            'codebase', 'http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=9,0,0,0',
            'width', '220',
            'height', '220',
            'src', 'menu',
            'quality', 'high',
            'pluginspage', 'http://www.macromedia.com/go/getflashplayer',
            'align', 'middle',
            'play', 'true',
            'loop', 'true',
            'scale', 'showall',
            'wmode', 'transparent',
            'devicefont', 'false',
            'id', 'menu',
            'bgcolor', '#ffffff',
            'name', 'menu',
            'menu', 'true',
            'allowFullScreen', 'false',
            'allowScriptAccess','sameDomain',
            'movie', 'menu',
            'salign', ''
            ); //end AC code
    }
</script></div>
            <div id="main"><img src="images/nieuws_title.png" />
<div id="main_inhoud">
    <div id="error"></div>
    <form name="comment" method="post" action="index.php?page=home&amp;nieuws_id=3&amp;action=add">
        naam: <input type="text" name="naam" />
        <textarea rows="10" cols="22" name="bericht">Berichtje</textarea>
        <input type="button" name="send" value="toevoegen" onclick="checkComment()" />
    </form>
    <br /><a href="index.php?page=home">terug</a>   </div>

</div>
        </div>
    </body>
</html>

zaterdag 27 september 2008 14:27

Acties:

GlowMouse

code:

1	onclick="javascript:checkComment()"

zaterdag 27 september 2008 14:31

Acties:

Verwijderd

Topicstarter

GlowMouse schreef op zaterdag 27 september 2008 @ 14:27:
code:
1
onclick="javascript:checkComment()"

Ook zo werkt het niet, is dat trouwens verplicht? het heeft voor mij altijd zonder gewerkt. toch bedankt

zaterdag 27 september 2008 14:34

Acties:

GlowMouse

Zonder werkt het ook zie ik nu

of het verplicht is weet ik niet, maar je ziet het wel vaak staan.

Ik heb je code maar even zelf gedraaid, en de Foutconsole in Firefox (ctrl+shift+J) zie je gelijk de melding dat er een } teveel staat. Op regel 57 in jouw codeblok sluit je de if al af, en op regel 59 nog een keer.

zaterdag 27 september 2008 14:36

Acties:

Verwijderd

GlowMouse schreef op zaterdag 27 september 2008 @ 14:34:
Zonder werkt het ook overal, of het verplicht is, weet ik niet, maar je ziet het wel vaak staan.

Het is hartstikke fout. Dat het er vaak staat is omdat de meeste mensen het verkeerd hebben aangeleerd of simpelweg niet begrijpen wat ze doen.
In onclick en andere on-attributen hoort niet de javascript pseudo-scheme te staan. Het best kun je het helemaal nooit gebruiken.

[ Voor 15% gewijzigd door Verwijderd op 27-09-2008 14:37 ]

zaterdag 27 september 2008 14:43

Acties:

Verwijderd

Topicstarter

Dank u!
Met firebug vond ik dat niet.

zaterdag 27 september 2008 14:47

Acties:

NMe

Quia Ego Sic Dico.

Debuggen van je code mag je de volgende keer zelf doen. Hier had je prima zonder onze hulp uit kunnen komen; bij eigenaardigheden zoals zit is haakjes en puntkomma's tellen zo'n beetje het eerst dat je hoort te doen.

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.