Javascript onclick срабатывает, не нажимая его в цикле

-3

Я использую AJAX для получения одной из таблиц моей базы данных. Я пытаюсь улучшить его, создав функцию таблицы обновлений после нажатия кнопки.Javascript onclick срабатывает, не нажимая его в цикле

success: function(result) { 
    for(var i = 0; i < result.length; i++) { 
     document.getElementById('tableT').innerHTML += 
      "<tr>" + 
       "<td>"+result[i]['id']+"</td>" + 
       "<td>"+result[i]['terminalId']+"</td>" + 
       "<td>"+result[i]['departmentId']+"</td>" + 
       "<td>"+result[i]['profileId']+"</td>" + 
       "<td>"+result[i]['created']+"</td>" + 
       "<td>"+result[i]['modified']+"</td>"+ 
       "<td>"+"<div style='margin-bottom:10px; text-align: center;'>"+ 
       "<a class='btn btn-info edit-btn' data-toggle='modal' onclick='"+updApplication(result[i]['id'])+"' data-target='#myModal-44' title='Edit'><i class='fa fa-edit'></i></a>"+ 
        "</div>"+"</td>"+ 
      "</tr>"; 

    } 
},

Почему я получаю предупреждения со всеми моими идентификаторами, даже не нажимая кнопку?

function updApplication(id){ 
    alert(id); 
};

источник

2016-09-12 B. Hurray

вы вызываете функцию каждый раз, она должна быть строкой, а не вызовом функции, если вы не хотите, чтобы она вызывала его ... 'onclick = 'updApplication (result [" + i + "] [' id '])' ' – depperm

При написании предупреждений было бы полезно узнать текст предупреждающих сообщений. Но я уверен, что * depperm * указывает в правильном направлении. – gus27

У вас есть проблема в кавычках, теперь вы вызываете updApplication() функции в каждой итерации цикла, ваш код должен быть как:

..data-toggle='modal' onclick='updApplication("+result[i]['id']+")' data-target='#myModa..

Надеется, что это помогает.

источник

2016-09-12 16:44:57

Вы получаете предупреждения, потому что на самом деле вы вызываете функцию при создании разметки.

Попробуйте определить onclick с помощью функции в качестве строки.

"onclick=\"updApplication(" + result[i]['id'] + ")\"..."

источник

2016-09-12 16:42:15 IrkenInvader

При использовании 'onclick' не будет доступа к локальной переменной' result'. И 'id' должен быть в кавычках или просто использовать' .id'. –

Хороший звонок, отредактирован. – IrkenInvader

Да, выглядит лучше, имея в виду, что если 'id' является строкой, ее нужно будет обернуть в кавычки в позиции аргумента. Если это число, то проблем нет. –

Javascript onclick срабатывает, не нажимая его в цикле

ответ

Смежные вопросы