Discussion :

[guigui2454000]

Bonjour, je suis actuellement en train d'essayer de créer une application, et j'ai un soucis avec une activité. En effet, celle-ci doit permettre à l'utilisateur de choisir un produit qui dysfonctionne dans un spinner, ensuite il décrit succinctement le problème dans un Edittext, enfin il clique sur envoyer et là une boite de dialogue apparaît pour dire que le message a bien été envoyé.

Mon problème c'est que depuis que j'ai inséré les lignes de code pour la boite de dialogue, ma liste déroulante dans mon spinner ne fonctionne plus, et je vois pas pourquoi. Enfin je pense je pense que c'est du à la façon dont est organisé mon code dans l'activité.java, et du coup Eclipse pense qu'il n'y a qu'une seule action à réaliser.

Voilà le code de l'activité concernée:

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
package com.blabla.blablaapp;
 
 
 
import android.app.Activity;
import android.app.AlertDialog;
import android.content.DialogInterface;
import android.os.Bundle;
import android.widget.AdapterView;
import android.widget.AdapterView.OnItemSelectedListener;
import android.widget.ArrayAdapter;
import android.widget.Button;
import android.widget.Spinner;
import android.view.View;
import android.view.View.OnClickListener;
 
public class ServiceMeldungActivity extends Activity implements OnClickListener{
 
	Spinner spin;
    String spin_val;
    String[] equipment = { "Equipment1", "Equipment2", "Equipment3", "Equipment4" };
    Button button1;
 
	@Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_service_meldung);
 
button1=(Button)findViewById(R.id.button1);
 
        button1.setOnClickListener(this);
}      
 
    public void onClick(View arg0) {
        switch(arg0.getId()){
          case R.id.button1:
                AlertDialog.Builder boite;
                boite = new AlertDialog.Builder(this);
                boite.setTitle("Sendebericht");
                boite.setIcon(R.drawable.ic_launcher);
                    boite.setMessage("Die Nachricht wurde gesendet");
                boite.setPositiveButton("Ok", new DialogInterface.OnClickListener() {
 
                    public void onClick(DialogInterface dialog, int which) {
 
                    }
                    }
                );
                boite.show();
                     break;
        }
 
        spin = (Spinner) findViewById(R.id.spinner2);
        spin.setOnItemSelectedListener(new OnItemSelectedListener() {
 
        	@Override
            public void onItemSelected(AdapterView<?> arg0, View arg1,
                    int position, long id) {                
                spin_val = equipment[position];
 
 
            }
 
            @Override
            public void onNothingSelected(AdapterView<?> arg0) {
 
            }
 
        });
        ArrayAdapter<String> spin_adapter = new ArrayAdapter<String>(ServiceMeldungActivity.this, android.R.layout.simple_spinner_item, equipment);
        spin.setAdapter(spin_adapter);
    }
}

Pourriez-vous m'expliquer l'erreur s'il vous plait et/ou comment on organise, dans le code en java, les différentes actions à réaliser?

[grunk]

Ton code lié à ton spinner est mal placé. Actuellement il est dans le onclick et non conditionné , ce qui veux dire qu'à chaque élément de ton interface qui va générer un onClick le code du spinner va être appelé.

Déplace toute cette partie :

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
spin = (Spinner) findViewById(R.id.spinner2);
spin.setOnItemSelectedListener(new OnItemSelectedListener() {
 
	@Override
	public void onItemSelected(AdapterView<?> arg0, View arg1,
			int position, long id) {                
		spin_val = equipment[position];
 
 
	}
 
	@Override
	public void onNothingSelected(AdapterView<?> arg0) {
 
	}
 
});
ArrayAdapter<String> spin_adapter = new ArrayAdapter<String>(ServiceMeldungActivity.this, android.R.layout.simple_spinner_item, equipment);
spin.setAdapter(spin_adapter);

dans le onCreate.
Je suis pas certains qu'un alertDialog soit le meilleur choix pour notifier l'utilisateur. Si aucune action de sa part n'est requise, un simple toast est suffisant à mon avis.

[guigui2454000]

J'ai déplacé le bout de code en question dans le OnCreate, de cette manière:

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
package com.esiemensbt.siemensbtserviceapp;
 
 
 
import android.app.Activity;
import android.app.AlertDialog;
import android.content.DialogInterface;
import android.os.Bundle;
import android.widget.AdapterView;
import android.widget.AdapterView.OnItemSelectedListener;
import android.widget.ArrayAdapter;
import android.widget.Button;
import android.widget.Spinner;
import android.view.View;
import android.view.View.OnClickListener;
 
public class ServiceMeldungActivity extends Activity implements OnClickListener{
 
	Spinner spin;
    String spin_val;
    String[] equipment = { "Equipment1", "Equipment2", "Equipment3", "Equipment4" };
    Button button1;
 
	@Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_service_meldung);
        spin = (Spinner) findViewById(R.id.spinner2);
        spin.setOnItemSelectedListener(new OnItemSelectedListener() {
 
            @Override
            public void onItemSelected(AdapterView<?> arg0, View arg1,
                    int position, long id) { 
                spin_val = equipment[position];
 
 
            }
 
            public void onNothingSelected(AdapterView<?> arg0) {
 
            }
 
        });
        ArrayAdapter<String> spin_adapter = new ArrayAdapter<String>(ServiceMeldungActivity.this, android.R.layout.simple_spinner_item, equipment);
        spin.setAdapter(spin_adapter);
	}        
 
    public void onClick(View arg0) {
        switch(arg0.getId()){
          case R.id.button1:
                AlertDialog.Builder boite;
                boite = new AlertDialog.Builder(this);
                boite.setTitle("Sendebericht");
                boite.setIcon(R.drawable.ic_launcher);
                boite.setMessage("Die Nachricht wurde gesendet");
                boite.setPositiveButton("Ok", new DialogInterface.OnClickListener() {
 
                    public void onClick(DialogInterface dialog, int which) {
 
                    }
                    }
                );
 
                boite.show();
                     break;
        }
 
    }
   }

Désormais j'ai bien mon spinner qui fonctionne mais il ne se passe plus rien quand je clique sur "envoyer"...
Et concernant la boite de dialogue, j'utilise cela parce que mon maître de stage m'a dit d'utiliser "des messagebox" pour simuler des réponses du serveur et que ça s'en rapprochait mais c'est vrai qu'un toast fait l'affaire.

[grunk]

A aucun moment tu ne défini de listener pour ton button1 , dans ton cas dans le onCreate rajoute :

Code :

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

button1.setOnClickListener(this);

[guigui2454000]

Ca ne fonctionne toujours pas. Mais est-ce que c'est important de mettre

Code :

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

button1.setOnClickListener(this);

?
Car au début quand mon spinner ne fonctionnait pas, le bouton lui marchait très bien, j'avais bien la boite de dialogue avec le bouton ok qui s'ouvrait.

Est-ce que ça ne serait pas dû au fait que le morceau de code concernant le bouton, ne se trouve pas dans le OnCreate?

[grunk]

Le listener était défini dans le premier code que tu as donnés , puis ne l'était plus dans le second.

Fonctionne par étape , fait fonctionner ton bouton , puis rajoute le spinner (ou inversement).

[guigui2454000]

Oui c'est bon ça marche! Merci beaucoup