Tibr Mar 27 2011 at 16:48

Отслеживание утечек памяти в Android приложениях

3 min

8.3K

Development for Android*

+19

Comments 13

zochek Mar 27 2011 at 17:08

Реализация Singleton как всегда неправильная. Вот тут всё подробно расписано.

Eleph Mar 27 2011 at 17:43

Не могу согласиться с тем что реализация Singleton в корне не правильная. Проблема данной реализации хорошо известна, и для однопоточного кода пример работает, как мне кажется, на ура. В любом случае, спасибо за комментарий.

zochek Mar 27 2011 at 17:59

Вот вам еще несколько замечаний:
1. Зачем пустой конструктор?
2. Статические поля не должны иметь префикс m
3. Название метода getThis() это просто ужас!

Eleph Mar 27 2011 at 18:15

Спасибо.
В данной статье я не пытался изобразить нечто вроде идеально написанного класса. Если вы внимательней прочитаете статью, то обратите внимание на следующее предложение: «Класс по понятным причинам был упрощен и слегка модифицирован, но свою основную функцию он выполняет». Главное — познакомить читателей с возможностью использования такого подхода и я очень даже верю в то, что существует реализация данного класса куда более «правильная» даже в сравнении с той что используется в нашем реальном проекте.
Если же реализация вам ну совсем не нравиться — присылайте свой вариант — я обязательно добавлю его к статье.

zochek Mar 27 2011 at 18:31

> «Класс по понятным причинам был упрощен и слегка модифицирован, но свою основную функцию он выполняет»

Упрощен и небрежно написан — это две большие разницы.

> Главное — познакомить читателей с возможностью использования такого подхода…
Согласен, но гораздо приятнее знакомиться с подходом на примере хорошего кода, а не того, который только и делает что «свою основную функцию выполняет».

skorn Mar 28 2011 at 00:16

Пустой конструктор затем, что private.

zochek Mar 28 2011 at 07:43

Согласен. Пустой приватный конструктор конечно же нужен. Не нужен явный вызов super().

zochek Mar 27 2011 at 18:09

И на закуску ;) Класс должен быть объявлен как final

Eleph Mar 27 2011 at 18:16

Уговорили :) Жду от вас «идеальную» реализацию.

Tqb Mar 28 2011 at 03:34

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

Что-то я не понял, дык вы же и есть Tibr. Кто кому приглашением то поделился? о_0

Tibr Mar 28 2011 at 04:51

Согласен, было понятнее, пока мой комментарий насчет авторства загадочно не пропал (спс НЛО).

vadia Mar 28 2011 at 08:23

Огромное спасибо автору! сегодня обязательно потрачу время и проверю объекты и их коллекции на вшивость.

sagus Jan 14 2013 at 10:16

Видимо никто кроме меня не пользуется этим классом)
У Вас ошибка вот тут

if (index == -1) {
                names.add(names.getClass().getSimpleName());
            }

Надо сделать вот так, полагаю

if (index == -1) {
            names.add(className);
        }