Javascript «не может прочитать свойство не определено»

Question

При выполнении следующего кода, я получаю ошибку

«не может прочитать свойство„слово“неопределенных на звёздный.

I «м действительно застряли, почему так что любая помощь будет оценена по достоинству!

function MyStim(word,instruction) { 
    this.word = word; 
    this.instruction = instruction; 
}; 

var data = [new MyStim("Cat","Forget"), new MyStim("Dog","Remember"), new MyStim("Sour Patch Kids","Never Forget"), new MyStim("Cookies", "Remember")] 

function getpresent() { 
    return { 
     type: "single-stim", 
     **stimulus: "<p style= font-size:100px;text-align:center;padding-top:300px;>"+data[i].word+"</p>",** 
     is_html: true, 
     timing_stim: 250, 
     timing_response: 2000, 
     response_ends_trial: false, 
    } 
}; 

for (var i = 0; i<80; i++){ 
    timeline.push(getpresent(i)); 
    i++; 
}; 

Answer 1

getpresent не имеет ссылки на i переменные, вы ожидающие I t иметь.

Изменить

function getpresent() { 

Для

function getpresent (i) { 

Answer 2

Проблема заключается в том, что вы не определили аргумент называется i вашей функции:

function getpresent (i) { 
    return { 
     type: "single-stim", 
     stimulus: "<p style= font-size:100px;text-align:center;padding-top:300px;>"+data[i].word+"</p>", 
     is_html: true, 
     timing_stim: 250, 
     timing_response: 2000, 
     response_ends_trial: false, 
    } 
}; 

Итак, когда вы назвали функцию у i будет значение. Следовательно, вы пытаетесь прочитать значение word по значению undefined.

Кроме того, у вас есть еще две ошибки в цикле for.

• Вам не нужно i++ в теле вашего цикла
• Вы должны петля в диапазоне [0, data.length)

Скорректированная петли следующим образом:

for (var i = 0; i<data.length; i++){ 
    timeline.push(getpresent(i)); 
} 

Answer 3

параметр «i» в вашей функции «getpresent» отсутствует.

Answer 4

После устранения проблемы function getpresent(i) я заметил три оставшихся проблемы с этим фрагментом кода.

Этот цикл -

for (var i = 0; i<80; i++){ 
    timeline.push(getpresent(i)); 
    i++; 
}; 

i++ линии внутри блока контура не является необходимым. Ваш IIFE ((var i = 0; i<80; i++)) делает это за вас.

Этот IIFE также выполняет цикл длиннее длины массива данных. Поэтому после того, как он попадает к последнему элементу массива данных, он ловит «слово», не определено, поскольку больше нет объектов для повторения. Изменение 80 до data.length разрешает это. (В качестве альтернативы вы можете проверить наличие данных [i] перед вызовом функции).

Внутри временной линии цикла. Нигде в этом кодовом блоке не инициализирована временная шкала и поэтому не будет доступна нигде вне этого цикла. Добавление var timeline = [] решает эту проблему (если у вас нет этого набора в другом месте).

function MyStim(word,instruction) { 
    this.word = word; 
    this.instruction = instruction; 
}; 

var data = [new MyStim("Cat","Forget"), new MyStim("Dog","Remember"), new MyStim("Sour Patch Kids","Never Forget"), new MyStim("Cookies", "Remember")] 

function getpresent (i) { 
    return { 
     type: "single-stim", 
     stimulus: "<p style= font-size:100px;text-align:center;padding-top:300px;>"+data[i].word+"</p>", 
     is_html: true, 
     timing_stim: 250, 
     timing_response: 2000, 
     response_ends_trial: false, 
    } 
}; 

//added initialization 
var timeline = []; 

//changed '80' to 'data.length' 
for (var i = 0; i < data.length; i++){ 
    timeline.push(getpresent(i)); 
    //removed "i++" 
}; 


console.log(timeline) 

Answer 5

Вы никогда не должны пытаться обращаться к данным таким образом. Скорее передайте данные в вашу функцию. Не обращайтесь к своим данным из-за пределов области действия функции. Храните ваши данные в пределах области действия и релевантно для функции, которая ее обрабатывает. Написание такого кода - это разведение вредных привычек.

<script> 
function PresentBuilder() 
{ 
    this.type = "single-stim"; 
    this.is_html = true; 
    this.timing_stim = 250; 
    this.timing_response = 2000; 
    this.response_ends_trial = false; 

    this.set_type = function (type){ 
     this.type = type; 
     return this; 
    }; 

    this.set_is_html = function (is_html){ 
     this.is_html = is_html; 
     return this; 
    }; 

    this.set_timing_stim = function (timing_stim){ 
     this.timing_stim = timing_stim; 
     return this; 
    }; 

    this.set_timing_response = function (timing_response){ 
     this.timing_response = timing_response; 
     return this; 
    }; 

    this.set_response_ends_trial = function (response_ends_trial){ 
     this.response_ends_trial = response_ends_trial; 
     return this; 
    }; 

    this.getpresent = function (data){ 
     return { 
      type: this.type, 
      stimulus: "<p style= font-size:100px;text-align:center;padding-top:300px;>"+data.word+"</p>", 
      is_html: this.is_html, 
      timing_stim: this.timing_stim, 
      timing_response: this.timing_response, 
      response_ends_trial: this.response_ends_trial, 
     } ; 
    }; 
}; 

function MyStim(word, instruction) { 
    this.word = word; 
    this.instruction = instruction; 
}; 

var data = [new MyStim("Cat","Forget"), new MyStim("Dog","Remember"), new MyStim("Sour Patch Kids","Never Forget"), new MyStim("Cookies", "Remember")]; 

timeline = []; 
for (var i = 0; i<data.length; i++){ 
    timeline.push(new PresentBuilder().set_response_ends_trial(true).getpresent(data[i])); 
}; 

for (var i = 0; i<timeline.length; i++){ 
    console.log(timeline[i]); 
} 
</script> 

Я действительно не пишу javascipt, но если бы мне пришлось это сделать, я бы предпочел попробовать это. Важно то, как передаются данные в , а также.