javascript - onclick事件点击不起作用

Question

1 不知道为什么onclick事件不起作用了。。。
html
<p id="leftSide_xia" style="display:none;text-align:center;">

                    <ul class="" style="margin-top:10px;">
                    <!-- <li  style="height:52px;cursor:pointer;" onclick="openIllegalPage('isOnList')">wwwwwww</li>-->
                    <p onclick="openIllegalPage('websiteList')"><img name="leftSide_xia_menu"  onclick="openIllegalPage('websiteList')" src="/images/frame/icon_a.png" style="margin-left:35px;float:left;margin-top:6px;"/><li  onclick="openIllegalPage('websiteList')" style="height:30px;cursor:pointer;" ><a  onclick="openIllegalPage('websiteList')" style="margin-right:25%;">wwww览</a> </li><hr  style="border:0;background-color:rgb(8,30,44);height:2px;margin-top:0px;width:90%;"/>
                       </p>
                    <p onclick="openIllegalPage('websiteNameList')"><img name="leftSide_xia_menu" src="/images/frame/icon_a.png" style="margin-left:35px;float:left;margin-top:6px;"/><li  style="height:30px;cursor:pointer;" ><a style="margin-right:25%;">wwwwwwww</a></li><hr  style="border:0;background-color:rgb(8,30,44);height:1px;margin-top:0px ;width:90%;"/>
                       </p>
                    <p>
                        <img name="leftSide_xia_menu" src="/images/frame/icon_a.png" style="margin-left:35px;float:left;margin-top:6px;"/><li  style="height:30px;cursor:pointer;" onclick="openIllegalPage('recordNumberList')"><a style="margin-right:28%;">wwwwwwww/a></li></li><hr  style="border:0;background-color:rgb(8,30,44);height:1px;margin-top:0px ;width:90%;"/>
                    </p>
                    <p><img name="leftSide_xia_menu" src="/images/frame/icon_a.png" style="margin-left:35px;float:left;margin-top:6px;"/><li  style="height:30px;cursor:pointer;" onclick="openIllegalPage('unitNameList')"><a style="margin-right:25%;">wwwwww</a></li><hr  style="border:0;background-color:rgb(8,30,44);height:1px;margin-top:0px ;width:90%;"/>
                       </p>
                    <p><img name="leftSide_xia_menu" src="/images/frame/icon_a.png" style="margin-left:35px;float:left;margin-top:6px;"/><li  style="height:30px;cursor:pointer;" onclick="openIllegalPage('contactNameList')"><a style="margin-right:28%;">wwwwwwww</a></li><hr  style="border:0;background-color:rgb(8,30,44);height:2px;margin-top:0px ;width:90%;"/>
                       </p>
                    <p><img name="leftSide_xia_menu" src="/images/frame/icon_a.png" style="margin-left:35px;float:left;margin-top:6px;"/><li  style="height:30px;cursor:pointer;" onclick="openIllegalPage('contactNameList')"><a style="margin-right:20%;">wwwwww</a></li><hr  style="border:0;background-color:rgb(8,30,44);height:1px;margin-top:0px ;width:90%;"/>
                       </p>
                    <p><img name="leftSide_xia_menu" src="/images/frame/icon_a.png" style="margin-left:35px;float:left;margin-top:6px;"/><li  style="height:30px;cursor:pointer;" onclick="openIllegalPage('contactTelephoneList')"><a style="margin-right:10%;">wwwwwww</a></li><hr  style="border:0;background-color:rgb(8,30,44);height:1px;margin-top:0px ;width:90%;"/>
                </p>
                    </ul> 
                </p>
                

Answer 1

<p onclick="openIllegalPage('websiteList')">

function openIllegalPage(ppp,pageName){
    if(pageName=="websiteList")
}

注意一下参数

Answer 2

如果运行没有出错的话是会执行呢，你可以在openIllegalPage这方法alert() 看一下。我刚复制你代码看了一下是可以执行这函数的
你这养写多个点击事件它会冲突的，比如你点里面的元素，外面的元素都会执行。
我优化了一下，你可以参考一下：
html：
<ul class="" style="margin-top:10px;">

        <!-- <li  style="height:52px;cursor:pointer;" onclick="openIllegalPage('isOnList')">网站在线监察</li>-->
        <p>
            <img name="leftSide_xia_menu" src="/images/frame/icon_a.png" style="margin-left:35px;float:left;margin-top:6px;" />
            <li style="height:30px;cursor:pointer;">
                <a style="margin-right:25%;">不规范网站总览</a>
            </li>
            <hr style="border:0;background-color:rgb(8,30,44);height:2px;margin-top:0px;width:90%;" />
        </p>

</ul>

js:
window.onload = function() {

            var onUl = document.querySelector('ul');
            onUl.onclick = function(e) {
                /**
                 * e.target为事件源(也就是你点哪个就是哪个)
                 * 这时候你点哪个就是拿到哪个的节点了，
                 * 然后根据(e.target.tagName)判断
                 */
                console.log(e.target.tagName);
            }

        }

Answer 3

你在p上添加了点击事件，然后你还在p下的img添加点击事件，你这样就造成了参数的重复，要么你就加在img上就行了，手机端的不想敲代码。

Answer 4

实在不行断点调试

Answer 5

代码不忍直视