1.<meta charset=utf-8" />少写了一个双引号
2.if(node.nextSibling.nodeType=1)这里应该是双等号
3.

{ var e1=document.getElementsByTagName("h1");
    for(var i=0;i<e1.length;i++)
      {e2=nextElement(e1);//这里是循环，所以应该是e1[i]。e1是个对象集合，而不是单个dom节点，你的写法导致了nextElement(node)函数调用失败，因为这个函数的参数是一个dom节点
       e2.style.color="red";}
}

怎么写的这么复杂，这个明明就可以用css3选择器；

var h1s = document.getElementsByTagName("h1");
for (var i = 0, len = h1s.length; i < len; i++) {
    var nextNode = h1s[i].nextSibling;
    if (nextNode.nodeType !== 1) {
        nextNode = nextNode.nextSibling;
    }
    nextNode.style.color = "red";
    
};

楼主，你的if(node.nextSibling.nodeType=1),应该是==才对，而=是直接复制了，导致永远不会执行else语句