Ошибка с кодировкой

[leon0399]

Ошибка с кодировкой

[sshikov]

Приводит к другим, непредсказуемым проблемам. Например, приходит ответ 400 от сервера, который не может обработать запрос в неверной кодировке. Если эту проблему исправить - то все работает.

[leon0399]

@sshikov Сегодня-завтра исправлю, плюс будут добавлены новые функции из новых обновлений API

[sshikov]

Спасибо. Было бы очень неплохо, если бы обработка исключений не только писала что-то в логи, но и возвращала какой-то результат (JSON) с ошибками. А то сейчас возвращается null, который потом вызывает всякие другие проблемы при попытке его обработать. Ну или уж пробросить исключение наружу, чтобы не было иллюзий, что запрос успешно завершился.

[leon0399]

@sshikov Наверное, лучше будет пробрасывать ошибку, так как методы API возвращают какие-либо классы, а не JSON-строку

[vapv]

Спасибо, ждем обновлений!

Согласен с @sshikov , по исключениям хотелось бы хотя бы видеть проброс ошибок хотя бы для основных ошибочных кейсов :

1. Ошибка аутентификации
2. Нет денег на балансе
3. Сервер не доступен
4. Ошибка нормализации адреса
   ..

Сейчас в IOException заворачиваются ошибки 1-2, что затрудняет их обработку.

[sshikov]

Я имел в виду, что можно вместо JSON ответа сервера сформировать свой, содержащий описание ошибки (или просто вернуть объект с ее описанием). Но просто пробросить exception тоже вполне нормально.

[sshikov]

Да, неплохо было бы toString() реализовать для классов модели, а то там от Object, т.е. бесполезный.

[leon0399]

@vapv @sshikov Народ, вы конечно извините, что пообещал, но так ничего и не сделал. Просто сессия подкралась незаметно и времени почти нет

[sshikov]

Да ладно, не стоит. Код же открыт, я лично для себя кодировку уже исправил, а остальное - подождем )