java.lang.OutOfMemoryError. Android

Question

I'm not too far on in developing my app and i'm getting a java.lang.OutOfMemoryError. Here's the logcat:

02-24 14:02:54.957: E/art(28628): Throwing OutOfMemoryError "Failed to allocate a 28 byte allocation with 8 free bytes and 8B until OOM" (recursive case)
02-24 14:02:54.965: E/art(28628): "FinalizerDaemon" daemon prio=5 tid=8 Runnable
02-24 14:02:54.965: E/art(28628):   | group="system" sCount=0 dsCount=0 obj=0x12c260e0 self=0xac4a5400
02-24 14:02:54.965: E/art(28628):   | sysTid=28641 nice=0 cgrp=apps sched=0/0 handle=0xac4b9b00
02-24 14:02:54.965: E/art(28628):   | state=R schedstat=( 2442370612 111306164 635 ) utm=229 stm=15 core=1 HZ=100
02-24 14:02:54.965: E/art(28628):   | stack=0xb40fe000-0xb4100000 stackSize=1036KB
02-24 14:02:54.965: E/art(28628):   | held mutexes= "mutator lock"(shared held)
02-24 14:02:54.965: E/art(28628):   at com.android.internal.os.BinderInternal$GcWatcher.finalize(BinderInternal.java:51)
02-24 14:02:54.965: E/art(28628):   at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:190)
02-24 14:02:54.965: E/art(28628):   at java.lang.Daemons$FinalizerDaemon.run(Daemons.java:173)
02-24 14:02:54.965: E/art(28628):   at java.lang.Thread.run(Thread.java:818)
02-24 14:02:54.965: E/System(28628): Uncaught exception thrown by finalizer

I'm trying to make a list and anytime I retrieve the data to make the list I get the error.

Here is class that extends the list fragment

`

public class BillReminderListFragment extends ListFragment {
private ArrayList<Bill> mBills;

public void onCreate(Bundle savedInstanceState){
    super.onCreate(savedInstanceState);
    mBills = BillLab.get(getActivity()).getBills(); <---(this is what is causing the error)
}
}

Here is the class that holds the data

public class BillLab {
private ArrayList<Bill> mBills;
private static BillLab sBillLab;
private Context mAppBillContext;

private BillLab(Context appBillContext){
    mAppBillContext = appBillContext;
    mBills = new ArrayList<Bill>();
    for (int i = 0; 1 < 100; i ++){
        Bill b = new Bill();
        b.setTitle("Bill #" + i);
        b.setDate(new Date());
        mBills.add(b);
    }
}

public static BillLab get(Context c){
    if (sBillLab == null){
        sBillLab = new BillLab(c.getApplicationContext());


    }
    return sBillLab;
}

public Bill getBill(UUID id){
    for (Bill b: mBills) {
        if (b.getId().equals(id))
            return b;
    }
    return null;
}

public ArrayList<Bill> getBills(){
    return mBills;
}

}

Please help. This is my final project in college and I don't have much time. Thank you.

`

there is an infinite loop.

Gopal Singh Sirvi
– Gopal Singh Sirvi

2015-02-24 07:20:11 +00:00
Commented Feb 24, 2015 at 7:20 — Gopal Singh Sirvi
– Gopal Singh Sirvi, Commented Feb 24, 2015 at 7:20

ekchang · Accepted Answer · 2015-02-24 06:37:03Z

5

Your for loop has a typo: for (int i = 0; 1 < 100; i ++)

should be i < 100. It's running out of memory because 1 is always less than 100.

answered Feb 24, 2015 at 6:37

ekchang

9395 silver badges11 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

user3332929 Over a year ago

Thank you so much. I was really stressed when I was writing the code and I wasn't testing it. I'm suppressed I made such a silly mistake `

ekchang Over a year ago

Happens to all of us!

Nagaraju V · Accepted Answer · 2015-02-24 06:39:40Z

1

Change following line

for (int i = 0; 1 < 100; i ++){

to

for (int i = 0; i < 100; i ++){

answered Feb 24, 2015 at 6:39

Nagaraju V

2,7571 gold badge16 silver badges19 bronze badges

Comments

zeisuke · Accepted Answer · 2015-02-24 08:41:11Z

this method had 2 root cause, one is mention by @Nag edi, but the root cause for made java.lang.OutOfMemoryError is: -

private BillLab(Context appBillContext){
    mAppBillContext = appBillContext;
    mBills = new ArrayList<Bill>();
    for (int i = 0; 1 < 100; i ++){  // this line will cause the infinity loop only    
        Bill b = new Bill();      // this is the root cause to create every single object for each loop.
        b.setTitle("Bill #" + i);
        b.setDate(new Date());
        mBills.add(b);
    }
}

to :-

   private BillLab(Context appBillContext){
        mAppBillContext = appBillContext;
        mBills = new ArrayList<Bill>();
        Bill b = null;
        for (int i = 0; i < 100; i ++){
            b = new Bill();
            b.setTitle("Bill #" + i);
            b.setDate(new Date());
            mBills.add(b);
        }
    }

Collectives™ on Stack Overflow

java.lang.OutOfMemoryError. Android

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related