2016-01-03 4 views
0

Параметр onclick не работает, почему?Ошибка Javascript с ошибкой

mydiv.innerHTML = mydiv.innerHTML + '<div style="border-bottom: 1px solid #D1D1D1; height: 50px; width: 100%; color: #464646; font-family: RobotoMedium;" onclick="seite5_wechseln('+variable+')">'; 

Надеюсь, что есть решение.

+0

Что такое значение 'variable'? Вероятно, проблема заключается в том, что вы хотите использовать его как строчную константу, но вы остановили кавычки. – Pointy

+0

его что-то вроде: acc2 –

+0

демо на jsfiddle было бы лучше – Gintoki

ответ

1

Кажется, что вы передаете строку функции seite5_wechseln. Но вы не включаете его с кавычкой.

попробовать:

mydiv.innerHTML = mydiv.innerHTML + '<div style="border-bottom: 1px solid #D1D1D1; height: 50px; width: 100%; color: #464646; font-family: RobotoMedium;" onclick="seite5_wechseln(\''+variable+'\')">'; 
+0

Хотя это правильно, есть много неправильного кода OP, который можно и нужно адресовать, вместо того, чтобы исправлять один бит, который не работает. –

+0

СПАСИБО !!! Это было решение –

+0

@ T.Jansen, пожалуйста. Если это было полезно для вас, просьба отметить его как 'anaswer' –

1

Есть так много вещей неправильно с этим ...

Попробуйте вместо этого. Я называю это «делать это правильно»;)

var newdiv = document.createElement('div'); 
newdiv.style.cssText = [ 
    "border-bottom: 1px solid #D1D1D1", 
    "height: 50px", 
    "width: 100%", 
    "color: #464646", 
    "font-family: RobotoMedium" 
].join(";"); 
newdiv.onclick = function() { 
    seite5_wechseln(variable); 
}; 
mydiv.appendChild(newdiv); 

Дальнейшие усовершенствования могут быть сделаны, например, определение класса CSS для элемента и положить стили там (используйте newdiv.className = "myclass";, чтобы установить его на элемент), а также как использование addEventListener, а не назначение .onclick, но это уже огромное улучшение по сравнению с исходным кодом.

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