Подскажите, не работает код, не могу понять почему - вроде бы все правильно

@ЕвгенийЧе · Регистрация: 11.01.2013

Author24 — интернет-сервис помощи студентам

Доброго времени суток
Есть задача - при клике на кнопку должен создаваться параграф в теле документа, в который добавляется значение кнопки. Код вроде бы корректный, но не работает. Буду признателен за советы/критику/подсказки
Код:

PHP/HTML

    <script type="text/javascript">
        window.onload = function () {
 
           //создаем массив кнопок и вешаем на каждую кнопку событие онклик
            var inputs = document.getElementsByTagName("input");
            for (var i = 0; i < inputs.length; ++i) {
                inputs[i].onclick = clickButton;
            }
           
            //прописываем функцию clickButton
            var clickButton = function () {
                addLine(this.value);
            }
 
            //прописываем функцию addLine, которая создает параграф в теле документа и помещает в него аргумент функции - text - т.е. значение кнопки
            var addLine = function (text) {
                var newLine = document.createElement("p");
                newLine.innerHTML = text;
                document.body.appendChild(newLine);
            }
        }
 
       
         
    </script>
</head>
<body>
 
 
        <form name="calc" id="knopki">
    <table>
 
    <td class="buttons">
        <input type="button" id="one" value="1" >
        <input type="button" id="two" value="2" >
        <input type="button" id="three" value="3" >
        
        <br>
        <input type="button" id="four" value="4" >
        <input type="button" id="five" value="5" >
        <input type="button" id="six" value="6" >
        
        <br>
        <input type="button" id="seven" value="7" >
        <input type="button" id="eight" value="8" >
        <input type="button" id="nine" value="9" >
        
        <br>
        <input type="button" id="zero" value="0" >
        
    </td>
    </tr>
    </table>
</form>
 
</body>

@Eva Rosalene · 23.09.2014, 22:04

ЕвгенийЧе,

Javascript

window.onload = function () {
 
           //создаем массив кнопок и вешаем на каждую кнопку событие онклик
            var inputs = document.getElementsByTagName("input");
            for (var i = 0; i < inputs.length; ++i) {
                inputs[i].onclick = clickButton;
            }
           
            //прописываем функцию clickButton
            var clickButton = function (e) {
                addLine(e.target.value);
            }
 
            //прописываем функцию addLine, которая создает параграф в теле документа и помещает в него аргумент функции - text - т.е. значение кнопки
            var addLine = function (text) {
                var newLine = document.createElement("p");
                newLine.innerHTML = text;
                document.body.appendChild(newLine);
            }
        }

Добавлено через 20 секунд
Основное отличие - 10 и 11 строки

@ЕвгенийЧе · 23.09.2014, 22:11 **[ТС]**

Спасибо, смысл изменения понял. Но, к сожалению, не работает(( Возможно причина в IDE? Я использую Visual Studio 2012 for Web

@Eva Rosalene · 23.09.2014, 22:41

ЕвгенийЧе, плюс ко всему синтаксис объявления функции через присвоение её анонимной переменной Вы используете логически неверно. На момент исполнения строк 5-6 она ещё недоступна.

Javascript

window.onload = function () {
             
            var clickButton = function (e) {
                addLine(e.target.value);
            }
 
           //создаем массив кнопок и вешаем на каждую кнопку событие онклик
            var inputs = document.getElementsByTagName("input");
            for (var i = 0; i < inputs.length; ++i) {
                inputs[i].onclick = clickButton;
            }
 
 
            //прописываем функцию addLine, которая создает параграф в теле документа и помещает в него аргумент функции - text - т.е. значение кнопки
            var addLine = function (text) {
                var newLine = document.createElement("p");
                newLine.innerHTML = text;
                document.body.appendChild(newLine);
            }
        }

@ЕвгенийЧе · 23.09.2014, 23:14 **[ТС]**

Большое спасибо

@ЕвгенийЧе 0 / 0 / 1 Регистрация: 11.01.2013 Сообщений: 38
	23.09.2014, 22:11 [ТС]	3
	Спасибо, смысл изменения понял. Но, к сожалению, не работает(( Возможно причина в IDE? Я использую Visual Studio 2012 for Web 0

@ЕвгенийЧе 0 / 0 / 1 Регистрация: 11.01.2013 Сообщений: 38
	23.09.2014, 23:14 [ТС]	5
	Большое спасибо 0

Подскажите, не работает код, не могу понять почему - вроде бы все правильно

Решение