1

Evening :)

I have a little problem...

As you can see in my code below, the Textview which i am trying to setText on, is getting me a Nullpointer :(

Is there anyone which can tell me what i am doing wrong ? :)

And the string s isnt null or, it is the view itself. It is working when setting the text in the oncreate().

Thanks..

public class smsShower extends Activity{

    private TextView status;
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        requestWindowFeature(Window.FEATURE_NO_TITLE);
        setContentView(R.layout.show);
        //status = (TextView)findViewById(R.id.status);

        status = (TextView)findViewById(R.id.status);
    }

    public void displayText(String s){

        status.setText(s);

    }

}

smsReceiverclass:

public class smsreceiver extends BroadcastReceiver {


    private String str;

    private boolean received = false;

    private smsShower smsshow;

    @Override
    public void onReceive(Context context, Intent intent) {
           //---get the SMS message passed in---
        Bundle bundle = intent.getExtras();       
        smsshow = new smsShower();

        SmsMessage[] msgs = null;
        str = "";            
        if (bundle != null)
        {

            Object[] pdus = (Object[]) bundle.get("pdus");
            msgs = new SmsMessage[pdus.length];  


            for (int i=0; i<msgs.length; i++){

                msgs[i] = SmsMessage.createFromPdu((byte[])pdus[i]); 

                if(msgs[i].getDisplayOriginatingAddress().equals("1990"))
                {
                    received = true;
                    str += msgs[i].getMessageBody().toString();

                    smsshow.displayText(str);



                }

Layout:

<?xml version="1.0" encoding="utf-8"?>
<RelativeLayout
  xmlns:android="http://schemas.android.com/apk/res/android"
  android:layout_width="fill_parent"
  android:layout_height="fill_parent">

<TextView android:id="@+id/status" android:layout_alignParentTop="true" android:text="Status:" android:layout_height="match_parent" android:layout_width="match_parent"></TextView>
</RelativeLayout>

Error Log:

02-18 01:14:20.064: ERROR/AndroidRuntime(18964): Caused by: java.lang.NullPointerException
02-18 01:14:20.064: ERROR/AndroidRuntime(18964):     at com.sms1990.smsShower.displayText(smsShower.java:36)
02-18 01:14:20.064: ERROR/AndroidRuntime(18964):     at com.sms1990.smsreceiver.onReceive(smsreceiver.java:47)
02-18 01:14:20.064: ERROR/AndroidRuntime(18964):     at android.app.ActivityThread.handleReceiver(ActivityThread.java:2941)
1
  • The s string is passed from a broadcastreceiver... And it is passed succesfully. I checked with some log.w's :)
    – Roskvist
    Feb 18, 2011 at 0:34

4 Answers 4

4

Where is displayText invoked? Is it possible it is being called before onCreate (or at least before onCreate has completed)? If it is you will get a NullPointerException because status hasn't been set yet.

2
  • Well... I have a class which sends and sms, and therefore a PendingIntent pi = PendingIntent.getActivity(this, 0, new Intent(sms1990.this, smsShower.class), 0); is created... The smsShower.class is the one where displayText is located... By making this the onCreate is used ? right ? ... And displayText is invoked from a broadcast reciever (handles the return sms). The returning sms is passed as the s parameter in displayText.
    – Roskvist
    Feb 18, 2011 at 0:37
  • @Roskvist Where does the BroadcastReceiver get its reference to the activity instance in order to invoke displayText?
    – Dan Dyer
    Feb 18, 2011 at 1:10
0

Add a basic null check as follows:

public void displayText(String s){
  if (status != null)
    status.setText(s);

}

you'll not get NPE. How do you start activity and get a reference to it in broadcast reciever?

0

I tried to check when onCreate is invoked...

public class smsShower extends Activity{

    private TextView status;
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        requestWindowFeature(Window.FEATURE_NO_TITLE);
        setContentView(R.layout.show);


        status = (TextView)findViewById(R.id.status);
        status.setText("NewTEXT!!!!!");
    }

i made a setText in onCreate, and when the application sends the sms:

public void sendStatusSMS(String fra,String til)
    {        
    String s = "DSB "+fra+" "+til;

    PendingIntent pi = PendingIntent.getActivity(this, 0,
            new Intent(sms1990.this, smsShower.class), 0);                
    SmsManager sms = SmsManager.getDefault();
    sms.sendTextMessage("1990", null, s, pi, null); 

The pendingIntent is responsible for starting the SmsShower and by doing so invoking the oncreate.

And therefore the smsreceiver is invoking displayText() after oncreate is called.. if i am understanding correctly..

But still that frigin nullpointer

0

As I see it the fourth parameter to the sms.sendTextMessage("1990", null, s, pi, null) function, the pi PendingIntent, will start your smsShower Activity after the message has been sent.

What I don't see is where the Intent that is caught in your broadcast receiver comes from? Can't you parse the "pdus" from the Intent directly in the smsShower onCreate() instead? (especially as the pi PendingIntent starts the Activity rather than triggers the BroadcastReceiver as it is stated in your code).

Also I have a bad feeling for the way you get a reference to your smsShower Activity in the onReceive() function. You create a new smsShower on the second line within the function but that is no guarantee that it's the same Activity that was previously started. It's actually more or less a guarantee that it's not the same Activity.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.