[DD12]

Hello Everyone

I have following code:

import net.rim.device.api.ui.component.*;
import net.rim.device.api.ui.decor.*;
import net.rim.device.api.synchronization.*;
import net.rim.device.api.math.*;
import net.rim.device.api.system.*;
import net.rim.blackberry.api.phone.phonelogs.*;

class Phone_Call_Log extends UiApplication
{
MainScreen _screen = new MainScreen();
Phonethread _phone;
PhoneLogs _logs;

public static void main(String arg[])
{
Phone_Call_Log application = new Phone_Call_Log();
application.enterEventDispatcher();
}

public Phone_Call_Log()
{
_screen.setTitle("Phone Log";
_screen.add(new RichTextField("Activities on your Phone");
_screen.add(new SeparatorField());
pushScreen(_screen);
_phone = new Phonethread();
_phone.start();
}

private final class Phonethread extends Thread
{

public Phonethread()
{
super();
}

public void run()
{
_logs = PhoneLogs.getInstance();

for(int i=0;i< (_logs.numberOfCalls(_logs.FOLDER_NORMAL_CALLS));i ++)
{

PhoneCallLog callLog = (PhoneCallLog) _logs.callAt(i, _logs.FOLDER_NORMAL_CALLS);

String number=callLog.getParticipant().getNumber();

if(callLog.getType() == callLog.TYPE_RECEIVED_CALL)
{
_screen.add(new RichTextField ("You received a call from: " + number));
}

if(callLog.getType() == callLog.TYPE_PLACED_CALL)
{

_screen.add(new RichTextField ("The number you dialed was: ");
// _screen.add(new RichTextField ("it was made on: "+data));
}



}

}
}
}

---- End of Code ---

I receive java.lang.IIIlegalStateException error at line which is in bold.

Any Ideas why?

Thanks

[SamuelD]

Hello,
I don't see the line in bold, but I think your problem is near there :

_screen.add(new RichTextField ("You received a call from: " + number));

To modify a screen from a thread, you need to be synchronized with the Event lock. Try this before the line :

synchronized (Application.getEventLock()) {

[DD12]

Sorry about that. Yes that is the line I get error on.

I will try that.

Found this too.

Livelink - Redirection


Thanks.

[zechariahs]

To generalize a bit more, anytime you're updating the UI you want to make sure you have the event lock.

Like SamuelD mentioned, use the following code:

synchronized (Application.getEventLock()) 
{
   // Update UI here
}

[Ted Hopp]

Alternatively, you can use

Application.getApplication().invokeLater(new Runnable() {
  public void run() {
    // Update UI here
  }
});

(or use invokeAndWait() if necessary). The Runnable can also be kept in a variable and reused.