Discussion :

[xouzi]

Bonjour,

j'essaye de rafraichir mon activity (via un bouton) avec des nouveaux paramètres ( transfere et info).
Code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
OnClickListener rafraichir = new OnClickListener()
       {
		public void onClick(View actuelView)
		{	
			finish();
			Transfere transfere = new Transfere("6");
			Info info = new Info("1");
			startActivity(getIntent().putExtra("transfere", transfere).putExtra("info", info));
		}
        };
 
      //BOUTON 
        Button bouton = (Button) findViewById(R.id.image);
        bouton.setOnClickListener(rafraichir);

Je récupère bien les variables qui me permettent d’exécuter ma requête.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 String req = " SELECT * FROM tranfere t where tranfere.id_tranfere ="+tranfere+" and tranfere.id_info='"+info+"';";
  	  Log.e("log_tag", "req"+req);
  	    //on initialise le cursor
  	     FromBdd=null;
  	    //on executela requete est on alimente le cursor
  	    FromBdd = bdd.rawQuery(req,null);
 
 
  	    //On vérifie que la requete a bien renvoyer quelque chose
  	    if(FromBdd != null)
  	    {
  	    	 Log.e("log_tag", "FromBdd "+FromBdd.getCount());

Mais ma requête ne renvoi aucun champs ce qui ne devrait pas être le cas.
Je voudrais savoir si c'est la bonne méthode pour actualiser et pourquoi ma requête renvoi 0 ligne.

[nicroman]

Ben je sais pas ... tu as regardé le contenu de la commande SQL ?

Plein de questions:

Pourquoi ne pas simplement stocker les paramètres "Transfere" et Info, localement à l'activité, et juste raffraichir l'interface de celle-ci (là une nouvelle activité sera générée à chaque fois, beaucoup de code inutile donc).
Par exemple:

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
 
class XXXActivity extends Activity
{
     Transfere trans;
     Info  info;
 
     public void onCreate(Bundle bundle)
     {
         super.onCreate(bundle);
         setContentView(....);
 
         if (bundle == null) {
             // get trans/info from Intent
             Intent i = getIntent();
             if (i == null) {
                 // default trans/info values
             } else {
                this.trans = i.getExtra(...);
                this.info = i.getExtra(...);
             }
         } else {
             // get trans/info from bundle
         }
 
         Button bouton = (Button) findViewById(R.id.image);
         bouton.setOnClickListener(new View.OnClickListener() {
             public void onClick(View actuelView)
             {	
	          XXXActivity.this.trans  = new Transfere("6");
		  XXXActivity.this.info = new Info("1");
		  XXXActivity.this.refreshUI();
	     }
         });
 
         refreshUI();
     }
 
     protected void onSaveInstanceState(Bundle bundle) {
          super.onSaveInstanceState(bundle);
          // save this.trans & this.info in bundle
     }
 
     public void refreshUI()
     {
         // use of this.trans & this.info
     }
}

Pourquoi ne pas utiliser les paramètres (au lieu de construire la commande SQL à la main avec paramètres intégrés, au risque d'oublier ou de mal 'escaper' ces paramètres) ?
Là en plus, je ne sais pas ce que fait transfere.toString() et info.toString(), mais c'est ce qui sera utilisé pour construire la chaine....

Code :

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

1
2
3
4
5
 
public static final String REQUEST_PARAMS = "SELECT * FROM tranfere t where tranfere.id_tranfere=? and tranfere.id_info=?"; // un point virgule de trop ?
 
//on initialise le cursor
Cursor fromBdd = bdd.rawQuery(REQUEST_PARAMS,new String[] { transfere.toString(), info.toString() });

Pourquoi utiliser Log.e pour logger des choses qui ne sont manifestement pas des erreurs ?

[xouzi]

juste raffraichir l'interface

Je ne pense pas que je peux que rafraichir car par la suite je vais avoir d’autre bouton ou la valeur des variables ne seront pas les même et par exemple : info=2 et transfere = 7.

Pourquoi ne pas utiliser les paramètre

Je ne connaissais pas cette méthode qui est plus intéressante que la mienne.

Code :

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

Pourquoi utiliser Log.e

Parce que j'aime le rouge.

Je n'ai pas compris la methode refreshUI();
Je suppose qu'elle donne l'action de rafraichir mais comment je suppose que ce n'est pas un intent ?

[nicroman]

Imagine l'activité comme un "objet"....

L'action de raffraichir les données affichée (refreshUI()) est appelée à chaque fois que les filtres de base sont modifiés... Ces filtres étant "Info" et "Transfere"...

Quand modifie-t-on ces filtres ?
* A l'appui sur un des boutons => mise à jour des filtres, puis appel de refreshUI()
* Au démarrage de l'activité
.... * Une nouvelle activité (pas de 'saved-instance') => lecture des filtres depuis l'intent
.... * Parceque l'activité a été reveillée par Android (rotation écran par exemple) => lecture des filtres depuis le bundle [qu'il ne faut pas oublier de sauvegarder avant]
.... Dans tous les cas refreshUI()

La fonction refreshUI() se chargera de modifier l'interface pour correspondre aux filtres.

Ensuite chaque bouton peut avoir ses propres valeurs... Cela n'a rien à voir avec les "filtres courant" de l'activité.

[nicroman]

Parce que j'aime le rouge.

C'est une bonne raison, mais qui risque de faire disparaitre le "vrai" rouge dans le reste... d'autant qu'il est facile de filtrer le logcat par "tag" (ils servent à ca justement).