新手写的代码,请指出问题?

1.
Script全小写,language不需要使用
<Script language="JavaScript" type="text/javascript"> 
换成
<script type="text/javascript">2.
function borderco(obj) 
{t1.style.border-collapse=obj.value;} 这样的DOM选择方式不是很好
document.getElementById('t1');比较好
更好的方法
function $(e)
{return document.getElementById(e);}
$('t1')
3.
需要引号括起属性值,最好把样式外观放到css中
<table id=t1 border=1>
换成
<style type="css/text">
#t1{border:1px;}
</style>
<table id="t1">4.
<input type="button" name="se" value="separate" onclick="borderco("this")"> 
没有dom节点结束标记,没有脚本结束分号,this引号也不对
<input type="button" name="se" value="separate" onclick="borderco(this);"/> 
更好的方式
<button name="se" onclick="borderco(this);">separate</button>5.
<script type="text/javascript">
w3c严格的情况下这个标签要放到<head>标签里

解决方案 »

  1.   

    还真没有注意到这些细节.
    1:
    <Script language="JavaScript" type="text/javascript"> 
    <script type="text/javascript"> 
    两种方式没什么区别吧.默认是JavaScript,如果用vbScript要写上.
    2:
    一般用document.getElementById('t1');来获取对象
    4:
    <input type="button" name="se" value="separate" onclick="borderco(this);"/>
    onclick="borderco(this);"加不加分号无所谓吧??
      

  2.   

    参数是string还是HTML Object?
      

  3.   

    <input type="button" name="se" value="separate"onclick="borderco(this)">
      

  4.   


    对
    这个也是错误的t1.style.border-collapse不能直接使用Css属性.应该用其对应的js转义属性.t1.style.borderCollapse这段代码里汇聚了很多典型问题.O
      

  5.   

    <Script type="text/javascript">
    //<![CDATA[
    function borderco(obj)
    {t1.style.borderCollapse=obj.value;}
    //]]>
    </Script>
    <table id="t1" border="1">
    <tr> <td>EST </td> <td>9:00 A.M. </td> </tr>
    <tr> <td>CST </td> <td>8:00 A.M. </td> </tr>
    <tr> <td>PST </td> <td>6:00 A.M. </td> </tr>
    </table>
    <input type="button" name="se" value="separate" onclick="borderco(this)">
    <input type="button" name="co" value="collapse" onclick="borderco(this)">maybe............