Intent put Extra übergibt falsche Daten

Hallo zusammen,

ich sitze gerade an einem Formular (Test). Ein User trägt in zwei EditText Felder "Artikel" und "Meng"e ein, daraus entsteht ein Einkaufszettel.
Artikel und Menge werden einer ArrayList hinzugefügt.
Der User kann über den Button b1 die Daten auf den Einkaufszettel speichern und danach weitere Elemente hinzufügen. Über den Button b2 wird eine neue Activity (EinkaufszettelAnzeigen) aufgerufen, die über eine ListView seine Eingaben darstellt. Wenn alles passt, wird der Zettel gespeichert, ansonsten verworfen.
Mein Problem dabei ist, dass der letzte Eintrag (Menge) immer doppelt übertragen wird. Ich finde den Fehler nicht, da auch im Debugger bis zum Intent alles korrekt zu sein scheint.
Bsp. Artikel "a" Menge "1" wird in der neuen Activity als a,1,1 ausgegeben, obwohl der Vergleich (Methode, die den letzten Eintrag des Users abfängt und prüft, ob sich dieser Wert bereits in der ArrayList befindet) true ergibt.

Anbei poste ich meinen Code
(sorry für die Länge, unwichtiges hab ich so weit es geht entfernt)

[code]

public class Test extends Activity{

private static EditText Artikel=null;
private static EditText Menge=null;

ArrayList<String> einkauf=new ArrayList<String>();

String artikel=null;
String menge=null;

public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.test);
setTitle("Einkaufszettel anlegen");

// wenn ein rekursiver Aufruf stattfindet
Bundle bundle=getIntent().getExtras();
if(bundle==null){

}else{
String [] array=bundle.getStringArray("array");
int l=array.length;
for(int i=0;i<l;){
einkauf.add(array[i]);
i++;
}}

Artikel=(EditText)findViewById(R.id.eintragenArtikel);
Menge=(EditText)findViewById(R.id.eintragenMenge);
Artikel.setInputType(1);
Menge.setInputType(2);

Button weiter=(Button)findViewById(R.id.b1);
Button speichern=(Button)findViewById(R.id.b2);
Button abbrechen=(Button)findViewById(R.id.b3);

weiter.setOnClickListener(new View.OnClickListener() {
public void onClick(View v) {
addArtikel();
}
});
speichern.setOnClickListener(new View.OnClickListener() {
public void onClick(View v) {
speicherEinkaufzettel();

}
});
abbrechen.setOnClickListener(new View.OnClickListener() {
public void onClick(View v) {
Intent in=new Intent();
in.setClass(Test.this, Artikel_anzeigen.class);
startActivity(in);
}
});
Artikel.setOnKeyListener(new View.OnKeyListener() {
public boolean onKey(View v, int keyCode, KeyEvent event) {
event.getDisplayLabel();
if(keyCode==66 & event.getAction()==KeyEvent.ACTION_DOWN){
getArtikel();
return true;
}
return false;
}
});
Menge.setOnKeyListener(new View.OnKeyListener() {
public boolean onKey(View v, int keyCode, KeyEvent event) {
event.getKeyCode();
if(keyCode==66 & event.getAction()==KeyEvent.ACTION_DOWN){
getMenge();
return true;
}
return false;
}
});
}
public void getArtikel(){
artikel=Artikel.getText().toString();
einkauf.add(artikel);
}
public void getMenge(){
menge=Menge.getText().toString();
einkauf.add(menge);
}

public void speicherEinkaufzettel(){
int a=einkauf.size();
Intent in=new Intent();
in.setClass(Test.this, EinkaufszettelAnzeigen.class);

if(Vergleich()==true){
String []ar =new String[a];

einkauf.toArray(ar);
in.putExtra("array",ar);
startActivity(in);
}
String []ar =new String[a+1];
einkauf.add(menge);
einkauf.toArray(ar);
in.putExtra("array",ar);
startActivity(in);
}

public void addArtikel(){
int a=einkauf.size();
Intent in=new Intent();
in.setClass(Test.this, Test.class);

if(Vergleich()==true){
String []ar =new String[a];
einkauf.toArray(ar);
in.putExtra("array",ar);
startActivity(in);
}
String []ar =new String[a+1];
einkauf.add(menge);
einkauf.toArray(ar);
in.putExtra("array",ar);
startActivity(in);
}
public boolean Vergleich(){
String vergleich=einkauf.get(einkauf.size()-1);
menge=Menge.getText().toString();
if (vergleich.equals(menge)){
return true;
}
return false;
}
}

// Die zweite Activity
public class EinkaufszettelAnzeigen extends Activity{

public void onCreate(Bundle SavedInstanceState){
super.onCreate(SavedInstanceState);
setContentView(R.layout.einkaufzettel_anzeigen);

Bundle bundle=getIntent().getExtras();
String [] array=bundle.getStringArray("array");

ListAdapter adapter=new ArrayAdapter<String>(this, android.R.layout.simple_list_item_multiple_choice, array);
ListView einkauf=(ListView)findViewById(R.id.added_food);
einkauf.setChoiceMode(ListView.CHOICE_MODE_MULTIPLE);
einkauf.setAdapter(adapter);
}
[/code]

vielen Dank im Voraus

Viktor

Hallo zusammen,

ich bin noch mal Schritt für Schritt mit dem Debugger durch den Code gegangen und habe folgendes festgestellt:
Im Formular sind die Daten der ArrayList korrekt, zwei Eingaben bewirken zwei Elemente. Bsp.(a,2)
Seltsamerweise wird die neue Activity zwei mal durchlaufen. Daraufhin habe ich statt eines
OnClickListeners ein OnTouchListener verwendet und das gleiche Ergebnis.
Wie bekomme ich es hin, dass bei einem Event (Touch,Click etc.) die nachfolgende Anweisung nur ein Mal ausgeführt wird?

gruß

Viktor

Hallo Felix,

der Code war nicht gut durchdacht.
Den OnKey Listener hab ich auch rausgeschmissen, genauso wie den rekursiven Aufruf. Die Eingaben werden alle in der ArrayList einkauf gespeichert und dann an die nächste Activity übertragen.

Aber ich stehe jetzt vor einem anderen Problem.
in der Nachfolgenden Activity sollen alle Eingaben des Users (Artikel / Menge) in einer ListView angezeigt werden.
Ich habe dafür eine ListView verwendet, bestehend aus zwei TextViews und einer CheckBox pro Line. Die CheckBox hat den Status checked. Hat der User etwas falsch eingegeben, nimmt er den Haken raus und diese Zeile wird nicht in eine Datenbank geladen.

Mein Problem im Moment ist, dass weder für die CheckBox noch für die TextViews Listener einsetzen kann, da sofort eine Fehlermeldung kommt.
Ich habe ein xml File für die ListView und ein xml File für die Spaltensicht mit der Checkbox erstellt.
Zusätzlich habe ich einer seperaten Klasse einen BaseAdapter implementiert.

Sollte ich die Listener in der getView Methode des BaseAdapter definierten?
Das habe ich gemacht, mit dem Problem, dass ich nicht weiß, wie ich an die selektierten Items komme.

gruß

Viktor

Hallo Felix,

das mit dem Einkaufswagen geht in die richtige Richtung. Ich habe eine Activity "EinkaufszettelAnlegen". Es gibt zwei Eingabefelder, Artikel und Menge. Beide Felder in einer ArrayList gesammelt, bis der Button "speichern" gedrückt wird. Nach dem speichern rufe ich die Activity EinkaufszettelAnzeigen auf.
In dieser Activity wird der Einkaufszettel in einer ListView angezeigt. Der user prüft seine Eingabe und kann noch Artikel hinzufügen oder entfernen. Nach Klick auf den Button "beenden" wird dieser Einkaufszettel in der Datenbank geschrieben und beim nächsten Einkauf aufgerufen.
Mein Problem dabei ist folgendes:
Ich habe eine Activity mit einem "TwoElemenAdapter" extends BaseAdapter. Dazu habe ich ein Layout.xml bestehend aus Header, ListView und den Buttons am Fußenende. Die ListView wiederum bekommt ihr Layout einem weiteren xml file bestehend aus zwei TextViews und einer Checkbox.
Ich habe mich an deinen Tips versucht, weiß im Moment aber nicht wirklich weiter, wie ich die Position des Elementes bestimmen kann, welches der User durch die Checkbox abwählt. Durch die TextViews erhalte ich keine Position.

Anbei mein Adapter:


Und die Activity EinkaufszettelAnzeigen (
[code]
public class Einkaufszettel_Anzeigen extends Activity{

ListView einkaufszettel;
TwoElementAdapter adapterTwo;
Button speichern;
Button verwerfen;
CheckBox check;

String DB_name="WWS_APP";
String DB_table="Warengruppe";

final ArrayList<String> ListeArtikel=new ArrayList<String>();
final ArrayList<String> ListeMenge=new ArrayList<String>();
String [] artikelliste;
String [] mengenliste;
String [] ListeEingaben;
int indexArtikel=0;
int indexMenge=1;
int feld;

public void onCreate(Bundle icicle){
super.onCreate(icicle);

setContentView(R.layout.einkaufszettel_anzeigen);

einkaufszettel=(ListView)findViewById(R.id.einkaufszettel);
adapterTwo=new TwoElementAdapter(this,getArtikel(),getMenge());
einkaufszettel.setChoiceMode(ListView.CHOICE_MODE_MULTIPLE);
einkaufszettel.setAdapter(adapterTwo);

check.setChecked(true);
}

public String[] getArtikel(){
getEingabe();
getFeld();
artikelliste=new String[feld/2];

for(int i=0;i<feld/2;){
artikelliste[i]=ListeEingaben[indexArtikel];
indexArtikel=indexArtikel+2;
i++;
}
return artikelliste;
}

public String[] getMenge(){
getEingabe();
getFeld();
mengenliste=new String[feld/2];

for(int i=0;i<feld/2;){
mengenliste[i]=ListeEingaben[indexMenge];
indexMenge=indexMenge+2;
i++;
}
return mengenliste;
}

public String[] getEingabe(){
Bundle bundle=getIntent().getExtras();
ListeEingaben=bundle.getStringArray("array");
return ListeEingaben;
}

public int getFeld(){
feld=ListeEingaben.length;
return feld;
}

}
[/code]

gruß

Viktor

Hallo Felix,

auf genau die Art und Weise (2. Vorschlag) werden die Arrays in der ListView gesetzt.
Ich übergeb die beiden Arrays an den Kontruktor des Adapters und die Elemente werden an die entsprechende Position geseztz. Diesen weg möchte ich jezt auch rückwärts gehen, für die elemente, bei denen die CheckBox auf false gesetzt wird durch den User.
Ich habe die getView Mehode etwas angepasst:

Muss ich für die ArrayList noch eine getMethode definieren, um auf die Liste aus der ListView Activity zugreifen zu können? Ich sehe, dass ich an den Konstruktor Daten übergebe, weiß im Moment aber nicht, wie ich von diesem Daten bekomme.
Oder benutze ich die getTag() Methode in der ListView Activity?

gruß

Viktor

Hallo Felix,

ich habe den TwoElementAdapter entsprechend umgeschrieben. Ich hab der CheckBox einen Listener hinzugefügt je eine getter und setter Methode:
[code]
check.setOnClickListener(new View.OnClickListener() {
public void onClick(View v) {
int i=(Integer) check.getTag();

text1.getTag(position);
text1.setText(menge[position]);
check.setChecked(false);
String tmp=artikel[i];
setElemente(tmp);

}
});

return vi;
}
public void setElemente(String s){
Elemente.add(s);
}
public ArrayList<String> getElemente(){
return Elemente;
[/code]
Mit "text1.setText(menge[position]);" wollte ich nur prüfen, ob tatsächlich der richtige Wert an die Liste Elemente übergeben wird.
Was mit noch fehlt ist Verwendung der Liste Elemente. Wie greife ich aus der ListView Activity auf die Liste zu? Ich will nach einem Klick auf den Button "speichern" die markierten Elemente in die DB laden.
Wenn ich jedoch einen Listener in der ListView Activity verwende, erhalte ich eine Null Pointer Exception, lasse ich den Listener weg und schreibe stattdesseen

ArrayList<String> test=adapter.getElemente();

erhalte ich eine leere Liste. An diesem Punkt stecke ich jetzt fest.

gruß

Viktor

Hallo Felix,

vielen Dank für die Zeit die du dir nimmst.
wenn ich dich richtig verstanden soll das Programm folgendermaßen aussehen:
Ich habe die Activity "EInkaufszettel anlegen" und dazu eine Activity "irgendeinAdapter extends irgendeinAdapter" plus eine Klasse "Daten".
Aus der Klasse Daten würde ich mir in der AdapterActivity eine ArrayList erzeugen und die nicht markierten Elemente in die Liste eintragen.
Das laden in die DB soll aber erst erfolgen, nach dem der Button "speichern" (und dieses Widget ist nicht Bestandteil der ListView, also auch nicht im Adapter definiert) gedrückt wurde. Aus irgendeinem Grund erhalte ich aber immer, wenn für diesen Button in der Activity einen Listener definiere, eine Null Pointer Exception.


Und beim Item Vorschlag mit der Zusätzlichen Klasse weiß ich nicht so recht, wie ich die Einträge der ArrayList in die Activity "EinkaufszettelAnlegen" bekomme, außer ich schreibe sowas wie getListe() die kontinuierlich gefüllt wird, wenn ich den Haken bei einem Item entferne. Dann habe ich trotzdem noch das Problem mit dem Button, der die Liste in die DB eintragen soll.