Раскритикуйте код

// Singleton instance
private static LocationWorker instance = null;

1) не пиши коменты в стиле КО

2) если синглтон юзается в многопоточном окружении то жёпа

А что оно делает? И скриншоты можно? Мне лень ставить Android Studio чтоб это собрать.

Наверное это автосгенериованные комменты? Там во всех сорцах такое

вот ты нафигачил синглтонов моделей и тп, а кучу логики все также херачишь там где возникла физиологическая потребность поиспражняться кодом:

    private void updateUi() {
        runOnUiThread(new Runnable() {
            @Override
            public void run() {
                toggleRefreshAnimationOff();
                placeTv.setText(mCurrentWeather.getPlace());
                if (mCurrentWeather.getTemperature() < 0) {
                    int reversedTemperature = mCurrentWeather.getTemperature() * -1;
                    minusSymbolTv.setVisibility(View.VISIBLE);
                    temperatureTv.setText(String.valueOf(reversedTemperature));
                } else {
                    minusSymbolTv.setVisibility(View.INVISIBLE);
                    temperatureTv.setText(String.valueOf(mCurrentWeather.getTemperature()));
                }
                String formattedHumidity = String.format("%s", mCurrentWeather.getHumidity() + "%");
                humidityValueTv.setText(formattedHumidity);
                pressureValueTv.setText(String.valueOf(mCurrentWeather.getPressure() * 0.75));
                String uppercaseSummary = mCurrentWeather.getSummary()
                        .substring(0, 1)
                        .toUpperCase()
                        + mCurrentWeather.getSummary()
                        .substring(1);
                weatherSummaryTv.setText(uppercaseSummary);
            }
        });
    }

Про многопоточность и синхронизацию / блоки знаю. Но тут пока нет кейса, чтобы этот воркер дергали из нескольких потоков. Но на заметку возьму.

А вот тут поподробнее. Логику и форматирование в модель сувать не лучшая идея, как мне кажется. Контроллер он на то и контроллер: взять данные из модели, адаптировать и вывести на вьюхи, поправьте, если не прав.

Нет, все комменты написаны руками, возможно какие-то действительно лишние.
Скрины: https://pp.vk.me/c623629/v623629390/52499/abK_MGV3HCk.jpg
https://pp.vk.me/c623629/v623629390/524a3/g9cqXRKqAi0.jpg

./gradlew assembleRelease если не ошибаюсь

Логику и форматирование в модель сувать не лучшая идея, как мне кажется. Контроллер он на то и контроллер: взять данные из модели, адаптировать и вывести на вьюхи, поправьте, если не прав.

Ты возвел MVC в статус библии, в то время как речь хотяб о разделении на методы по выполняемым функциям - тогда читать эту спагеттину будет проще.

Чо?

https://github.com/Hochland/Luna/blob/master/app/src/main/java/com/hochland38...

очередной фейл

скорее всего это начальный этап после обработки мозга фразами «пиши коменты сцуко»

Как раз таки не фейл, у класса переопределен дефолтный конструктор и если он вызывается без параметров - то создается заглушка.

String.format("%s", mCurrentWeather.getHumidity() + "%");

Ы?

Почему это КО? Мне, как довольно далекому от джавы человеку, не очевидно, что это синглтон.

Логику и форматирование в модель сувать не лучшая идея

Это почему же? Как раз в модели и содержится логика, а контроллер только готовит данные модели для вьюшки. Даже паттерн есть такой «fat model skinny controller»

Может я чего не понимаю, но если к одной модели привязано 5 активити, например, и все дергают одинаковые данные, но отображают в конечном счёте по разному, не буду же я в таком случае пилить в модель методы для всех 5 вариантов.

Привет дружок, это типичный доступ к полю из разных потоков, а ты рассказываешь что знаешь про это.

гг, я даже не стал читать эти макароны, а там оказывается жемчуг есть!

Нефиг далекому от жавы человеку в оной копаться.

Джава-нубасы могу копаться. Какие-нибудь джуниоры, прочитывшие джава за 21 день, могут копаться. Да много кто может. Если код понятен и таком человеку, это лучше, чем если непонятен.

Они могу у меня в ботинках копаться, мне теперь и там коменты писать?

Если твои чертежи ботинков используют другие люди, то да.

все дергают одинаковые данные
одинаковые данные

в чем проблема то? Даже на уровне логики продумай - гораздо проще описать операции с моделью в 1м классе, а в 5 активити, используя методы модели, получить различные реализации интерфейсов для представления этих самых одинаковых данных.
А вообще, на лицо непонимание MVC (если, конечно, мы говорим о MVC).

Самое смешное, что 90% всех приложений для Android написано в стиле, подобном этому говно-коду, и ничего - пользуемся!

Как только отдашь код на саппорт жуниарами — так обязательно залезут в твои ботинки на тему «а хуле тут нипанятна? тут надо все переписать!»

говно-коду

Аргументы или gtfo

Значит ты их хреново воспитываешь, я парочку раз таких умников приколол и стали шелковыми.

Значит ты их хреново воспитываешь, я парочку раз таких умников приколол и стали шелковыми.

Экий ты простой :) Ты вообще не сможешь повлиять, кому и куда отдадут :)

Бгг, ну тогда беги из той конторы

Зачем? Я даже времени не буду тратить на твой говнокод.

Только сейчас посмотрел код. ТС нужно обратить внимание на последний пункт в этот статье - http://habrahabr.ru/post/59005/

Смотря что вы имели ввиду, выводы или God Object? Если GO, я стараюсь делить функциональность на классы, чтобы каждый из них выполнял только одну функцию, настолько, насколько это вообще возможно.

Предпоследний я хотел сказать, но написал не то. Извиняюсь. Бездумного комментирования у тебя более чем дофига, оно будет резать глаза, работодателям в том числе.

Хорошо, комментарии подчищу, мне лично они не нужны, я свой код понимаю, но вдруг другим пригодятся.