Уведомление проходит старые Intent Extras

134

я создаю уведомление внутри BroadcastReceiver с помощью этого кода:

String ns = Context.NOTIFICATION_SERVICE;
        NotificationManager mNotificationManager = (NotificationManager) context.getSystemService(ns);
        int icon = R.drawable.ic_stat_notification;
        CharSequence tickerText = "New Notification";
        long when = System.currentTimeMillis();

        Notification notification = new Notification(icon, tickerText, when);
        notification.defaults |= Notification.DEFAULT_VIBRATE;
        long[] vibrate = {0,100,200,200,200,200};
        notification.vibrate = vibrate;
        notification.flags |= Notification.FLAG_AUTO_CANCEL;

        CharSequence contentTitle = "Title";
        CharSequence contentText = "Text";
        Intent notificationIntent = new Intent(context, NotificationActivity.class);
        notificationIntent.putExtra(Global.INTENT_EXTRA_FOO_ID, foo_id);
PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, 0);

        notification.setLatestEventInfo(context, contentTitle, contentText, contentIntent);

        int mynotification_id = 1;

        mNotificationManager.notify(mynotification_id, notification);

Когда я нажимаю на уведомление, оно открывает NotificationActivity, и внутри Activity я могу получить foo_id из Intent-Bundle (например, 1)

Однако, если запускается другое уведомление, и я снова нажимаю на него, действие по-прежнему получает «старое» значение (1) из Intent-Bundle. Я пытался очистить пакет с помощью clear (), но получаю тот же эффект. Я думаю, что что-то не так с моим кодом ..

BrianM
источник
подскажите пожалуйста, как вы получаете данные от ожидающего намерения
user49557
Осознание того, что оно отправляло старые статисты, облегчило мою сортировку.
Утсав Гупта

Ответы:

268

Вы отправляете тот же код запроса для вашего ожидающего интенсив. Изменить это:

PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, 0);

Для того, чтобы:

PendingIntent contentIntent = PendingIntent.getActivity(context, UNIQUE_INT_PER_CALL, notificationIntent, 0);

намерения не создаются, если вы отправляете те же параметры. Они используются повторно.

IncrediApp
источник
1
так что UNIQUE_INT_PER_CALL - это целое число, которое я должен предоставить? или это статическая переменная где-то объявлена?
BrianM
23
android gotcha # 147 - так Intentчто объект с разными дополнениями (через putExtra) считается одним и тем же и используется повторно, потому что я не предоставил уникальный идентификатор некоторому ожидающему вызову намерения - ужасный api
wal
Вы знаете, что я был так небрежен. Просто думаю, как это могло остаться 0 в одном блоке (в моем случае) :(
Exigente05
3
Это было невероятно полезно для меня, просто подсказка для других, вполне вероятно, что вы создаете свое уведомление тем же методом, и поэтому вы можете просто установить идентификатор для нового ожидающего намерения таким же, как тот, который вы собираетесь использовать для уведомлений уникальный идентификатор!
Джеймс Макни
1
@IncrediApp, это то же самое с PendingIntent.getBroadcast (); ?
Шрути
139

В качестве альтернативы вы можете использовать следующий код для создания своего PendingIntent:

PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, PendingIntent.FLAG_UPDATE_CURRENT);

Из документа для PendingIntent.FLAG_UPDATE_CURRENT:

Если описанный PendingIntent уже существует, оставьте его, но замените его дополнительные данные тем, что есть в этом новом намерении. Это можно использовать, если вы создаете намерения, в которых изменяются только дополнения, и вам не важно, что какие-либо сущности, получившие ваше предыдущее PendingIntent, смогут запустить его с вашими новыми дополнениями, даже если они явно не были ему переданы.

ChristophK
источник
Спасибо ... отлично работает для этого флага, который добавляет "PendingIntent.FLAG_UPDATE_CURRENT" :)
Наджиб Ахмед Путавала
1
Работал для меня, используя ожидающее намерение передать состояние от установки будильника к вещательному приемнику.
Уильям Т. Маллард
Мне просто жаль, что я не знал, что на самом деле делают эти флаги, прежде чем я отправлю уведомления своим пользователям (!) Рад, что это решает мои проблемы ...
Джеймс Эндрю
42

Вы передаете тот же идентификатор. В такой ситуации создайте уникальный идентификатор из времени:

int iUniqueId = (int) (System.currentTimeMillis() & 0xfffffff);

И сформулируйте это так:

PendingIntent contentIntent = PendingIntent.getActivity(getApplicationContext(),iUniqueId, intentForNotification, 0);
hderanga
источник
3
почему бы не использовать new Random (). nextInt ()
exloong
@hderanga, что делает добавление "& 0xfffffff" к int выше?
AJW
3
@AJW System.currentTimeMillis()возвращает long, а requestIdпараметр PendingIntent.getActivity()принимает int. 0xffffffffэто битовая маска Хотя есть еще кое-что, простое объяснение состоит в том, что выполнение `long & 0xffffffff 'дает самые низкие 32-битные из длинных и отбрасывает самые высокие 32-битные, оставляя вас по существу 32-битным int. Это лучше, чем простое приведение к int, потому что это не приведет к искажению знакового бита (если вы приведете длинное значение, превышающее int, к int, знаковый бит будет переполнен, и вы потенциально получите отрицательное значение )
Jordan Bondo
8

Если вы ищете лучший подход после долгого времени, вам нужно передать PendingIntent.FLAG_UPDATE_CURRENT в качестве последнего аргумента, как показано ниже

PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, PendingIntent.FLAG_UPDATE_CURRENT);

вам даже не нужно предоставлять новый уникальный идентификатор.

Вы должны сделать это в следующий раз, а не в первый раз

пологий
источник
1
Это не работает, я попал сюда, потому что это то, что я делал.
Brill Pappin
Вы должны сделать это в следующий раз, а не в первый раз, это будет работать.
Нежный
0

Код вашего запроса - 0 для всех уведомлений. Измените следующую строку:

PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, 0);

С участием:

PendingIntent contentIntent = PendingIntent.getActivity(context, new Random().nextInt(), notificationIntent, 0);
Фейсал Шейх
источник
1
Есть ли преимущества в использовании «new Random (). NextInt ()» вместо «System.currentTimeMillis ()»?
AJW
использование random может легко восстановить одно и то же целочисленное значение снова в случае аварии, что приведет к очень трудному обнаружению ошибки при передаче старых намерений.
Сэм
@AJW было в моем случае. Я создал 2 разных уведомления за одну и ту же миллисекунду, поэтому одно из них получило неправильные дополнения.
artman
0

Просто хотел добавить еще один вариант

 PendingIntent.getBroadcast(context, 0, intent, PendingIntent.FLAG_ONE_SHOT | PendingIntent.FLAG_IMMUTABLE);
pellucide
источник