Как подтвердить AJAX-удаление?

Question

[DrunkMaster]

HTML

<span class="delete" onclick="return confirm('Точно удалить?');">удалить</span>

JS (ajax)

// Ajax отправка запросов на удаление
        $('span.delete').click(function(){
          ajax({
            url: 'https://delete.url',
            data: {
              'val'
            },
            ok : function(d){message('Запись успешно удалена');}
          });
        });

Проблема в том, что когда всплывающее окно появляется с вопросом "Точно удалить?" то без разницы что я там нажму, AJAX всё равно срабатывает и удаляет запись

Answer 1

[Дмитрий Еремин]

Самое простое
1. Убрать onclick
2 Изменить обработчик

$('span.delete').click(function() {
        let isConfirmed = confirm('Точно удалить?')
        if (isConfirmed) {
            ajax({
                url: 'https://delete.url',
                data: {
                    'val'
                },
                ok: function(d) {
                    message('Запись успешно удалена');
                }
            });
        });
}

Comments

[DrunkMaster]

Что за let, он там ругается на строку let confirm = confirm('Точно удалить?') и говорит не знаю что такое confirm.

[Kovalsky]

DrunkMaster, а какой текст ошибки?

[Adamos]

Попытка назвать переменную так же, как встроенную функцию, да еще без всякой на то необходимости.
if (confirm('Точно удалить?')) {

[Дмитрий Еремин]

Adamos, точняк)

[Дмитрий Еремин]

DrunkMaster, действительно
не let confirm, а let conf, например
"confirm" - зарезервированное слово)

[Adamos]

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

[DrunkMaster]

Adamos, зато отлаживать проект после другого разработчика который экономил на переменных это то ещё удовольствие...

[Дмитрий Еремин]

Adamos, дап
А еще, когда хочешь показать код другому человеку и хочешь сделать его максимально понятным

[Adamos]

Дмитрий Еремин, напоминаю, вы, "делая код максимально понятным", не только удлинили его вдвое, но и допустили в нем ошибку.
Собственно, предположение, что читающему будет непонятна конструкция if(confirm()), мне представляется весьма для него оскорбительным.

[Дмитрий Еремин]

Adamos, какой ты молодец. Наверное, лучше всех делаешь лабораторки в группе

[Adamos]

Дмитрий Еремин, "лабораторки" я за группой проверял в начале 90-х. Такой дичи начитался...
Не переводите полемику в эмоциональную плоскость, это неконструктивно.

Answer 2

[Кирюшка Цисарь]

onclick можно тоже

onclick="return confirm('are u shure?') ? true : false;"

Comments

[DrunkMaster]

Так тоже удаляет даже если на cancel нажать... ))

[Кирюшка Цисарь]

DrunkMaster, ну дак не надо мой код просто брать и копировать)

onclick="confirm('are u shure?') ? GO() : '' "

function GO(){
 ajax({
 })
}

для true - запускаем функцию, для false оставляем пусто