2016-11-06 2 views
-3

У меня есть PHP-страницу с формой:Javascript для запуска тревожного сообщения на форме

<form method="POST" action="multimod.php" id="multiple"> 
    <!-- some checkboxes here --> 
    <input type="image" name="submit" src="img/upd.png" alt="modify" class="multi" /> 
</form> 

Тогда я хочу, чтобы вызвать тревогу на кнопку. Когда изображение щелкнули, сообщение sweetAlert срабатывает:

<script> 
$('input.multi').click(function(e){ 
    e.preventDefault(); 
swal({ 
     title: "a title", 
     text: "some text", 
     type: "warning", 
     confirmButtonText: 'yes', 
     cancelButtonText: 'no', 
     showCancelButton: true, 
}, 
function(){ 
    document.forms["multiple"].submit(); 
}); 
}); 
</script> 

Как вы можете видеть id вида («множественный») используется для ссылки на него в функции Javascript.

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

+4

Я голосую, чтобы закрыть этот вопрос не по теме, потому что она о [улучшения рабочего кода в неспецифическим образом] (http://codereview.stackexchange.com/help/on-topic) – Quentin

+0

Ну, я пытаюсь найти конкретный способ его улучшить. – Alex

+0

SO - для проблем программирования/проблем. Вы должны опубликовать это на сайте [Code Review] (http://codereview.stackexchange.com/), потому что речь идет о пересмотре и улучшении уже действующего кода. –

ответ

2

Поскольку вы хотите вызвать его на отправке, нет причин добавлять слушателя на кнопку.

<script> 
function onFormSubmit() { 
    alert("Before submitting to server!"); 
    return true; // allow the submission! 
} 
</script> 
<form onsubmit="return onFormSubmit();" method="POST" action="multimod.php" id="multiple"> 
    <!-- some checkboxes here --> 
    <input type="image" name="submit" src="img/upd.png" alt="modify" class="multi" /> 
</form>