メソッドと関数を動作させる方法 - Javascript

Question

以下のメソッドと関数は動作しません。

hasMoreOscarsThan - this method accepts one actor object as a parameter and 
    returns true if the actor has more Oscars than the one that is passed as 
    a parameter and false otherwise. 

ここで、次の関数を作成：

getActorByName - this function expects a string as a parameter and returns 
    the object in the actors array whose name property is equal to the 
    string that is passed in (if there is one). 

マイコード：

function Person(firstName, lastName, age, numOscars) { 

    this.firstName = firstName; 
    this.lastName = lastName; 
    this.age = age; 
    this.numOscars = numOscars; 
    this.hello = function() { console.log("Hello, my name is " + this.firstName); } 
    this.hasMoreOscarsThan = function(x, y) { 
     if (this.numOscars > this.numOscars) { 
      return this.firstName; 
     } else { 
      return "False!"; 
     } 
    } 

}; 


var actors = new Array(); 
actors[0] = new Person("Leonardo", "DiCaprio", 41, 1); 
actors[1] = new Person("Jennifer", "Lawrence", 25, 1); 
actors[2] = new Person("Samuel L.", " Jackson", 67, 0); 
actors[3] = new Person("Meryl", "Streep", 66, 3); 
actors[4] = new Person("John", "Cho", 43, 0); 

actors.forEach(function(item) { 
    item.hello(); 
}) 

actors.forEach(function(item) { 
    item.hasMoreOscarsThan(); 
}) 


function getActorByName(person) { 
    console.log(actors.firstName + " " + actors.lastName); 
} 


function list() { 
    var actorsLength = actors.length; 
    for (var i = 0; i < actorsLength; i++) { 
     getActorByName(actors[i]); 
    } 
} 


var search = function(lastName) { 
    var actorsLength = actors.length; 
    for (var i = 0; i < actorsLength; i++) { 
     if (lastName == actors[i].lastName) { 
      getActorByName(actors[i]); 
     } 
    } 
} 


search("DiCaprio"); 

var getAverageAge; 

getAverageAge = (actors[0].age + actors[1].age + actors[2].age + actors[3].age + actors[4].age)/actors.length; 
console.log(getAverageAge); 

は、事前にどうもありがとうございました!!!

Answer 1

に変更私はあなたのコードを編集した：

function Person(firstName, lastName, age, numOscars) { 

    this.firstName = firstName; 
    this.lastName = lastName; 
    this.age = age; 
    this.numOscars = numOscars; 
    this.hello = function() { console.log("Hello, my name is " + this.firstName); } 
    this.hasMoreOscarsThan = function(x) { // x is a compare parameter 
     if (this.numOscars > x) { // comparing numOscars with argument x 
      return this.firstName; 
     } else { 
      return "False!"; 
     } 
    } 

}; 


var actors = new Array(); 
actors[0] = new Person("Leonardo", "DiCaprio", 41, 1); 
actors[1] = new Person("Jennifer", "Lawrence", 25, 1); 
actors[2] = new Person("Samuel L.", " Jackson", 67, 0); 
actors[3] = new Person("Meryl", "Streep", 66, 3); 
actors[4] = new Person("John", "Cho", 43, 0); 

actors.forEach(function(item) { 
    item.hello(); 
}) 

actors.forEach(function(item) { 
    console.log(item.hasMoreOscarsThan(2)); // I put compare argument 2 and print result to console 
}) 


function getActorByName(person) { 
    console.log(person.firstName + " " + person.lastName); // changed actors to person 
} 


function list() { 
    var actorsLength = actors.length; 
    for (var i = 0; i < actorsLength; i++) { 
     getActorByName(actors[i]); 
    } 
} 


var search = function(lastName) { 
    var actorsLength = actors.length; 
    for (var i = 0; i < actorsLength; i++) { 
     if (lastName == actors[i].lastName) { 
      getActorByName(actors[i]); 
     } 
    } 
} 


search("DiCaprio"); 

var getAverageAge; 

getAverageAge = (actors[0].age + actors[1].age + actors[2].age + actors[3].age + actors[4].age)/actors.length; 
console.log(getAverageAge); 

結果：

こんにちは、私の名前はこんにちは私の名前は

メリル

で、私の名前はこんにちは

サミュエル・L.で、私の名前はジェニファー

こんにちはです、こんにちは

レオナルド

です、私の名前はジョンです

偽！ False！
False！
メリル
偽！
レオナルド・ディカプリオ48.4

Answer 2

引数はpersonオブジェクトですが、関数内でどこにも定義されていないいくつかのactorsを参照している、だから、person.firstName & person.lastName

function getActorByName(person) { 
    console.log(person.firstName + " " + person.lastName); 
} 

JSFIDDLE