Android Activity не получава съобщения от IntentService с помощта на BroadcastReceiver

Опитвам се да разработя приложение за Android за измерване на QoS на мрежи. Приложението изпраща UDP пакети към UDP сървър, работещ на моята система. Тъй като това е дълъг процес, внедрих UDP връзката в клас, който разширява IntentService. Причината зад използването на IntentService е, че трябва да предам някои параметри на услугата. Имам BroadcastReceiver в моята дейност, който слуша за съобщения от IntentService и го отпечатва. Проблемът ми е, че въпреки че IntentService работи гладко, но дейността не получава съобщенията от него. Нов съм в разработката за Android, така че моля да ме извините за липсата на разбиране и всички насоки/предложения ще бъдат високо оценени. Публикувам някои части от моя код по-долу. Logcat не показва никакви грешки. Виждал съм да се използва метод intent.setAction() в някои примери, но не съм много наясно как да го използвам в моя случай.

BroadcastReceiver (дефиниран в моя клас Activity)

public class UdpResponseReceiver extends BroadcastReceiver {
 public static final String ACTION_RESP = "com.example.udpmessageclient.intent.action.MESSAGE_PROCESSED";
    @Override
    public void onReceive(Context context, Intent intent) {

        System.out.println(UdpService.PARAM_OUT_MSG);

    }

Регистрирал съм приемника:

IntentFilter filter = new IntentFilter(UdpResponseReceiver.ACTION_RESP);
filter.addCategory(Intent.CATEGORY_DEFAULT);
receiver = new UdpResponseReceiver();
registerReceiver(receiver, filter);

Клас IntentService:

public class UdpService extends IntentService {

//..variable declarations

public UdpService() {
    // TODO Auto-generated constructor stub
    super("UdpService");
}

@Override
protected void onHandleIntent(Intent intent) {
    // TODO Auto-generated method stub
    host = intent.getStringExtra("host");
    port = intent.getIntExtra("port", 4000);
    pType= intent.getIntExtra("pType", 0);
    delay = intent.getIntExtra("delay", 0);
    msg= intent.getStringExtra("msg");
    broadcastIntent = new Intent();
    broadcastIntent.addCategory(Intent.CATEGORY_DEFAULT);
    broadcastIntent.addFlags(Intent.FLAG_INCLUDE_STOPPED_PACKAGES); 

    try {
        addr = InetAddress.getByName(host);
        // addr=InetAddress.getLocalHost();
        socket = new DatagramSocket();
        // socket.connect(addr,port);

        System.out.println("\nSocket Connected");
    } catch (Exception e) {
        System.out.println("\nConnection failed");
        return;
    }

    send=true;
    switch (pType) {
    case 0:
        while (send) {
            sendPacket(msg);
        }
    case 1:
        while (send) {
            try {
                Thread.currentThread().sleep(delay);
            } catch (Exception e) {
            }
            sendPacket(msg);
        }
    case 2:
        while (send) {
            int u = want(30);
            String data1 = "";
            while ((u--) > 0)
                data1 = data1 + msg;
            sendPacket(data1);
        }
    case 3:
        while (send) {
            int u = want(30);
            System.out.println(u);
            String data1 = "";
            while ((u--) > 0)
                data1 = data1 + msg;
            System.out.println("data length :" + data1.length());
            try {
                Thread.currentThread().sleep(delay);
            } catch (Exception e) {
            }
            sendPacket(data1);
        }

    }
}
public void onDestroy(){
    super.onDestroy();
    send=false;
    socket.close();
    socket=null;

}
void sendPacket(String text) {

    try {
        System.out.println("\nClient:: Sending packet: " + " to " + addr
                + port);
        byte[] data = text.getBytes();
        spacket = new DatagramPacket(data, data.length, addr, port);
        socket.send(spacket);
        String resultTxt="Sent Packet at:"+DateFormat.format("MM/dd/yy h:mmaa", System.currentTimeMillis());

// this is where I am trying to send message back to the activity

    broadcastIntent.putExtra(PARAM_OUT_MSG, resultTxt);
        sendBroadcast(broadcastIntent);
    } catch (Exception e) {
        System.out.println("Error:" + e.getMessage());
        e.printStackTrace();
        return;
    }
}

}

съобщения за грешка в logcat, когато услугата е спряна:

01-14 15:53:41.446: W/System.err(1176): java.lang.NullPointerException
01-14 15:53:41.456: W/System.err(1176):     at com.example.udpmessageclient.UdpService.sendPacket(UdpService.java:123)
01-14 15:53:41.466: W/System.err(1176):     at com.example.udpmessageclient.UdpService.onHandleIntent(UdpService.java:74)
01-14 15:53:41.466: W/System.err(1176):     at android.app.IntentService$ServiceHandler.handleMessage(IntentService.java:65)
01-14 15:53:41.466: W/System.err(1176):     at android.os.Handler.dispatchMessage(Handler.java:99)
01-14 15:53:41.466: W/System.err(1176):     at android.os.Looper.loop(Looper.java:137)
01-14 15:53:41.476: W/System.err(1176):     at android.os.HandlerThread.run(HandlerThread.java:60)

person Poonam Anthony    schedule 14.01.2013    source източник
comment
Регистрирахте ли своя BroadcastReceiver?   -  person Royi    schedule 14.01.2013
comment
къде си регистрираш приемника? Ако не, направете го..... регистрирайте го в onCreate() и го дерегистрирайте в onDestroy().   -  person Pankaj Kumar    schedule 14.01.2013
comment
Регистрирал съм го в моя клас по дейности. Редактирах публикацията си и добавих кодовете. Моля, проверете   -  person Poonam Anthony    schedule 14.01.2013
comment
Вижте отговора ми.. ако има объркване, споделете   -  person Pankaj Kumar    schedule 14.01.2013
comment
проблемът продължава да съществува   -  person Poonam Anthony    schedule 14.01.2013
comment
Вижте актуализирания отговор и коментар   -  person Pankaj Kumar    schedule 14.01.2013


Отговори (1)


Променете кода на UdpService като ...

broadcastIntent = new Intent(UdpResponseReceiver.ACTION_RESP); // You forgot to add your custom intent filter
broadcastIntent.addCategory(Intent.CATEGORY_DEFAULT);
//broadcastIntent.addFlags(Intent.FLAG_INCLUDE_STOPPED_PACKAGES); // I don't think you really need it. So you can remove this flag. 

АКТУАЛИЗАЦИЯ

public static final String ACTION_RESP = "com.example.udpmessageclient.intent.action.MESSAGE_PROCESSED";

private final BroadcastReceiver UdpResponseReceiver = new BroadcastReceiver() {
        @Override
        public void onReceive(Context context, Intent intent) {

            //TODO handle results

        }
    }; 

И го регистрирайте като

registerReceiver(UdpResponseReceiver, new IntentFilter(ACTION_RESP))
person Pankaj Kumar    schedule 14.01.2013
comment
Благодаря Панкадж. Добавих този ред код. Но проблемът продължава. Операторът за печат в рамките на метода onReceive() на BroadcastReceiver не се изпълнява. - person Poonam Anthony; 14.01.2013
comment
Можете ли да проверите дали sendBroadcast(broadcastIntent) се извиква или не използва регистрационни файлове? И още един тестов опит е да го проверите без да използвате broadcastIntent.addCategory(Intent.CATEGORY_DEFAULT) и да извикате registerReceiver(UdpResponseReceiver, new IntentFilter(UdpResponseReceiver.ACTION_RESP)). Където декларацията на получателя добавих в отговора си. - person Pankaj Kumar; 14.01.2013
comment
Проблемът беше решен чрез замяна на System.out.println(UdpService.PARAM_OUT_MSG); с System.out.println(intent.getStringExtra(UdpService.PARAM_OUT_MSG)); Бях пренебрегнал тази грешка. Благодаря за отделеното време и помогнете на Pankaj - person Poonam Anthony; 14.01.2013
comment
Имам още едно запитване относно същия код. Да го публикувам ли тук или да започна нова тема? - person Poonam Anthony; 14.01.2013
comment
когато спра услугата, това предизвиква NullPointerExecption. Спирам услугата от дейността, като извикам stopService(udpServiceIntent); Това, което разбрах от logcat е, че програмата все още се опитва да изпълни socket.send(spacket); дори след като услугата е спряна и сокетът е затворен. Как може да се избегне това? Публикувам съобщенията на logcat в основния въпрос - person Poonam Anthony; 14.01.2013
comment
Сортирайте отговора, не извиквайте метода stopService. Направете променлива send като публична статична в UdpService. И когато искате да спрете услугата (логично искате да излезете от цикъла while, нали?), просто задайте тази send променлива на false от Activity. Това ще свърши работа. И да, още нещо, имам връзка, която може да обясни защо казвам да не се обаждам на stopService за IntentService. връзката е тук stackoverflow.com/questions/10362981/ - person Pankaj Kumar; 14.01.2013
comment
Съжалявам, не ми е много ясно защо предлагате да не извиквате stopService(). бихте ли разяснили, моля? - person Poonam Anthony; 14.01.2013
comment
Понякога IntentServices изчакват, докато задачата в onhandleintent() не приключи, въпреки че сте извикали stopService(). Ето защо казвам да не се обаждате и да използвате алтернативно решение. НО ЗАПОМНЕТЕ, че моят коментар е само за това изискване (както във вашия въпрос). - person Pankaj Kumar; 14.01.2013
comment
Благодаря много за вашите предложения. Но добавих допълнително условие if, за да проверя дали socket е null или не, преди да изпълня socket.send(), и то работи за мен. - person Poonam Anthony; 15.01.2013