J
они отдельные
Size: a a a
2020 September 02
J
в сумме придёшь к тому что ты написал
С
Это лишь на словах так. У меня не получается добавить что-то не изменив уже написанное и сложность постоянно усиливается
J
не лепи всё в отдельную кучу, старайся чтобы каждый метод был как можно меньше
VV
Ну, это уже вопрос к гибкости твоего кода
J
Это лишь на словах так. У меня не получается добавить что-то не изменив уже написанное и сложность постоянно усиливается
это оттого что ты делаешь методы которые делают сразу всё, их всегда очень тяжело поддерживать потом и читать код
J
это нормальная ошибка в начале изучения, когда ты думаешь зачем мне много мелких когда я могу написать сразу всё в одном, но когда у тебя становится кода больше это всё превращается в нечитаемую массу
J
вытащи отсюда получение location внутри и ты поймёшь как это делается
С
вытащи отсюда получение location внутри и ты поймёшь как это делается
Что это даст ? Если там всё равно есть получение из бд которому location не нужен. Вообще не знаю как это должно быть организовано ведь location тоже надо в бд сохранять и так же получать ... Хз я застрял
С
А потом ещё тоже самое дублировать для списка погоды на неделю
J
Начни делать как говорят
J
Не думай сразу обо всем
VV
@serg_mgn а ещё старайся меньше зависеть от реализации и больше опирайся на абстракции, это тоже поможет твоему коду быть более гибким ;)
С
это нормальная ошибка в начале изучения, когда ты думаешь зачем мне много мелких когда я могу написать сразу всё в одном, но когда у тебя становится кода больше это всё превращается в нечитаемую массу
Я и не пишу всё в одном методе, просто я не знаю как это иначе писать т.к репозиторий должен вернуть данные либо из базы либо из интернета, а для интернета ему нужно место положение... при это местоположение тоже надо как то вернуть
При этом там нет обработки на случай если в базе тоже пусто ... и наверно ещё что не учтено
При этом там нет обработки на случай если в базе тоже пусто ... и наверно ещё что не учтено
ЮЛ
Я и не пишу всё в одном методе, просто я не знаю как это иначе писать т.к репозиторий должен вернуть данные либо из базы либо из интернета, а для интернета ему нужно место положение... при это местоположение тоже надо как то вернуть
При этом там нет обработки на случай если в базе тоже пусто ... и наверно ещё что не учтено
При этом там нет обработки на случай если в базе тоже пусто ... и наверно ещё что не учтено
Тебе не понятно как выстроить архитектурно ? Или тебе не понятно какие инструменты использовать?
J
Я и не пишу всё в одном методе, просто я не знаю как это иначе писать т.к репозиторий должен вернуть данные либо из базы либо из интернета, а для интернета ему нужно место положение... при это местоположение тоже надо как то вернуть
При этом там нет обработки на случай если в базе тоже пусто ... и наверно ещё что не учтено
При этом там нет обработки на случай если в базе тоже пусто ... и наверно ещё что не учтено
у тебя этот метод что должен сделать, получить данные? Или получить данные и местоположение? Второе это нарушение принципа единственной ответственности. Ты должен одним методом получить место, вторым запросить погоду. Сделай так.
ЮЛ
Я и не пишу всё в одном методе, просто я не знаю как это иначе писать т.к репозиторий должен вернуть данные либо из базы либо из интернета, а для интернета ему нужно место положение... при это местоположение тоже надо как то вернуть
При этом там нет обработки на случай если в базе тоже пусто ... и наверно ещё что не учтено
При этом там нет обработки на случай если в базе тоже пусто ... и наверно ещё что не учтено
Тебе нужно смотреть в сторону реактивного программирования RxJava
J
чтобы совсем утонуть в новых знаниях, дай человеку с декомпозицией разобраться ))
ЮЛ
чтобы совсем утонуть в новых знаниях, дай человеку с декомпозицией разобраться ))
Тема сложная не спорю, но это мазафака базука на любой случай жизни))))
С
Я выбрал вместо rx корутины для начала. потом когда всё будет написано перепишу на rx , а потом на mvp