Discussion :

[guigui14077]

Bonjour, je suis débutant en programmation java et android, et je réalise une application pour le bac. Actuellement je rencontre un petit problème lors du lancement d'une de mes activité, la méthode que j'ai créée semble mal agir sur mon application (voire pas du tout). Je poste le petit bout de code qui me pose, problème, pense que c'est une erreur de débutant .

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
 
public class NewactivityActivity extends Activity implements OnClickListener {	
 
 
      ImageView perso;
      private int nbrepif;
 
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_newactivity);
 
        ImageView perso=(ImageView)findViewById(R.id.imageView1);	
 
 
        valider.setOnClickListener(this);
        passer.setOnClickListener(this);
 
        nbrepif=(int)(Math.random()*78);
 
        rep();
 
 
    }
 
 public void rep(){
 
		    	switch(nbrepif)
		        {
		        case 1:
 
		        	perso.setImageResource(R.drawable.image1);
 
		        break;
 
..............
	        case 78:
 
		        	perso.setImageResource(R.drawable.image78);
 
		        break;
 
		        }
      }
}

Voilà j'ai allégé mon code au maximum pour être lisible. Je suis convaincu que le problème se trouve au niveau de ma méthode rep(), car sans elle l'activité se lance normalement.

Merci pour votre aide!

[nicroman]

Code :

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

ImageView perso=(ImageView)findViewById(R.id.imageView1);

Fait deux choses: déclarer une variable "perso" (qui va cacher la variable membre de l'activité) et lui assigner une valeur. Pendant ce temps là, la valeur de l'activité n'est pas modifiée => la fonction rep() va fonc envoyer un NullPointerException.

D'ailleurs, la description du problème aurait du commencer par là:
Mon application quitte "XXXX has stopped", avec dans les logs (logcat) une erreur (et une stacktrace) qui aurait montré le problème directement.

Eclipse propose tout un tas de warning là dessus qu'il est bon d'utiliser (forcer "this." pour les membres, mettre un warning quand une variable en cache une autre, ...).

Pour la fonction rep(), il existe plein d'autres moyen qu'un gros switch / case...
Par exemple:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
int ret;
try {
   String name = "image"+Integer.toString(nbrepif);
   ret = R.drawable.class.getField(name).getInt(null);
} catch (Excption ex) {
   Log.e(CLASS_TAG,"Erreur de récupération de l'image pour "+nbrepif,ex);
   ret = 0;
}
return ret;

Autre solution, sans passer par la reflection de Java:

Code :

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

1
2
3
4
5
6
7
 
private static int[] IMAGE_IDS = { R.drawable.image1, .... R.drawable.image78 };
 
public void rep()
{
    perso.setImageResource(IMAGE_IDS[nbrepif]);
}

[guigui14077]

Salut, merci de ta réponse rapide. Je ne comprend pas la structure que tu a utilisé dans ta première solution, je n'ai jamais vu cela, et je préfère comprendre ce que j'utilise plutôt que de le recopier bêtement. Pour ce qui qui est de la seconde j'avais déjà essayé cette structure mais il me semblait que la première image du tableau n'était pas systématiquement associée au nombre 1 (dit moi si je me trompe).

Sinon il n'y a pas possibilité de procéder de la même que sur mon code de départ?

Merci

[nicroman]

Non... les indices d'un tableau commencent à 0 en java.... 0 à (n-1)

Mais en même temps: "nbrepif=(int)(Math.random()*78);" va donner un nombre entre 0 (inclus) et 77 (inclus), donc ca colle....

Oui du coup, mon premier code ne marche pas... il faut faire Integer.toString(nbrepif+1);
Ton "switch" non plus au passage....


Pour expliquer le 1er code:
Qu'est ce que "R.drawable.imageXX" ? c'est un membre public (imageXX) de type "int" appartenant à la classe "R.drawable".
Java étant pleinement reflexif, on va demander à la classe une référence sur ce membre: "R.drawable.class.getField("imageXX")", et à cette référence sa valeur (getInt()). Comme c'est un membre statique, on ne passe pas "d'instance" de classe (donc getInt(null)).

[guigui14077]

C'est bon, ça marche nickel. J'ai utilisé le tableau de drawable et c'est vrai que si j'associe les bonnes variables aux bonnes images ça marche mieux .

Merci beaucoup, je passe en résolu.