ansaurus

Question

I can't solve this problem...( After a while my app error this...)

Answer 1

+2 A:

public void parse(InputStream is){
    if(!mjpeghttp.isCanceled())
      go();

What exactly does go() do? It calls connect(), what calls

if (!cancel) {
  onReturn(res.getStatusLine().getStatusCode(), is);
 }

And onReturn() calls parse() again. parse() calls go() calls connect() calls onReturn() calls parse() calls... It never(?) ends. Why does parsing have to connect? Every function call needs a bit of memory. After a lot of call, there is no more memory left and you get a Stack overflow exception.

You need to break the circle some where. You could remove this:

if(!mjpeghttp.isCanceled())
  go();

I don't think it even sleeps once

try {

  cancel = true;
  System.out.println("Sleep sleep!");
  Thread.sleep(60);

 } catch (Exception ie) {

Does it print at all?

How about?

imageSet=false;
while(!imageSet)
{
   if (connect())
     if(parse())
       imageSet=true;
}

Ishtar 2010-10-28 12:17:04

parsing have to connect again, because connect brings jpeg image, parse sets image. On the other hand, my handler sets this image to screen. Always a picture shoulds be coming...

atasoyh 2010-10-28 12:31:26

Answer 2

+2 A:

It looks as though you keep nesting calls over and over until the stack overflows (very appropriate problem for this site =P).

I'm guessing that you want to actually have the connection methods return a value instead of calling an onReturn method. That would prevent your recursive nesting and should get rid of this particular problem.

Marc 2010-10-28 12:17:31

thank you. But why did you think " very appropriate problem for this site =P)."

atasoyh 2010-10-28 12:34:40

Because the site is called stackoverflow, and the error you're getting is 'stackoverflow'...

Mark Ingram 2010-10-28 13:00:27

Answer 3

A:

I solve like this.. Thank you again...:) Absolutely I remove if(!mjpeghttp.isCanceled()) go();

// start connection to ip-camera
public void go() {

    if (mjpeghttp == null) {
        mjpeghttp = new MjpegHttp();
        mjpeghttp.start();

    }
    if (type.isMjpegStream()) {
        mjpeghttp.go(this);
    } else {
        while (mjpeghttp.isCanceled()==false)
            mjpeghttp.go(this);
    }

}

atasoyh 2010-10-28 12:59:05

ansaurus

tags:

views:

answers:

I can't solve this problem...( After a while my app error this...)

related questions