2014-02-03 6 views
2

фонMisplaced Скобка

ИТАК Я работаю на пути остановки формы из освежающих, но по-прежнему проводков, а затем показывать предупреждение JS.

Проблема

У меня нет достаточно опыта Java Script, чтобы собрать два куска кода вместе, первый будучи Ajax пост, а второе событие OnClick.

Я не могу заставить скобки правильно совпадать и/или получить синтаксис правильно.

Как вы можете догадаться, это закрывающие круглые скобки, с которыми у меня проблемы.

Любая помощь будет отличной.

+1

использовать что-то вроде Notepad ++, jsHint или даже консоль ошибок с этого момента. – dandavis

+0

вы можете следить за отступом кода, чтобы поместить скобки – Huangism

+0

Я пробовал все виды программ сегодня днем, пока Netbeans, Visual Studio и Web Storm. Веб-шторм был самым полезным, но он все еще довольно сложный по сравнению с DreamWeaver. Я дам JsHint, спасибо @dandavis. –

ответ

2

Похоже, вам просто нужно некоторое форматирование, то вы можете увидеть, что не закрыты и добавить соответствующие фигурные скобки и круглые скобки:

$(document).ready(function() { 
    //Event Listener for form submit 
    $('#form1').submit(function(e){ 
    e.preventDefault(); 
    console.log('Form Submitted'); //Debug line 
    $.ajax({ 
     type: 'POST', 
     url: 'indextest2.php', 
     data: $("#form1").serialize(), 
     error: function() {console.log('Ajax Error');}, //<-- Add this 
     success: function(response) { 
     console.log('Ajax Success'); //Debug Line 
     $(document).on("click", ".alert", function(e) { 
      bootbox.alert("Hello world!", function() { 
      console.log("Alert Callback"); 
      }); 
     }) 
     } 
    }); // < -- Add this 
    }); // <-- Add this 
}); // <-- Add this 
+0

Благодарим за помощь, я внесу необходимые изменения. Как я уже сказал, я все еще довольно новичок в JavaScript, и я был уверен, когда нужно закрыть «}); или просто "}". –

+1

В этом случае вы передаете функции в другие функции в качестве параметров, поэтому вам нужно закрыть ')' для этих вызовов функций. Да, сбивчиво - использование хорошей IDE с подсветкой синтаксиса помогает. Для каждой открывающей круглой скобки должна быть закрывающая скобка. Для каждой открывающей фигурной скобки должна быть закрывающая фигурная скобка. –

+0

Благодарим за помощь, ваше решение отлично поработало! Я собираюсь попробовать, как предложили другие пользователи, использующие Firebug. Еще раз спасибо. –

0

Вы пропускаете } и , здесь:

error: function(){console.log('Ajax Error'); //Debug Line 

Он должен иметь скобки, чтобы закрыть функцию и запятую, чтобы отделить следующую строку/свойство, как:

error: function(){console.log('Ajax Error'); }, //Debug Line 
              ^^ 

Последняя , но не в последнюю очередь вам нужно закрыть некоторые другие функции ...

Попробуйте следующее:

$(document).ready(function() { 
    //Event Listener for form submit 
    $('#form1').submit(function (e) { 
     e.preventDefault(); 
     console.log('Form Submitted'); //Debug line 
     $.ajax({ 
      type: 'POST', 
      url: 'indextest2.php', 
      data: $("#form1").serialize(), 
      error: function() { 
       console.log('Ajax Error'); 
      }, //Debug Line 
      success: function (response) { 
       console.log('Ajax Success'); //Debug Line 

       $(document).on("click", ".alert", function (e) { 
        bootbox.alert("Hello world!", function() { 
         console.log("Alert Callback"); 
        }); 
       }); 
      } 
     }); // added by me 
    }); // added by me 
}); // added by me 
+0

Благодарим вас за помощь, я не хочу звучать неблагодарно, но ваше решение требует настройки. Удаление одного ";" в третьей закрывающей скобке. Еще раз спасибо. –

+0

@ DanCundy, вы правы! Я не должен был помещать ';' внутри объекта! Все самое лучшее :) – Sergio

0

где определен alert function?

Если это

bootbox.alert("Hello world!", function() { 
    console.log("Alert Callback"); 
}); 

быть на самом деле это?

alert("Hello world!"); 
console.log("Alert Callback"); 
+0

Насколько я знаю @Jason. [Bootbox] (http://bootboxjs.com/#usage) –

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