Помогите прошу, как найти эти ресурсы повторяющиеся

Duplicate value for resource 'attr/labelVisibilityMode' with config 'DEFAULT' and product ''

Вот это выдает, что блин вообще такое, впервые встречаюсь

Вот на этот участок кода перекидывает

и вот еще

продолжение

- view binding у StartFragment используется не по уставу, чревато утечками
- там же: вы сделали binding nullable, т.е. извратили саму идею того, что он должен быть всегда доступен и без nullable
- там же: кривое использование lateinit - если вы вынуждены вызывать isInitialized, значит, вы что-то делаете не так.
- hardcoded размеры в лейаутах

Дальшн не копал. В целом вполне прилично для уровня стажера и я не вижу причин отказа в стажировке. Наверное, у них очень высокие требования или зажрались :)

Спасибо большое

https://github.com/nikbrik/OpenWeatherMapClient/blob/main/app/src/main/java/com/nikbrik/openweathermapclient/extensions/Extensions.kt

то что у тебя 2 метода для показа тоаста уже говорит о том что ты не знаешь про DRY


https://github.com/nikbrik/OpenWeatherMapClient/blob/main/app/src/main/java/com/nikbrik/openweathermapclient/network/Networking.kt

этот класс просто покаывает что ты не в курсе про сервис локатор, инкапсуляцию, нейминг и т.д.


https://github.com/nikbrik/OpenWeatherMapClient/blob/main/app/src/main/java/com/nikbrik/openweathermapclient/ui/start/StartViewModel.kt

нет понимания солида. репа создается прямо внутри вм

Wearable sendMessageClient только в одну сторону работает или с часов тоже можно отправлять обратно?
у меня с телефона норм идут сообщения, а с часов обратно нет
кто-нибудь сталкивался?

Помогите пожалуйста, вопрос выше

Я проглядел, у вас binding зануляется. Это хорошо. Но то, что само поле nullable - нехорошо. Избегайте nullable и lateinit везде, где возможно.

Есть всякие недочеты, выше писали, но в целом вполне прилично, тем более для стажера, видимо звезды не совпали или перенабор

Есть понимание, что при каждом обращении к этим полям буду создаваться новые инстансы Api?

ApiKey намеренно не private кста?

Я бы openWeatherApi и geotreeApi через by lazy { retrofit.create} бы объявил, и нормалин.

и оно будет висеть в памяти...

А чо б не?

ну если оно юзается в скоупе апликейшн то ок

Статик как бы намекает на скоуп. Наверное. Хз, вижу статик, сразу подразумеваю максимальный из возможных скоупов.