А подскажите по кодстилю. Есть 3 класса: GeneralErrorHandler(private val onFailure: ((GeneralError) ->

Question

А подскажите по кодстилю. Есть 3 класса: GeneralErrorHandler(private val onFailure: ((GeneralError) ->

Unit)? = null), который является наследником Consumer<Throwable> rxJava и хендлит ошибку.

Есть GeneralErrorFabric, со статическим методом error(throwable: Throwable): GeneralError который принимает на себя Throwable, парсит ошибку и превращает её в стандартную для проекта GeneralError (наследник Throwable).

И сам GeneralError

Я поместил их в один файл, т.к. в общем каждый класс занимает не более 10-40 строк кода и сам файл вышел в 80 строк c учётом импортов, ведь если верить https://kotlinlang.org/docs/reference/coding-conventions.html#source-file-organization то так делать можно (ведь они связаны по себе логикой и + достаточно небольшие классы).

Но будучи единственным android\kotlin разрабом мой код глянули ios'ники, чтобы и самим понимать как андроид приложение работает, так и в целом что бы говнокода по минимум было и им жутко не понравилось что это всё в одном файле живёт и что это делать так не надо. Вопрос: действительно ли стоит разбить это на файлы или просто в iOS другой мир которым и не привычно такое

#kotlin #programming #russian

0

29.10.2019

1 ответов

2 просмотра

Andrey Antipov · Accepted Answer

Andrey Antipov

С точки зрения официальной конвенции оба варианта вполне приемлемы. Видимо стоит сделать так, чтобы команде было проще понимать код.

0

29.10.2019

169 похожих чатов

А подскажите по кодстилю. Есть 3 класса: GeneralErrorHandler(private val onFailure: ((GeneralError) ->

1 ответов

Похожие вопросы