你写的是对的，应该是前面onclick调用函数的地方多了个分号

 function checkall(){

            var hobby = document.getElementsByTagName("input");

            for(var i=0;i<hobby.length;i++){

                hobby[i].checked = true;

            }

        }

        function clearall(){

            var hobby = document.getElementsByName("hobby");

            for(var i=0;i<hobby.length;i++){

                hobby[i].checked = false;

            }

        }