条件が配列の繰り返し内で常にtrueを返す場合

Question

Udemy Web Development Bootcamp（Colt Steele）のコードの一部として、配列内のアイテムをリストする次のjavascriptを用意しています。ブール値「hasWatched」条件。 Console.logはすべての配列項目を真であるかのように返します。

// Create an array of objects. Each movie should have a title, rating and hasWatched properties. Iterate through the array and print out results and whether it has been hasWatched 

var movieArr = [ 
    { 
     title: "LOTR", 
     rating: 5, 
     hasWatched: true 
    }, 
    { 
     title: "Fast and the Furious", 
     hasWatched: false, 
     rating: 1 
    }, 
    { 
     title: "Let the Right One In", 
     rating: 5, 
     hasWatched: false 
    } 
] 


for(i = 0; i < movieArr.length; i++){ 
    if(movieArr[i].hasWatched = true){ 
     console.log("You have seen " + movieArr[i].title + ": Rating: " + movieArr[i].rating); 
    } else { 
     console.log("You have not seen " + movieArr[i].title + ": Rating: " + movieArr[i].rating); 
    } 
} 

ここでは何が欠けていますか？

多くの感謝！ Rick

Answer 1

trueをプロパティに割り当てますが、値を確認する必要があります。比較を省略して値を直接使用することもできます。副作用を防ぐために

if (movieArr[i].hasWatched = true) { 
//      ^

、

if (true = movieArr[i].hasWatched) { // throws: Invalid left-hand side in assignment 

のように、チェックするための切り替え条件は今の状態が例外ANにはないスローでの条件で間違って割り当てて、あなたは、Yoda conditions（YC）を使用することができます値を割り当てます。

YCで完全な作業のチェックは、真のチェックが原因hasWatchedの与えられたと期待値と、不必要であるこの文

if (true == movieArr[i].hasWatched) { 

ようになります。

最終チェック条件がtruthynessの値とチェックを使用して：

if (movieArr[i].hasWatched) { 

var movieArr = [{ title: "LOTR", rating: 5, hasWatched: true }, { title: "Fast and the Furious", hasWatched: false, rating: 1 }, { title: "Let the Right One In", rating: 5, hasWatched: false }]; 
 

 
//first attempt 
 
for (i = 0; i < movieArr.length; i++) { 
 
    if (movieArr[i].hasWatched) { 
 
    console.log("You have seen " + movieArr[i].title + ": Rating: " + movieArr[i].rating); 
 
    } else { 
 
    console.log("You have not seen " + movieArr[i].title + ": Rating: " + movieArr[i].rating); 
 
    } 
 
} 
 

 
//second attempt 
 
movieArr.forEach(function(i) { 
 
    var result = "You have "; 
 
    if (i.hasWatched) { 
 
    result += "watched "; 
 
    } else { 
 
    result += "not watched "; 
 
    } 
 
    result += "\"" + i.title + "\" - "; 
 
    result += i.rating + " stars"; 
 
    console.log(result) 
 
});

Answer 2

if (movieArr[i].hasWatched = true) { 

は次のようになります

if (movieArr[i].hasWatched == true) { 

あなたの割り当てはhasWatchedからtrueまで毎回です。

Answer 3

hasWatched変数をtrueと比較する必要がある場合は、変数をtrueに設定しています。

if (movieArr[i].hasWatched = true) {

対

if (movieArr[i].hasWatched == true) {