Activité qui plante

Version imprimable

Activité qui plante

Bonsoir,

J'ai un soucie avec une de mes activités qui plante lorsque je l'éxécute.

Après avoir testé la mise en commentaires de certaines variables, apparement le problème viendrait plus spécifiquement de ma seule Int.

voici le code :

Code:

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
import android.app.Activity;
 
public class PompeActivity extends Activity implements OnClickListener {
	TextView SeriePompe;
	TextView KcalPompe;
 
	EditText NbrPompe;
 
	Button ValiderPompe;
 
	int NombrePompe;
 
	/** Called when the activity is first created. */
	@Override
	public void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
 
		setContentView(R.layout.pompe);
 
			SeriePompe = (TextView) findViewById(R.id.seriepompe);
			 SeriePompe.setOnClickListener(this);
 
			NbrPompe = (EditText) findViewById(R.id.nbrpompe);
			NbrPompe.setOnClickListener(this);
 
			KcalPompe = (TextView) findViewById(R.id.kcalPompe);
 
			NombrePompe = Integer.parseInt(NbrPompe.getText().toString());
	}
 
	public void onClick(View v) {
 
		switch (v.getId()) {
 
		case R.id.validerPompe:
			NombrePompe = NombrePompe*2;
			KcalPompe.setText(NombrePompe);
 
		}
 
	};
 
}

C'est certainement une faute bête mais je viens de reprendre la programmation android et j'ai peut-etre zappé quelques notions.

Merci !

31/10/2011, 00h12
michel.di
Salut,
ça fait un moment que j'en ai pas fait aussi mais :
Code:

1 2 setContentView(R.layout.pompe);
ne doit pas se mettre à la fin?
31/10/2011, 00h44
nicroman
Salut !

Plusieurs remarques:
- Tu devrais utiliser les balises [code], ca rendra le code plus lisible.
- D'autre part, Java recommende de n'utiliser le "Full Camel Case" (premiere lettre majuscule des noms) que pour les types (classes, enum, interfaces). Les fonctions & variables doivent commencer par une minuscule.
- Pour finir, ne pas conserver de "View" en membres de la classe, pour une raison toute bete, c'est que chaque view conserve le contexte (et donc l'activité), si l'activité garde une trace de la view tu risques de tomber dans des comportement mémoires bizarres (même si le garbage collector de java est assez intelligent, c'est "moche" en terme de programmation ;) ). Rien ne t'empeche de faire des choses genre:
  Code:
  
  1 2 ( (TextView) findViewById(R.id.seriepompe)).setOnClickListener(this);
Sinon, que se passe-t-il quand que le contenu de R.id.nbrpompe est vide ? Le TextEdit est bien marqué en décimal ? Quelle est l'erreur affichée dans le LogCat ?
31/10/2011, 10h15
yan

Citation:

Envoyé par nicroman

Pour finir, ne pas conserver de "View" en membres de la classe, pour une raison toute bete, c'est que chaque view conserve le contexte (et donc l'activité), si l'activité garde une trace de la view tu risques de tomber dans des comportement mémoires bizarres (même si le garbage collector de java est assez intelligent, c'est "moche" en terme de programmation ;) ). Rien ne t'empeche de faire des choses genre:

:koi:
En quoi es ce moche? et quel type de comportement mémoire parle tu?
31/10/2011, 10h17
yan

Citation:

Envoyé par michel.di

ne doit pas se mettre à la fin?

Non. Car c'est cette ligne qui instancie toutes les view.
31/10/2011, 20h39
cp-08jhu
J'ai changé le nom de mes variables puis j'ai essayé d'appliquer ta méthode avec l'écouteur d'événement directement à la suite de l'initialisation via l'id :

Code:

seriePompe = ((TextView) findViewById(R.id.seriepompe)).setOnClickListener(this);

Mais le message " Cannot convert from Void to Textview " s'affiche.

J'avoue que je n'ai pas bien compris non plus le problème de comportement mémoire et je ne crois pas que le problème vienne de là.

Je précise que l'activité plante avant même de s'affiche, directement au lancement depuis une autre activité, cependant comme indiqué au 1er post, si je place les lignes suivante en commentaire l'activité se lance correctement:
Code:

1 2 3 4 5 6 7 //int nombrePompe; //nombrePompe = Integer.parseInt(nbrPompe.getText().toString()); /* case R.id.validerPompe: nombrePompe = nombrePompe*2; kcalPompe.setText(nombrePompe); */
02/11/2011, 13h47
Feanorin

Bonjour,

Peux tu nous montrer ton logcat pour voir quel message d'erreur apparaît ?

Car vu de nez, si tu n'as aucune valeur dans nbrPompe.GetText() lors de ton 'parse', tu aura une erreur.

Pour corriger le tir tu peux l'entourer d'un try catch en initialisant ta valeur dans le catch. l'exception a attrapé est NumberFormatException.

Merci
02/11/2011, 14h06
nicroman

Citation:

Envoyé par yan

:koi:
En quoi es ce moche? et quel type de comportement mémoire parle tu?

C'est moche dans le sens ou l'on rajoute un tas de réferences pas forcément nécessaires (en l'occurence, dans le code montré, seule la méthode onCreate les utilise).

C'est sur que ca pourrait être pire (avec une variable statique par exemple)....
http://developer.android.com/resourc...ory-leaks.html

Mais même sans parler de memory leak, conserver l'objet View dans l'activity veut dire qu'il ne pourra pas être marqué comme déréférencé (et donc susceptible d'etre collected par le GC) tant que l'activity n'est pas déréférencée elle-même, ce qui n'arrive qu'apres le onStop()...
02/11/2011, 14h52
yan

Citation:

Envoyé par nicroman

C'est moche dans le sens ou l'on rajoute un tas de réferences pas forcément nécessaires (en l'occurence, dans le code montré, seule la méthode onCreate les utilise).
C'est sur que ca pourrait être pire (avec une variable statique par exemple)....
http://developer.android.com/resourc...ory-leaks.html

Merci pour le lien.

Citation:

Envoyé par nicroman

Mais même sans parler de memory leak, conserver l'objet View dans l'activity veut dire qu'il ne pourra pas être marqué comme déréférencé (et donc susceptible d'etre collected par le GC) tant que l'activity n'est pas déréférencée elle-même, ce qui n'arrive qu'apres le onStop()...

Vue qu'avant il n'y as pas de raison qu'elles soient collectées (sauf volonté du développeur et la il peux mettre en null les référence), je ne voit pas le problème.

Citation:

Envoyé par Feanorin

Car vu de nez, si tu n'as aucune valeur dans nbrPompe.GetText() lors de ton 'parse', tu aura une erreur.

Le problème venait bien de là, aucune valeur n'étais encore attribuée.

Maintenant l'activité se lance correctement, je peux entrer une valeur dans mon EditText mais elle plante lorsque je presse le bouton Valider qui exécute le code suivant :
Code:

1 2 3 4 5 6 7 8 9 10 11 switch (v.getId()) { case R.id.validerPompe: nombrePompe = Integer.parseInt(nbrPompe.getText().toString()); nombrePompe = nombrePompe*10; //kcalPompe.setText(nombrePompe); }
Le problème vient plus spécifiquement de la partieen commentaire (kcalPompe.setText(nombrePompe)) car autrement l'activité ne plante pas mais évidement rien ne se passe.

Je souhaiterai simplement qu'une fois que l'utilisateur à pressé le bouton Valider, la valeur en int soit reprise de l'EditText, qu'un calcul soit fait puis qu'il soit afficher dans un TextView.

Je suis navré de faire durer se cet article en longueur mais une fois cette activité réglé je pourai passer aux nombreuses autres qu'il me reste à faire.

Merci de votre intêret.

04/11/2011, 10h13
Feanorin

Code:

kcalPompe.setText(nombrePompe);

Euh nombrePompe est un entier non ?

dans ce cas ton code devrait ressembler à ceci

Code:

kcalPompe.setText(Integer.toString(nombrePompe));
04/11/2011, 21h39
cp-08jhu

Citation:

Envoyé par Feanorin

Code:

kcalPompe.setText(nombrePompe);

dans ce cas ton code devrait ressembler à ceci

Code:

kcalPompe.setText(Integer.toString(nombrePompe));

Merci beaucoup le problème venait bien de là, je vais enfin pouvoir passer à la suite.

Bonne soirée et bon week-end à tous.