Новичок HTML - кнопки не работают правильно

Question

Новичок HTML - кнопки не работают правильно

0

Я пытаюсь создать свой первый код HTML. Мой код:

<html>
<head>
</head>
<body>
<input type="button" id="apply" value="Push Me" onclick="javascript:faa();" />
<input type="button" id="apply" value="No, Push Me Instead" onclick="javascript:foo();"     />
Official website of Foocorp Inc. (Not really.)
</body>
<script type="text/javascipt">
function faa(e)
{

alert('Nope, it is the other button.');
}
</script>
<script type="text/javascript">
function foo(e)
{

alert('You have destroyed your computer. Thank you for your time.');
window.close();
}
</script>
</html>

Всякий раз, когда я нажимаю кнопку со значением "Нет, Push Me вместо", она работает нормально. Кнопка "Push Me" ничего не делает, когда я ее нажимаю. Что я делаю не так?

user1845203 17 дек. 2013, в 20:20

Источник

3

Помимо всего прочего, у вас есть два элемента с одинаковым id : это недопустимый HTML (« Атрибут [ id ] назначает имя элементу. Это имя должно быть уникальным в документе »). Также удалите javascript: из обработчиков событий в вашем HTML они совершенно не нужны.
David Thomas 17 дек. 2013, в 19:04
1

Когда вы начинаете, вы должны начать с лучших практик, а не с вредных привычек. Встроенный JavaScript не очень хорош. И, как указал Дэвид Томас, вы не можете иметь 2 элемента с одинаковым идентификатором. Я не думаю, что это проблема, но первым шагом отладки является наличие корректного кода.
Laurent S. 17 дек. 2013, в 19:05
0

Вы пошли и неправильно написали javascript в первой функции.
arandompenguin 17 дек. 2013, в 19:07
0

Кроме того, в дополнение к вышеприведенным комментариям, я не считаю правильным размещать <script></script> между тегами </body> и </html> . Насколько я знаю теги скрипта поддерживаются только в head или body .
War10ck 17 дек. 2013, в 19:18

Показать ещё 2 комментария

Теги:

html

3 ответа

Ещё вопросы

Помимо всего прочего, у вас есть два элемента с одинаковым id : это недопустимый HTML (« Атрибут [ id ] назначает имя элементу. Это имя должно быть уникальным в документе »). Также удалите javascript: из обработчиков событий в вашем HTML они совершенно не нужны.
Когда вы начинаете, вы должны начать с лучших практик, а не с вредных привычек. Встроенный JavaScript не очень хорош. И, как указал Дэвид Томас, вы не можете иметь 2 элемента с одинаковым идентификатором. Я не думаю, что это проблема, но первым шагом отладки является наличие корректного кода.
Вы пошли и неправильно написали javascript в первой функции.
Кроме того, в дополнение к вышеприведенным комментариям, я не считаю правильным размещать <script></script> между тегами </body> и </html> . Насколько я знаю теги скрипта поддерживаются только в head или body .

Quentin · Answer 1 · 2013-12-17T17-58-00.000Z

text/javascipt должен иметь r в нем.

HTML 5 делает атрибут type для элементов скрипта необязательным при написании JavaScript. Когда вы имеете дело с JS, атрибут не имеет никакой другой цели, чтобы быть возможностью сделать типы, которые нарушают ваш код, так что опустите его полностью.

Julien Altieri · Answer 2 · 2013-12-17T17-46-00.000Z

Прекрасно работает, когда я перемещаю другую функцию в тот же скрипт:

<script type="text/javascript">
    function foo(e)
    {

        alert('You have destroyed your computer. Thank you for your time.');
        window.close();
    }

        function faa(e)
    {

        alert('Nope, it is the other button.');
    }
</script>

PVR · Answer 3 · 2013-12-17T17-43-00.000Z

Вы должны изменить орфографию типа сценария, так как есть орфографическая ошибка

<script type="text/javascript">