Отправил тестовое задание на позицию Android разработчика, но в ответ они написали:
После ознакомления с тестовым заданием, мы отметили, что у вас недостаточный уровень знания принципов ООП и принципов построенийAndroid-приложений. Если вы подтянете свой уровень знаний, лучше изучите ООП и каким образом применять его принципы для построения приложений, то мы могли бы вернуться к вопросу рассмотрения вашей кандидатуры в будущем.
Я самоучка, учился по книгам, статьям и т.д. , можете опытным глазом посмотреть что в моем коде не так в плане ООП и принципов построения? И как апргрейдить эти навыки? Код на gitHub, т.к. классов не один и не два https://github.com/Vlad161/Products
catch (JSONException e) { }
То, что у Вас много классов, еще не говорит, что Вы умеете пользоваться ООП. И вообще, в java сложно написать что то без единого класса.JSONReadFromAsset.readJsonFromAsset()
молча глотаются все исключения, в следствии чего метод может вернутьnull
, на вызывающей стороне же отсутствуют соответствующие проверки. Как результат — потенциальныйNullPointerException
вMainActivity.fillLists()
. И таких моментов в коде немало. Ну и вообще, отлавливать исключения по базовому типу (Exception
,Throwable
) — плохая идея.