JQuery: как сделать свой $.post?

Question

[PlatinumArcade]

У меня во всем проекте отправка $.post выглядит примерно так:

$.post('/some/page',{'ajax':'1', 'some':'some'}, function(rJson) <br>
{<br>
    var r = $.parseJSON(rJson);<br>
    if(r.status === 'error') {<br>
        alert(r.msg);<br>
        return;<br>
    }<br>
    ...<br>
});<br>

Отличаются только параметры ну и собственно сам код, если статус ответа не ошибка.



Как можно сделать свой $.postJson и использовать только его? А то чувствую что не дело писать одно и то же… Надо, чтоб было так:

$.postJson('/some/page',{'some':'some'}, function(r) <br>
{<br>
    // Тут уже всё будет проверено на r.error<br>
    ...<br>
});<br>

Answer 1

[sdevalex]

$.postJson = function(url, data, callback){
   $.post('/some/page',{'ajax':'1', 'some':'some'}, function(rJson) 
   {
       var r = $.parseJSON(rJson);
       if(r.status === 'error') {
          alert(r.msg);
           return;
       }
    
       callback(r);
   });
}

Comments

[sdevalex]

Вторая строчка такая…

$.post(url, data, function(rJson) ...

[PlatinumArcade]

О, то что надо. Спасибо!

[lair]

«Thou shalt not touch library objects».

Answer 2

[akral]

В оригинальном коде три мелочи, достойные улучшения.

1. Парсить JSON самому не надо:

$.ajax({
  dataType: 'json'
});

2. Передавать {'ajax':'1'} не надо, AJAX посылает заголовок X-Requested-With: XMLHttpRequest, по которому нужно определять, что это AJAX.

3. Кавычками обрамлять ключи в словаре не нужно:

{
  some: 'some'
}

Ну и да, ошибки должны возвращатся с кодом HTTP, а не своим полем.

Comments

[PlatinumArcade]

По поводу XMLHttpRequest — вроде бы он иногда не отправляется. Зависит толи от браузера, толи еще от чего. Или прям в 100% случаев сработает?

[akral]

Этот хедер ставит сам jQuery. Никаких упоминаний о браузерах, не поддерживающих этого я не слышал.

Answer 3

[Mithgol]

Действуйте согласно руководству Шевчука.

Answer 4

[lair]

У вас, на самом деле, ошибка в проекте. Если на сервере что-то не так, он не должен возвращать успешный JSON со свойством status, он должен возвращать HTTP error status. После чего вы радостно вешаете глобальный обработчик на $.ajaxError и наслаждаетесь результатом.

Comments

[PlatinumArcade]

Ну такую обработку уже можно допилить в $.postJson.

[lair]

Это называется «костыль». Вместо того, чтобы разделить ситуации на нормальные и аварийные еще на сервере, а потом на клиенте использовать штатные средства без врапперов и прочих костылей, вы заранее усложняете себе работу.

Поверьте, исключения придумали не просто так.

[Wott]

Вы таким макаром смешиваете контексты — и ошибки в работе транспорта и проблемы с представлением и проблемы на уровне приложения вы мешаете в одну кучу. Конечно если цель — сделать запись в логе — то да, так проще. Но если возможны корректные обработки ситуаций, перезапросы, то status в ответе и эксепшен на нем правильнее.

[lair]

Я как раз ничего не смешиваю. Я разделяю — ошибочную ситуацию и нормальную. Ошибочная должна обрабатываться по-одному, нормальная — по-другому.

Вам надо разделить разные виды ошибок? Пожалуйста: обрабатывайте error, коды http и содержимое ответа внутри ajaxError.

Вам нужен retry? Так вам тем более нужно лезть в ajaxError, чтобы его сделать, потому что недоступность сервера будет только там. А следовательно, если вы не хотите дублировать код, все остальные ошибки должны идти туда же.

Answer 5

[ServDev]

заверните все в функцию и передавайте туда 2 параметра url и data
data = как json строка — потому что по сути {'ajax':'1', 'some':'some'} и есть json
и тогда получиться

function sendJson(url, data){
	$.post(url,data, function(result) {
	    
	    if(r.status === 'error') {
	        alert(r.msg);
	        return;
	    }
	  //TODO check result
	}, 'json');
}

Answer 6

[lair]

Еще можно использовать $.ajaxPrefilter, и спрятать обработку в него. Помогает, если действительно везде одинаково.