Discussion :

[x031183]

bonjour,

J'ai un listener nommé RetourPageListener lié à un include footer (button valider) de ce type :

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
package com.dinasoft.TrouveVelo.listener;
 
import android.app.Activity;
import android.view.View;
import android.view.View.OnClickListener;
 
 
/**
 * @author didier
 *
 */
public class RetourPageListener implements OnClickListener{
 
	private Activity mContext;
 
	public RetourPageListener(Activity context){
		this.mContext=context;
	}
 
	@Override
	public void onClick(View v) {
		// TODO Auto-generated method stub
 
		mContext.finish();
	}
 
}

dans mon main j'ai l'appel au listener :

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
public void onCreate(Bundle savedInstanceState) {

		super.onCreate(savedInstanceState);
		setContentView(R.layout.creersavoiture10);

		RetourPage = (Button) findViewById(R.id.RetourPreced);
		LongVehicule = (TextView) findViewById(R.id.taille_vehicule);
		
		boutonType= (Button) findViewById(R.id.creervalider);
		ParamBd = new RequeteSeGarer(this);
		
		// On ouvre la base de données pour écrire dedans
		ParamBd.open();
	

		// listener quand j'appuie sur le bouton type de véhicule
		boutonType.setOnClickListener(new typeVoiture1Listener(this));
		
	
		RetourPage.setOnClickListener(new RetourPageListener(this));		
		
	}

Normalement en lui passant THIS le Finish devrait appeler le Destroy de l'activité et bien non. Il fait effectivement un Finish mais pas celui de l'activité.

Je ne comprend pas pourquoi, surtout que la méthode destroy ferme la base de données est passe des valeurs à une autre activité.

merci de vos retours

cordialement

[Feanorin]

Bonjour,

La méthode finish appartient à la classe Activiy et non à la classe Context.

Donc ta méthode ne peux pas marcher.

Tu peux soit passer l'activity à ta classe RetourPageListener en tant que menbre pour pouvoir la fermer.

Soit passer par un Handle qui enverra un message à ton Activity ou une autre méthode de message.

à toi de voir.

[nicroman]

D'apres le code, il passe bien une activité....

On peut voir le code du onDestroy ? (parceque si j'ai bien compris il est overridé)

[Feanorin]

D'apres le code, il passe bien une activité....

Yep

Code :

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

private Activity mContext;

Je n'avais pas bien vu, ça pique les yeux quand même comme déclaration .

[x031183]

voici la méthode destroy(). Comme vous le verrez j'opte pour un singleton plutôt q'un passage par l'intent en parcelable.

merci de l'aide.

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
@Override
	public void onDestroy() {
		super.onDestroy();
 
 
		Intent intent = new Intent();
		if (VehiculeRetenu != null)
		{
 
			// Création et ouverture de la base de données
			long j=ParamBd.Update_mon_vehiculeDefault0();
			long i= ParamBd.Insert_mon_vehicule(VehiculeRetenu);
			PositionCourante.getInstance().setIdVoiture(VehiculeRetenu.getId());
			PositionCourante.getInstance().setCouleur(VehiculeRetenu.getCouleur());
			PositionCourante.getInstance().setNumImat(VehiculeRetenu.getNnumero_mmatriculation());
			ParamBd.close();
			intent.putExtra("IMMAT", VehiculeRetenu.getNnumero_mmatriculation());
			setResult(RESULT_OK,intent);
		}
		else 
		{
			ParamBd.close();
			intent.putExtra("IMMAT", (String)null);
			setResult(RESULT_CANCELED, intent);
			//fermeture de la base de données 
		}
	}

[x031183]

en me mettant sur DDMS, je me suis apperçu d'une chose surprenante,

quand j'appelle le finish dans mon RetourPageListener il va directement à l'activité précédente et appelle la méthode onActivityResult donc mon Resultcode est à 0 et donc l'égalité if (resultCode == RESULT_OK) ne passe pas et j'ai une erreur (Null pointer).

Mais pendant ce temps du passage à l'activité précédente, il execute la méthode destroy() qui elle passe des valeurs par intent.

Code :

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

1
2
intent.putExtra("IMMAT", VehiculeRetenu.getNnumero_mmatriculation());
setResult(RESULT_OK,intent);

Mais il est déjà trop tard.

j'ai donc un problème d'attendre que destroy effectue son traitement avant qu'il passe à l'activité précédente.

[nicroman]

Oui mais non....

Une activité ce n'est pas une sous-fonction java... Android peut très bien décider d’exécuter plusieurs thread pour plusieurs activité...
Dans le onDestroy, c'est fini, l'activité n'existe plus... donc celle qui l'a appelé a déjà reçu son résultat...

En même temps... pourquoi ne pas surcharger la méthode "finish()" ? (plutot que onDestroy)...

Ou simplement créer ta propre méthode qui va faire le setResult() et appeler le finish.

[x031183]

effectivement j'ai surchargé Finish et cela marche mieux

merci beaucoup

cordialement