JQuery采用CSS实现DOM元素的显示和隐藏

今天参加了一个Code Review活动,看了一个同事的写的代码,感觉满足了当前的功能实现,但是从长远来看,无论从代码复用角度还是维护上看都显得非常差。先看看需求是什么吧。需求是通过下拉框的选择不同的选项,来显示和隐藏一个文本框。同事是这样写的:

Code 1

<select  name="select" onChange="disinput(this)">
                <option value="1">1</option>
                <option value="2">2</option>
                </select>
<input type="text" id="text" name="text" style="" value="" />
  
<script type="text/javascript">
    function disinput(obj){
        if(obj.value==2){
            document.getElementById("text").style.display="none";
        }else{document.getElementById("text").style.display="";}
    }
</script>

这种写法应该是所有JS初学者都习惯的思维,根据需求写一个方法实现功能即可。但我觉得如果在另外一个页面也有类似的需求,那么是不是又要把这个方法重写一遍呢,哪一天如果用户要求选择选项1就在文本框中显示XX内容,选择2显示另外的内容,那是不是所有的页面都要重新改一遍,能不能有更好的方法。这个时候可以考虑用CSS,当然是外部CSS,这里我写了一个例子是将CSS放在页面文件中了,如果实际操作肯定是放在外部CSS当中。

Code 2

<html xmlns="http://www.w3.org/1999/xhtml">
<head runat="server">
    <title></title>
    <script type="text/javascript" src="jquery.js"></script>
    <script type="text/javascript">

        $(function () {           
        });

        function changeSel() {
            if ($("#select1").val() == 2) {
                $("#text").addClass("class2");
            } else {
                $("#text").removeClass("class2");
            }
        }
    </script>
    <style type="text/css">
        .class2
        {
           display:none;
        }
    </style>
</head>
<body>
    <select id="select1"  name="select" onChange="changeSel()">
                 <option value="1">1</option>
                 <option value="2">2</option>
                    </select>
    <input type="text" id="text" name="text" style="" value="" />
</body>
</html>

Code 2中的代码是否更加简洁,更重要的是它很好的实现了代码复用,以及未来的可维护性。建议大家编写代码的时候多从复用和可维护性上去考虑实现。

郑重声明:本站内容如果来自互联网及其他传播媒体,其版权均属原媒体及文章作者所有。转载目的在于传递更多信息及用于网络分享,并不代表本站赞同其观点和对其真实性负责,也不构成任何其他建议。