JavaScriptコードで何が問題になっていますか？

Question

何が間違っているのか分かりません。

var CarObj = function(passengers, maxLoad, wheels, doors, maxSpeed) { 
    this.passengers = passengers; 
    this.maxLoad = maxLoad; 
    this.wheels = wheels; 
    this.doors = doors; 
    this.maxSpeed = maxSpeed; 
}; 
var ferrari = new CarObj(4, "700kg", 4, 2, "360km/h"); 
var output = new Array(); 
for (var i = 0; i < ferrari.length; i++) { 
    for (var a in ferrari) { 
     output[i] = a; 
    } 
} 
document.getElementById('ELEMENTHERE').innerHTML = (output.join(" ")); 

Answer 1

for (var i = 0; i < ferrari.length; i++) { 

ferrari配列ではありませんが、あなたはそれがあるようにそれにアクセスしようとしています。これと対応するエンドブレースを取り外します。

output[i] = ...;をoutput.push(...);またはoutput[output.length] = ...;に変更すると、iを手動で増やす必要がなくなります。

Answer 2

あなたのfor...inは、外側のforと同じオブジェクトをループしています。これはferrariに長さがある場合outputの各項目をferrariの最後のキーに設定します。そうでない場合は、未定義のプロパティについてエラーを表示します。

for (var a in ferrari) { 
    output[output.length] = a; // or ferrari[a] if you wanted values, not keys 
} 

と完全に外forループを取り除く：あなたはおそらく代わりにやってみたかった何

。

Answer 3

for (var i in ferrari) output.push(ferrari[i]); 

それは少しより堅牢行うにすることだろう。

for (var i in ferrari) 
    if (ferrari.hasOwnProperty(i)) 
    output.push(ferrari[i]); 

Answer 4

あなたは

i = 0; 
for (var a in ferrari) { 
    output[i++] = a; 
} 

を意味していますか？ ferrariにはlengthプロパティがありません。

Answer 5

ferraiはオブジェクトであり、名前の長さ（配列はありません）でプロパティを持ちません。以下のようにコードを変更し ：

<script type="text/javascript"> 
     var CarObj = function(passengers, maxLoad, wheels, doors, maxSpeed) {  
      this.passengers = passengers;  
      this.maxLoad = maxLoad;  
      this.wheels = wheels;  
      this.doors = doors;  
      this.maxSpeed = maxSpeed; 
     }; 
     var ferrari = new CarObj(4, "700kg", 4, 2, "360km/h"); 
     var output = new Array(); 
     var i=0; 
      for (var a in ferrari) 
      { 
       output[i] = a;  
       i++; 
      } 
     document.getElementById('ELEMENTHERE').innerHTML = (output.join(" ")); 
    </script> 

Answer 6

理由だけではなく、オブジェクトのtoString（）をオーバーライドしますか？

var CarObj = function(passengers, maxLoad, wheels, doors, maxSpeed) { 
    this.passengers = passengers; 
    this.maxLoad = maxLoad; 
    this.wheels = wheels; 
    this.doors = doors; 
    this.maxSpeed = maxSpeed; 
    this.toString = function() { 
    return this.passengers + " " + this.maxLoad + " " + this.wheels + " " + this.doors + " " + this.maxSpeed; 
    }; 
};