style.displayがnullです - 配列をループして表示プロパティを変更しようとしています

Question

私からの別の初心者の質問。

以下のコードが可能かどうかを確認するだけです。私は 'document.GetElementById's'で構成される新しい配列を作成します。私はかなりjavascriptに新しいので、私のコードは一般的に少し長い風が吹いているので、厄介さを許してください。

以下のコードは、 'style.display'が 'null'またはオブジェクトではないというエラーを表示しています。 誰かが明らかに何かを見たり、間違っていることがありますか？隠しフィールド（「personal2」など）が空白であれば

function Test(){ 

if(document.getElementById('inClient').value !=="FormViewer"){ 

    var visible =new Array("document.getElementById('personal2').value","document.getElementById('change_hours2').value"); 
    var change = new Array("document.getElementById('personal').value","document.getElementById('change_hours').value"); 

    for (var i=0; i <visible.length; i++) { 
     if(visible[i]!==""){ 
      change[i].style.display = "block" 
     } 
     } 
    } 
} 

は基本的に私は、DIV /セクション（「個人」など）は隠されたままにしたいが、それはテキストが含まれているならば、私はセクションを表示したいです。

ありがとうございます

Answer 1

DOMエレメントへの参照の代わりに文字列を配列に追加しています。引用符とvalueプロパティを削除しても機能します。繰り返し内でvalueをチェックする必要があります。

function Test(){ 

if(document.getElementById('inClient').value !=="FormViewer"){ 

    var visible =new Array(document.getElementById('personal2'), document.getElementById('change_hours2')); 
    var change = new Array(document.getElementById('personal'), document.getElementById('change_hours')); 

    for (var i=0; i <visible.length; i++) { 
     if(visible[i].value != ""){ 
      change[i].style.display = "block" 
     } 
     } 
    } 
} 

Answer 2

このように使用する必要があります。ダブルクオートでは動作しません。

var visible =new Array(document.getElementById('personal2').value,document.getElementById('change_hours2').value); 
    var change = new Array(document.getElementById('personal').value,document.getElementById('change_hours').value);