vbondarevsky / Connector

Коннектор: удобный HTTP-клиент для 1С:Предприятие 8
Apache License 2.0
637 stars 207 forks source link

Аккуратно обрабатывать ответы с пустым телом для методов КакойТоМетодJson #80

Closed vbondarevsky closed 10 months ago

vbondarevsky commented 3 years ago

http://forum.infostart.ru/forum9/topic182385/message2709746/#message2709746

Не всегда чей то API может возвращать чтото в теле ответа, При этом все в JSON. К примеру Обертка MongoDB на RestHeart при отправке POST запроса вернет только код 201 ОК. При этом коннектор упадет в исключение при попытке сериализации пустого тела в JSON

Как вариант в таких случаях возвращать Неопределено/Новый Структура/Новый Соответствие

zeegin commented 3 years ago

см #95

leemuar commented 10 months ago

Считаю что необходимость не достаточно обоснована. Методы с суффиксом Json - это хелперы, призванные упростить код там где точно приходит JSON. Использовать их стоит там где ты точно уверен в возвращаемом формате. Не уверен, может приходить что-то другое - не надо их использовать. Лучше брать пару Метод() + Если + КакJson()

Возврат Неопределено при пустом коде не улучшит код по сравнению с Метод() + Если + КакJson(), поэтому имхо не стоит это реализовывать. По крайней мере пока не появятся более весомые аргументы

zeegin commented 10 months ago

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

zeegin commented 10 months ago

Пример с монго от Володи прям живой)

leemuar commented 10 months ago

до большей полезности хелпера

@zeegin Нужны аргументы и критерии оценки полезности. Я не вижу бенефитов. Но может есть примеры, о которых я не знаю, где эта полезность будет очевидна. Приведи примеры кода без этой функциональность и такой же пример с ней, в чем будет заключаться полезность.