Discussion :

[Phago]

Bonjour,

Je suis actuellement sur un programe dont l'UI devrait normalement s'actualiser a la réception d'un SMS.
Le problème c'est que j'aimerai pouvoir faire un truc "java" du style :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public class SMSReceiver extends BroadcastReceiver
{
	public SMSReceiver (MonObjet obj)
	{
		this.obj= obj;
	}
 
            public void onReceive(Context context, Intent intent){
            ............
            obj.methode(texteSms);
           }
}

Malheureusement je n'y arrive pas, et le BroadcastReceiver n'accepte qu'un constructeur sans paramètre.
Es-ce possible à faire, sinon comment pourrais-je feinter pour y arriver quand même ? (et dsl pour l'indentation pourrie, j'ai encore du mal avec les copier-coller de bout de code )

Merci d'avance

[nicroman]

Je ne vois pas de problème dans le code que tu as posté ????

Qu'est ce qui te bloque ?

[pitbull78]

Indentation sous Eclipse :

Ctrl+a Ctrl+i

Pour passer des paramètres, tu peux créer des objets statiques

[nicroman]

Ca y'est je crois que j'ai compris la question....

Je pense qu'il y a un problème de conception, parce que normalement tu n'as pas besoin de faire cela...

Si tu veux vraiment faire comme cela, voici un exemple d'implémentation:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 
class MyReceiver extends BroadcastReceiver
{
      public static interface Listener
      {
            public void doSomething(...);
      }
 
      static WeakReference<MyReceiver> _instance = null;
 
      public MyReceiver() {
          super();
          _instance = new WeakReference<MyReceiver>(this);
      }
 
      public static synchronized MyReceiver getInstance()
      {
         MyReceiver ret = (_instance == null) ? null : _instance.get();
         return ret;
      }
 
      HashSet<Listener>  listeners = new HashSet<Listener>();
 
      public static void register(Listener l)
      {
          MyReceiver rec = getInstance();
          if (rec != null) {
              syncrhonized (rec) {
                  rec.listeners.put(l);
              }
          }
      }
 
      public static void unregister(Listener l)
      {
          MyReceiver rec = getInstance();
          if (rec != null) {
              syncrhonized (rec) {
                  rec.listeners.remove(l);
              }
          }
      }
 
      public void onReceive(....)
      {
           for (Listener l : this.listeners) 
              l.doSomething();
      }
}

Mais bon, c'est encore une fois un peu tiré par les cheveux...

[Hephaistos007]

Ca y'est je crois que j'ai compris la question....

Je pense qu'il y a un problème de conception, parce que normalement tu n'as pas besoin de faire cela...

Si tu veux vraiment faire comme cela, voici un exemple d'implémentation:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 
class MyReceiver extends BroadcastReceiver
{
      public static interface Listener
      {
            public void doSomething(...);
      }
 
      static WeakReference<MyReceiver> _instance = null;
 
      public MyReceiver() {
          super();
          _instance = new WeakReference<MyReceiver>(this);
      }
 
      public static synchronized MyReceiver getInstance()
      {
         MyReceiver ret = (_instance == null) ? null : _instance.get();
         return ret;
      }
 
      HashSet<Listener>  listeners = new HashSet<Listener>();
 
      public static void register(Listener l)
      {
          MyReceiver rec = getInstance();
          if (rec != null) {
              syncrhonized (rec) {
                  rec.listeners.put(l);
              }
          }
      }
 
      public static void unregister(Listener l)
      {
          MyReceiver rec = getInstance();
          if (rec != null) {
              syncrhonized (rec) {
                  rec.listeners.remove(l);
              }
          }
      }
 
      public void onReceive(....)
      {
           for (Listener l : this.listeners) 
              l.doSomething();
      }
}

Mais bon, c'est encore une fois un peu tiré par les cheveux...

Ta solution impose d'enregistrer programmatiquement le receiver pour y coller les instructions supplémentaires, c'est bien cela ?

Code java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
MyReceiver joliRecepteur = MyReceiver.getInstance();
joliRecepteur.register(new MyReceiver.Listener() { public void doSomething() { /*code ici*/ } };);
context.registerReceiver(joliRecepteur, new IntentFilter(...));

[nicroman]

Oui...

Le plus compliqué est bien concevoir l'architecture des objets...
Si le receiver contient un objet, il est "maître" de cet objet (car conceptuellement, le receiver peut être le seul à tourner).

Donc, de deux choses l'une:
* Ou le receiver manage l'objet (qu'il peut donc utiliser comme il l'entend), voir le partager à d'autres (une fonction getObject() du receiver).
* Ou l'objet est indépendant et il y a deux solutions:
- Un seul objet possible (donc une fonction statique "set...." dans le receiver)
- Plusieurs objets possibles (donc une fonction statique "register..." dans le receiver)