这个JavaScript我就是想不通，我明明有定义，为什么老报错没有定义！-PHP中文网问答

微信公众号讲师中心

首页

文章

web3.0 后端开发 web前端数据库开发工具 php框架常见问题科技 Java 系统教程电脑教程硬件教程手机教程软件教程游戏教程自媒体新闻

专题

后端开发 web前端数据库开发工具 php框架科技 Java 系统教程电脑教程硬件教程手机教程软件教程游戏教程新闻

AI工具

AI 聊天问答 Agent智能体 AI 文本写作 AI 绘画作图 AI 设计工具 AI 视频创作 AI 音频制作 AI 办公学习 AI 编程开发 Prompt指令

学习

大前端后端开发数据库移动端运维开发 UI设计计算机基础

编程手册

大前端 JavaScript 后端开发数据库移动端运维开发 UI设计计算机基础 XML Web Services

下载

js特效网站源码工具下载类库下载网站素材学习资源插件扩展手机/移动开发手机游戏

最近更新

搜索

web3.0 后端开发 web前端数据库开发工具 php框架常见问题科技 Java 系统教程电脑教程硬件教程手机教程软件教程

这个JavaScript我就是想不通，我明明有定义，为什么老报错没有定义！

大家讲道理 2017-04-10 15:58:58

[JavaScript讨论组]

757

<!doctype html>
<html lang="en">

<head>
    <meta charset="UTF-8">
    <title>Document</title>
    <style type="text/css">
        #p1,#p2,#p3 {
    width: 200px;
    height: 200px;
    background: red;
    margin: 10px;
}
    </style>
</head>

<body>
    <p id="p1"></p>
    <p id="p2"></p>
    <p id="p3"></p>
    <script type="text/javascript">
        var timer = null;
        op = document.getElementsByTagName("p");

        function test(i) {
            timer = setInterval(function() {
                op[i].style.height = op[i].offsetHeight + 2 + 'px'
            }, 30)
        };
        for (var i = 0; i < 3; i++) {
            op[i].onmouseover = function() {
                test(i);
            }
            op[i].onmouseout = function() {
                clearInterval(timer);
            }
        }
    </script>
</body>

</html>

大家讲道理

光阴似箭催人老，日月如移越少年。

全部回复(6)

巴扎黑2017-04-10 16:00:58 6楼

@小_秦已经指出问题了,也提供了解决方法。我再补充两种，其他朋友如果有其他方法的话不妨贴上来。

/*
        var timer = null;
        op = document.getElementsByTagName("p");

        for (var i = 0; i < 3; i++) {
            op[i].onmouseover = function() {
                var _this=this;
                timer = setInterval(function() {
                  _this.style.height = _this.offsetHeight + 2 + 'px'
            }, 30);
            //方法一：避免在闭包里面引用外包函数的变量，用this来代替
            }
            op[i].onmouseout = function() {
                clearInterval(timer);
            }
        }
    */

    //===================================================================

    /*
        var timer = null;
        op = document.getElementsByTagName("p");

        for (var i = 0; i < 3; i++) {
            op[i].onmouseover = function() {
                timer = setInterval(function() {
                  this.style.height = this.offsetHeight + 2 + 'px'
            }.bind(this), 30);
            //方法一改良
            }
            op[i].onmouseout = function() {
                clearInterval(timer);
            }
        }
    */

    //===================================================================

        var timer = null;
        op = document.getElementsByTagName("p");

        function test(i) {
            timer = setInterval(function() {
                op[i].style.height = op[i].offsetHeight + 2 + 'px'
            }, 30)
        };
        for (var i = 0; i < 3; i++) {
            op[i].onmouseover = function(num){
                return function() {
                    test(num);
                }
            }(i);
            //方法二：用iife来替换，事实上，就是用iife的形参作为桥梁在两个作用域之间传值
            op[i].onmouseout = function() {
                clearInterval(timer);
            }
        }

赞 +0

添加回复

伊谢尔伦2017-04-10 16:00:58 5楼

1.楼主最好去学习一下MARKDOWN怎么写。
2.这个是闭包陷阱

赞 +0

添加回复

ringa_lee2017-04-10 16:00:58 4楼

你可以使用闭包实验一下你这个for绑定事件肯定不对的，绑定的i都是3所以肯定op[3]都是不存在的，

赞 +0

添加回复

伊谢尔伦2017-04-10 16:00:58 3楼

1）
p1,#p2,#p3{width:200px;height:200px;background:red;margin:10px;}
==》
#p1,#p2,#p3{width:200px;height:200px;background:red;margin:10px;}
2）p，mouseover事件触发函数中的i为3,这里涉及到js中变量作用域
3) p的事件绑定推荐使用代理的方式-不用为每一个p绑定事件回调

赞 +0

添加回复

巴扎黑2017-04-10 16:00:58 2楼

timer ＝ null; 后面应该是逗号

var timer=null,
op=document.getElementsByTagName("p");

赞 +0

添加回复

伊谢尔伦2017-04-10 16:00:58 1楼

你声明变量时，要用var,如果是全局变量时函数内部不用var，儿最外层原始变量需要var的，这段代码，错误百出啊，以上楼层说了闭包的问题，就不多说了

赞 +0

添加回复

专题推荐

更多>

热门话题

热门教程

更多>