Besoin d'aide pour mon application

**wawator** · 14/05/2014, 08h56

Bonjour à tous, alors je suis en train de coder une application avec un nombre de "if".
J'ai pas mal d'images et je veux qui si l'image apparaît alors faire ...
Voici mon code actuel :

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
50
51
52
public class MainActivity extends ActionBarActivity {
 
 
 private final static Random random = new Random();
 private Button next;
 private ImageView cardImage,cardImage2;
 int currentCardIndex = 0;    
 
 
 int[] cards = {
     R.drawable.h2,  
     R.drawable.h3,  
     R.drawable.h4,  
     R.drawable.h5,  
     R.drawable.h6,
     R.drawable.h7,
     R.drawable.h8,
     R.drawable.h9,
     R.drawable.h10,
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    LinearLayout root = (LinearLayout) findViewById(R.id.clickscreen);
    shuffle(cards);
    next  = (Button)findViewById(R.id.button1);
    cardImage   = (ImageView)findViewById(R.id.cards);
    final Animation fadeInAnimation = AnimationUtils.loadAnimation(this, R.anim.abc_slide_in_top);
    new Dialog(getApplicationContext());
    final MediaPlayer mp = MediaPlayer.create(this, R.raw.siren);
    final MediaPlayer card = MediaPlayer.create(this, R.raw.cardflip);
 
    root.setOnClickListener(new OnClickListener() {
 
        public void onClick(View v) {
 
            cardImage.startAnimation(fadeInAnimation);
            cardImage.setImageResource(cards[currentCardIndex++]);
            card.start();
 
 
             if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.ta).getConstantState())){
                   AlertUtil.messageAlert(MainActivity.this,getString(R.string.cul_sec) , getString(R.string.cul_sec_regle));
 
                  }  
             if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.sa).getConstantState())){
                   AlertUtil.messageAlert(MainActivity.this,getString(R.string.cul_sec), getString(R.string.cul_sec_regle));
 
             }
            if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.ha).getConstantState())){
                   AlertUtil.messageAlert(MainActivity.this,getString(R.string.cul_sec), getString(R.string.cul_sec_regle));
                  }

Alors je voulais savoir s'il y a une manière plus simple et qui ferait que ça ne lag pas ?
Dans le code au-dessus je n'ai pas mis tout mes IF sinon ça ferait trop long.
Merci d'avance !

**Oussapik** · 14/05/2014, 09h26

Bonjour,

Etant donné que le cœur du if semble toujours le même, je te conseillerais de rentrer ces éléments dans deux HashMap une et une seule fois au lancement (en gros, dans le onCreate, mais pas dans le onClick).
Ensuite, l'intégralité de tes ifs serait remplacé par :

Code :

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

1
2
 
AlertUtil.messageAlert(MainActivity.this,hashmapString1.get(cardImage.getDrawable().getConstantState()) , hashmapString2.get(cardImage.getDrawable().getConstantState()));

Un exemple pour l'initialisation avec le premier if :

Code :

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

1
2
3
4
5
 
Map<ConstantState, String> hashmapString1 = new HashMap<ConstantState,String>();
Map<ConstantState, String> hashmapString2 = new HashMap<ConstantState,String>();
hashmapString1.put(getResources().getDrawable(R.drawable.ta).getConstantState(), getString(R.string.cul_sec));
hashmapString2.put(getResources().getDrawable(R.drawable.ta).getConstantState(), getString(R.string.cul_sec_regle));

Bien sur, je te conseille aussi de trouver des meilleurs noms pour ces hashmap qui reflètent ce que tu veux mettre dedans.

**wawator** · 14/05/2014, 10h28

Merci ça fonctionne,
c'est long a tapé mais bon...
sinon est il possible de comparer la première lettre des drawable ?

Code :

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

1
2
hashmapString1.put(getResources().getDrawable(R.drawable.Xa).getConstantState(), getString(R.string.cul_sec));
        hashmapString2.put(getResources().getDrawable(R.drawable.Xa).getConstantState(), getString(R.string.cul_sec_regle));

Par exemple tous ce qui ce termine par la lettre a alors

Merci pour ta réponse

**wawator** · 14/05/2014, 11h55

Je ne suis pas sur de bien comprendre
ce que tu m'as dit fonctionne pour le premier if après je fait comment ?

Comme ça ?

Code :

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

1
2
3
4
5
6
7
8
final Map<ConstantState, String> hashmapString1 = new HashMap<ConstantState,String>();
        final Map<ConstantState, String> hashmapString2 = new HashMap<ConstantState,String>();
        hashmapString1.put(getResources().getDrawable(R.drawable.ta).getConstantState(), getString(R.string.cul_sec));
        hashmapString2.put(getResources().getDrawable(R.drawable.ta).getConstantState(), getString(R.string.cul_sec_regle));
        final Map<ConstantState, String> hashmapString3 = new HashMap<ConstantState,String>();
        final Map<ConstantState, String> hashmapString4 = new HashMap<ConstantState,String>();
        hashmapString3.put(getResources().getDrawable(R.drawable.sa).getConstantState(), getString(R.string.cul_sec));
        hashmapString4.put(getResources().getDrawable(R.drawable.sa).getConstantState(), getString(R.string.cul_sec_regle));

Et dans Onclick :

Code :

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

1
2
3
4
5
6
7
			public void onClick(View v) {
 
				cardImage.startAnimation(fadeInAnimation);
				cardImage.setImageResource(cards[currentCardIndex++]);
				card.start();
				AlertUtil.messageAlert(MainActivity.this,hashmapString1.get(cardImage.getDrawable().getConstantState()) , hashmapString2.get(cardImage.getDrawable().getConstantState()));
				AlertUtil.messageAlert(MainActivity.this,hashmapString3.get(cardImage.getDrawable().getConstantState()) , hashmapString4.get(cardImage.getDrawable().getConstantState()));

Je ne suis pas sur de bien faire
Merci d'avance

**Oussapik** · 14/05/2014, 13h39

Tu ne dois déclarer que 2 maps en tout.
Par exemple, pour ton second item :

Code :

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

1
2
3
 
hashmapString1.put(getResources().getDrawable(R.drawable.sa).getConstantState(), getString(R.string.cul_sec));
hashmapString2.put(getResources().getDrawable(R.drawable.sa).getConstantState(), getString(R.string.cul_sec_regle));

En gros, dans la première hashmap, tu entreras tous les éléments qui doivent être passés en second argument de ton "messageAlert" et dans la seconde ceux du troisième argument.

Edit : et il ne doit y avoir du coup qu'un seul appel à "messageAlert" en replacement de tous les ifs.

**wawator** · 14/05/2014, 14h13

Ok merci je vais essayer ça

Merci pour vous réponse !

**wawator** · 14/05/2014, 14h15

Ah alors ça fonctionne niquel,
Mais j'avais certaine condition qui ouvrer une autre activity ou lancer de la musique comment faire du coup?

C’était comme ça :

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
 if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.d9).getConstantState())){
					 startActivity(new Intent(v.getContext(),Combat_Des.class));
	                  }
				 if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.t9).getConstantState())){
					 startActivity(new Intent(v.getContext(),Combat_Des.class));
	                  }
				 if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.s9).getConstantState())){
					 startActivity(new Intent(v.getContext(),Combat_Des.class));
	                  }
				 if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.h9).getConstantState())){
					 startActivity(new Intent(v.getContext(),Combat_Des.class));     
				 }
 
				 if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.dj).getConstantState())){
	           		   AlertUtil.messageAlert(MainActivity.this,getString(R.string.bizkit), getString(R.string.bizkit_regle));
	           		mp.start();
				 }
				 if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.tj).getConstantState())){
	           		   AlertUtil.messageAlert(MainActivity.this,getString(R.string.bizkit), getString(R.string.bizkit_regle));
	           		mp.start();
				 }
				 if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.sj).getConstantState())){
	           		   AlertUtil.messageAlert(MainActivity.this,getString(R.string.bizkit), getString(R.string.bizkit_regle));
	           		mp.start();
				 }
				 if (cardImage.getDrawable().getConstantState().equals(getResources().getDrawable(R.drawable.hj).getConstantState())){
	           		   AlertUtil.messageAlert(MainActivity.this,getString(R.string.bizkit), getString(R.string.bizkit_regle));      
	           		mp.start();
				 }

Merci a vous

**wawator** · 14/05/2014, 14h32

En faite je laisse comme ça.

Merci beaucoup a vous c'est beaucoup mieux !

Merci

**wawator** · 16/05/2014, 10h30

J'ai réussi à fini mon application merci.
Mais petit souci j'ai remarquer quelle ne ce lance pas sur tous les appareils, avez vous une idée du pourquoi ?

Voici le lien Playstore : https://play.google.com/store/apps/d....drinkingcards

Besoin d'aide pour mon application

Android

Discussions similaires

Partager

Partager