2015-07-26 4 views
0

Я пытаюсь учиться, поэтому, если вы можете просто указать на ошибку, не сказав мне, что это будет здорово, но если это потребует решения, я в порядке с этим - я просто хотел бы объяснить ,Простой код JS не работает

Цель - нажать кнопку и переместить ее в произвольное положение. Я не получаю ошибок, но я тоже не получаю вывод console.log. Если я переведу console.log из функции, я получаю вывод.

var buttonState = document.getElementById("clickMe"); 
var maxWidth = screen.width; 
var maxHeight = screen.height; 
var screenWidth = 0; 
var screenHeight = 0; 
// Find max width and height of screen and set variables to random number within parameters 
function moveButton() { 
    "use strict"; 
    screenWidth = Math.floor(Math.random() * (maxWidth - 1 + 1)) + 1; 
    screenHeight = Math.floor(Math.random() * (maxHeight - 1 + 1)) + 1; 
} 
buttonState.onClick = function() { 
    "use strict"; 
    movebutton(); 
    buttonState.style.left = screenWidth.page + "px"; 
    buttonState.style.top = screenHeight.page + "px"; 
    console.log(screenWidth); 
    console.log(screenHeight);   
}; 
+1

Я обнаружил четыре ошибки: два первых: напомните, что в javascript значение _letter в идентификаторах важно. Затем одна ошибка связана с предложением @ vinayakj. И, наконец, отлаживайте значения, которые вы фактически устанавливаете на 'left' и' top'. –

ответ

2
buttonState.onClick 

Имя свойства onclick (с нижним регистром c). JavaScript чувствителен к регистру.

+0

OK большой. Я также поймал еще несколько ошибок из вашей помощи. Спасибо. Он работает сейчас. – Mike

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