Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Заменить String в onError на Sealed class #36

Open
gorelov-a opened this issue Dec 16, 2019 · 0 comments
Open

Заменить String в onError на Sealed class #36

gorelov-a opened this issue Dec 16, 2019 · 0 comments

Comments

@gorelov-a
Copy link

Очень кривая реализация callback fun onError(error: String?), да и onSuccess(json: JSONObject) в OkListener.
Сделать в onError параметр Sealed class, для обработки ошибок. Мне важен тип ошибки: отмена пользователем доступа, или же это onBack, или это какая-то еще ошибка. Плюс вы возвращаете ЛОКАЛИЗИРОВАННЫЕ строки ошибок. Я не вывожу эти ошибки на экран, а завязываю логику. И для условия пришлось делать кучу const строк для одной ситуации. Это не правильный подход. Посмотрите как сделает Facebook SDK. Верните эти строке в поле класса.

Пример кода обработки ошибки, которого быть не должно:
private val OK_SDK_AUTH_CANCELED_CODES = listOf( "access_denied", "Authorization was canceled", "Авторизация была отменена" )

Так же в onSuccess хочется взять из объекта класса поле accessToken а не парсить JSON в режиме debug и вытаскивать из поля access_token, которое может поменяться и никто не заметит ошибки.

@gorelov-a gorelov-a changed the title Заменить String в onError на класс Sealed class Заменить String в onError на Sealed class Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant