lostm1nd / HackBulgaria

Projects related to the courses at HackBulgaria
1 stars 0 forks source link

Неконсистентност в $.ajax #10

Closed RadoRado closed 10 years ago

RadoRado commented 10 years ago

В MysterLock-a„ правиш следното:

    $.ajax({
      type: 'GET',
      url: 'http://localhost:3000/secret/' + token,
      dataType: 'json',
      success: function(response) {
        concatenatedSecrets += response.secret;
        disableButton(index);
        enableButton(index + 1);
      },
      error: function() {
        alert('Server not responding. Try later.');
      }
    });

В останалата част от кода се ползва:

$.ajax({..)).done(callback);

Ако искаш да хванеш и грешката, има .fail():

$.ajax({..))
.done(callback)
.fail(callback);

Тук има документация - http://api.jquery.com/deferred.fail/

Идеята е кодът да е консистентен в това, което използва - избери едно от двете (Deffered са за предпочитане, тъй като ги въведоха за борене с callback hell-a и за използване като Promises)

lostm1nd commented 10 years ago

Тази част от кода, която е с .done е първоначалния код. Но си прав, че когато имаме вече написан код трябва да се спазва въведения стил. Ще го оправя ; )