2015-11-18 5 views
1

У меня есть три рабочих бита javascript, которые я пытаюсь объединить ... безуспешно. Я пытаюсь сделать это, когда вы нажимаете кнопку, он знает, что два поля имеют одну и ту же запись. Если нет, это сделает другую строку в таблице в моем HTML с информацией.Динамически вставлять строку в таблицу HTML с JavaScript

document.getElementById("goRow").addEventListener("click", check) 

function check() { 
    if (document.getElementById("firstName").value ==  document.getElementById("lastName").value){ 
    prompt('First name can not be last'); 
    } 

    else { 
    function newRow() { 
     var table = document.getElementById("myTable"); 
     var row = table.insertRow(1); 
     var cell1 = row.insertCell(0); 
     var cell2 = row.insertCell(1); 
     var cell3 = row.insertCell(2); 
     var cell4 = row.insertCell(3); 
     cell1.innerHTML = document.getElementById("firstName").value; 
     cell2.innerHTML = document.getElementById("lastName").value; 
     cell3.innerHTML = document.getElementById("email").value; 
     cell4.innerHTML = " "; 
    } 
    } 

Как я уже сказал, все различные части работают сами по себе, но я не могу получить их все, чтобы играть хорошо друг с другом.

+0

Как указано выше в комментарии; удалите функцию newRow() {'и закрытие'} 'из инструкции else. – www139

+0

Вам также не хватает закрывающей скобки '}', чтобы закрыть 'функцию check()' – dave

ответ

5

Почему вы пытаетесь определить функцию как часть условия ELSE? Вы должны иметь, что само по себе, и вызывать его из ELSE:

function check() { 
    if (document.getElementById("firstName").value == document.getElementById("lastName").value) { 
     prompt('First name can not be last'); 
    } else { 
     newRow(); 
    } 
} 

function newRow() { 
    var table = document.getElementById("myTable"); 
    var row = table.insertRow(1); 
    var cell1 = row.insertCell(0); 
    var cell2 = row.insertCell(1); 
    var cell3 = row.insertCell(2); 
    var cell4 = row.insertCell(3); 
    cell1.innerHTML = document.getElementById("firstName").value; 
    cell2.innerHTML = document.getElementById("lastName").value; 
    cell3.innerHTML = document.getElementById("email").value; 
    cell4.innerHTML = " "; 
} 
+0

благодарю вас за ответ. Я пытался определить функцию как часть условия Else, потому что это только моя вторая неделя Javascript. Код работает сейчас, я подтвердил ваш ответ, но он не появится, пока у меня не будет больше очков репутации. –

0
window.onload() { 
    document.getElementById("goRow").addEventListener("click", check) 
} 

function check() { 
    if (document.getElementById("firstName").value == document.getElementById("lastName").value){ 
    prompt('First name can not be last'); 
    } else { 
    newRow(); 
    } 
} 

function newRow() { 
    var table = document.getElementById("myTable"); 
    var row = table.insertRow(1); 
    var cell1 = row.insertCell(0); 
    var cell2 = row.insertCell(1); 
    var cell3 = row.insertCell(2); 
    var cell4 = row.insertCell(3); 
    cell1.innerHTML = document.getElementById("firstName").value; 
    cell2.innerHTML = document.getElementById("lastName").value; 
    cell3.innerHTML = document.getElementById("email").value; 
    cell4.innerHTML = " "; 
} 
Смежные вопросы