2016-03-24 6 views
1

私は2つの それのボタンと、この関数の最後の3行は何をすべきかthis複数の変数と.appendChild(内のテキストは)

function insertRow (label) { 

    var list = document.getElementById('list'); 
    var row = list.insertRow(); 
    var cell1 = row.insertCell(0); 
    var slash = document.createTextNode('/'); 
    var upButton = document.createElement('button'); 
    var downButton= document.createElement('button'); 

      upButton.innerHTML = "Up"; 
      downButton.innerHTML = "Down"; 

    cell1.style.borderBottom = "2px solid black"; 
    cell1.style.borderLeft = "2px solid black"; 

    cell1.appendChild(upButton); 
    cell1.appendChild(slash); 
    cell1.appendChild(downButton); 

} 

のような、それらの間のスラッシュを持っているセルを持つテーブルを持っているI探していますが、私にはこれを行う方法を修正するようには見えません。これが私の最初のDOMスクリプトプロジェクトです。

+2

これは***正しい方法です、良い仕事です! – adeneo

+0

@adeneo excellent!ありがとう、 –

答えて

1

私たちは皆、「コードが機能する限り、それはいいです」という世界に住んでいます。しかし、いくつかの提案:

  1. jQueryのあなたのためのオプションですが、それは、ブラウザに依存しないコードを開発する他に、あなたはすべてのブラウザで、この(ジャバスクリプト)コードをテストすることになります助けることができるかどうかわかりません。
  2. テンプレートHTMLを作成してセルに追加します。コード行数が大幅に減少し、コードがより読みやすくなります。
  3. インラインスタイルを使用しないでください。代わりに、cssクラスを作成し、classNameメソッドを使用してください。
+0

こんにちは、ありがとう、これは大学の割り当てのため、JQueryは現時点ではオプションではありません。HTMLテンプレートを作成するほうがはるかに簡単ですが、このプロジェクトではDOMスクリプトに焦点を当てる必要があります。私はCSSクラスのメソッドを見ていきます。再度、感謝します! –

関連する問題