2016-04-06 1 views
2

Я пытаюсь настроить мою переменную count на get и частную вспомогательную функцию для настройки на основе других членов. Несколько потоков каждый вызывает setCount() как deadlineNoteVisible и referencesAvailable изменение.Android OutOfMemoryError: Не удалось выделить выделение байтов 57993496 с 16764448 бесплатными байтами

С synchronized для всего метода setCount(), все хорошо, но когда я добавить synchronized(this) к get() вызову, я получить:

OutOfMemoryError: Failed to allocate a 57993496 byte allocation with 16764448 free bytes and 32MB until OOM 

код (написанный на Котлин):

var count: Int = 0 
    // Why does adding synchronized here explode? 
    get() = synchronized(this) { count } 
    private set 

private fun setCount() { 
    synchronized(this) { 
     count = 0 
     if (deadlineNoteVisible) { 
      count += 1 
     } 
     if (referencesAvailable) { 
      count += 1 
     } 
    } 
} 

Траектория следа:

Error reporting crash 
java.lang.OutOfMemoryError: Failed to allocate a 57993496 byte allocation with 16764448 free bytes and 32MB until OOM 
    at java.lang.AbstractStringBuilder.enlargeBuffer(AbstractStringBuilder.java:95) 
    at java.lang.AbstractStringBuilder.append0(AbstractStringBuilder.java:125) 
    at java.lang.StringBuffer.append(StringBuffer.java:278) 
    at java.io.StringWriter.write(StringWriter.java:123) 
    at com.android.internal.util.FastPrintWriter.flushLocked(FastPrintWriter.java:358) 
    at com.android.internal.util.FastPrintWriter.appendLocked(FastPrintWriter.java:303) 
    at com.android.internal.util.FastPrintWriter.write(FastPrintWriter.java:625) 
    at com.android.internal.util.FastPrintWriter.append(FastPrintWriter.java:658) 
    at java.io.PrintWriter.append(PrintWriter.java:691) 
    at java.io.PrintWriter.append(PrintWriter.java:687) 
    at java.io.Writer.append(Writer.java:198) 
    at java.lang.Throwable.printStackTrace(Throwable.java:324) 
    at java.lang.Throwable.printStackTrace(Throwable.java:300) 
    at android.util.Log.getStackTraceString(Log.java:343) 
    at com.android.internal.os.RuntimeInit.Clog_e(RuntimeInit.java:61) 
    at com.android.internal.os.RuntimeInit.-wrap0(RuntimeInit.java) 
    at com.android.internal.os.RuntimeInit$UncaughtHandler.uncaughtException(RuntimeInit.java:86) 
    at com.crashlytics.android.core.CrashlyticsUncaughtExceptionHandler.uncaughtException(CrashlyticsUncaughtExceptionHandler.java:249) 
    at java.lang.ThreadGroup.uncaughtException(ThreadGroup.java:693) 
    at java.lang.ThreadGroup.uncaughtException(ThreadGroup.java:690) 
+2

Похоже, у вас есть обработчик сбоев «crashlytics», который был запущен, когда было выбрано неперехваченное исключение, и он пытается сгенерировать строку 55MB при печати трассировки стека. Прикрепление отладчика, который перехватывает нечеткие исключения, по крайней мере позволит вам идентифицировать сбой, который начинает работу. – fadden

+0

«Синхронизированный» метод, вероятно, является «красной селедкой», это просто то, что привело к тому, что протекающее приложение стало слишком большим. Можете ли вы профилировать приложение yoyr и узнать, что использует память? – sisyphus

+0

Зачем вам все равно синхронизировать чтение? –

ответ

5

На самом деле у вас есть StackOverflowError, потому что вы безоговорочно звоните getCount() с getCount(). Я предполагаю, что это просто опечатка и код должен быть:

var count: Int = 0 
    get() = synchronized(this) { field } 
    private set 

field идентификатора здесь позволяет получить доступ к полю подкладочных собственностей. Вы можете прочитать больше о базовых полях here.

Смежные вопросы

 Смежные вопросы