1. Advertising
    y u no do it?

    Advertising (learn more)

    Advertise virtually anything here, with CPM banner ads, CPM email ads and CPC contextual links. You can target relevant areas of the site and show ads based on geographical location of the user if you wish.

    Starts at just $1 per CPM or $0.10 per CPC.

Novice: can't find syntax error

Discussion in 'JavaScript' started by pitto, Jun 21, 2008.

  1. #1
    Hi,

    I know next to nothing about Javascript but I recently designed a site with Dreamweaver (big mistake) and it used Javascript to do a few things with menus and stuff. All said and done the site displays ok on my mac but there is a syntax error according to firebug in firefox and the syntax error is preventing IE from displaying the page at all. Since I know zero about Javascript and Dreamweaver doesn't seem to notice the syntax problem I was hoping someone would be able to do a quick scan of the following code and let me know what the problem is. I'd really appreciate it.

    
    <script type="text/JavaScript"> function MM_swapImgRestore() {
    //v3.0 var i,x,a=document.MM_sr;
    for(i=0;a&&i<a.length&&(x=a[i])&&x.oSrc;i++) x.src=x.oSrc; }
    
    function MM_preloadImages() { //v3.0 var d=document; if(d.images){
    if(!d.MM_p) d.MM_p=new Array(); var
    i,j=d.MM_p.length,a=MM_preloadImages.arguments; for(i=0; i<a.length;
    i++) if (a[i].indexOf("#")!=0){ d.MM_p[j]=new Image;
    d.MM_p[j++].src=a[i];}} }
    
    function MM_findObj(n, d) { //v4.01 var p,i,x;  if(!d) d=document;
    if((p=n.indexOf("?"))>0&&parent.frames.length) {
    d=parent.frames[n.substring(p+1)].document; n=n.substring(0,p);}
    if(!(x=d[n])&&d.all) x=d.all[n]; for (i=0;!x&&i<d.forms.length;i++)
    x=d.forms[i][n]; for(i=0;!x&&d.layers&&i<d.layers.length;i++)
    x=MM_findObj(n,d.layers[i].document); if(!x && d.getElementById)
    x=d.getElementById(n); return x; }
    
    function MM_swapImage() { //v3.0 var i,j=0,x,a=MM_swapImage.arguments;
    document.MM_sr=new Array; for(i=0;i<(a.length-2);i+=3) if
    ((x=MM_findObj(a[i]))!=null){document.MM_sr[j++]=x; if(!x.oSrc)
    x.oSrc=x.src; x.src=a[i+2];} }
    
    // CSS li:hover effect for IE6 function ieHover { var ieEls =
    document.getElementById("navigation").getElementsByTagName("LI"); for
    (var i=0; i<ieEls.length; i++) { ieEls[i].onmouseover=function() {
    this.className+=" iehover"; } ieEls[i].onmouseout=function() {
    this.className=this.className.replace(new RegExp(" iehover\\b"), ""); }
    } } if (window.attachEvent) window.attachEvent("onload", ieHover); //-->
    </script>
    
    Code (markup):
    Thanks again.
     
    pitto, Jun 21, 2008 IP
  2. Vooler

    Vooler Well-Known Member

    Messages:
    1,146
    Likes Received:
    64
    Best Answers:
    4
    Trophy Points:
    150
    #2
    OMG, the code was generated by DreamWeaver? this is so I mean messed up and unorganized code.
    FireFox or IE altest may be reporting the line number, it is hard to read all the code.

    regards
     
    Vooler, Jun 22, 2008 IP