Du bist nicht angemeldet.

Stilllegung des Forums
Das Forum wurde am 05.06.2023 nach über 20 Jahren stillgelegt (weitere Informationen und ein kleiner Rückblick).
Registrierungen, Anmeldungen und Postings sind nicht mehr möglich. Öffentliche Inhalte sind weiterhin zugänglich.
Das Team von spieleprogrammierer.de bedankt sich bei der Community für die vielen schönen Jahre.
Wenn du eine deutschsprachige Spieleentwickler-Community suchst, schau doch mal im Discord und auf ZFX vorbei!

Java Android - Denkfehler in for Schleife (?)

Werbeanzeige

MitgliedXYZ

Alter Hase

  • »MitgliedXYZ« ist der Autor dieses Themas

Beiträge: 1 369

Wohnort: Bayern

  • Private Nachricht senden

1

05.10.2014, 12:34

Java Android - Denkfehler in for Schleife (?)

Hi,

ich will den Nutzer auswählen lassen, mit welchem gekoppelten Bluetooth Device eine Verbindung hergestellt werden soll. Dazu sieht der Nutzer ein Meldungsfenster pro Handy und kann entweder auf ok, oder auf abbrechen klicken.
Nachdem ich auf Ok geklickt habe, sollten mir aber keine weiteren Meldungen mehr angezeigt werden. Trotzdem werden diese angezeigt. Hab ich irgendwas übersehen?



Quellcode

 
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

 Set<BluetoothDevice> pairedDevices = mBluetoothAdapter.getBondedDevices();
                if(pairedDevices.size() > 0)
                {
                    for(BluetoothDevice device : pairedDevices)
                    {   
                        if (sollVerbinden == false)
                        {
                            new AlertDialog.Builder(MainActivity.this)
                            .setTitle("Verbinden mit")
                            .setMessage(device.getName())
                            //.setView(input)
                            .setPositiveButton("Ok", new DialogInterface.OnClickListener() {
                                public void onClick(DialogInterface dialog, int whichButton) {
                                    sollVerbinden = true;
                                    Log.d("arduino", "sollVerbinden = true");
                                }
                            }).setNegativeButton("Cancel", new DialogInterface.OnClickListener() {
                                public void onClick(DialogInterface dialog, int whichButton) {
                                    sollVerbinden = false;
                                    Log.d("arduino", "sollVerbinden = false");
                                }
                            }).show();
                            
                            if(sollVerbinden == true) 
                            {
                                mmDevice = device;
                            }
                        }
                    }
                }


Z.b. Es werden vier gekoppelte Geräte gefunden, ich klicke beim zweiten auf ok, sehe aber trotzdem alle vier Meldungen und folgende Logausgabe:
sollVerbinden = false

sollVerbinden = true

sollVerbinden = false

sollVerbinden = false



Aber wie ist das möglich, wenn "sollVerbinden" = true ist, sollte doch mit dem Code keine Meldung angezeigt werden, oder? "sollVerbinden" ist übrigens eine globale Variable.


Gruß,
MitgliedXYZ

BlueCobold

Community-Fossil

Beiträge: 10 738

Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer

  • Private Nachricht senden

2

05.10.2014, 12:41

Das zweite if(sollVerbinden == true) kann übrigens nie true werden - sofern "show" asynchron ist. Ist es synchron oder asynchron? Wenn es asynchron ist, ist das Verhalten logisch, denn dann ist "sollVerbinden" für alle Loop-Durchläufe false und wird erst danach auf true gesetzt. Du kannst das einfach testen, indem Du innerhalb der Schleife und danach mal eine Log-Ausgabe machst. Debuggen hätte vermutlich auch geholfen.
Das erste if kannst Du Dir komplett schenken.
Teamleiter von Rickety Racquet (ehemals das "Foren-Projekt") und von Marble Theory

Willkommen auf SPPRO, auch dir wird man zu Unity oder zur Unreal-Engine raten, ganz bestimmt.[/Sarkasmus]

Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von »BlueCobold« (05.10.2014, 12:53)

MitgliedXYZ

Alter Hase

  • »MitgliedXYZ« ist der Autor dieses Themas

Beiträge: 1 369

Wohnort: Bayern

  • Private Nachricht senden

3

05.10.2014, 12:56

Danke für die schnelle Antwort.

Dann wird das wohl mit den asynchronen Meldungsfenstern zu tun haben.
Wie könnte ich denn das ändern, damit es trotzdem funktioniert?

Die erste if Abfrage weglassen und stattdessen beim zuweisen von mmDevice ein break einzufügen hat leider nichts am Ergebnis verändert.

Quellcode

 
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

if(pairedDevices.size() > 0)
                {
                    for(BluetoothDevice device : pairedDevices)
                    {   
                            new AlertDialog.Builder(MainActivity.this)
                            .setTitle("Verbinden mit")
                            .setMessage(device.getName())
                            //.setView(input)
                            .setPositiveButton("Ok", new DialogInterface.OnClickListener() {
                                public void onClick(DialogInterface dialog, int whichButton) {
                                    sollVerbinden = true;
                                    Log.d("arduino", "sollVerbinden = true");
                                }
                            }).setNegativeButton("Cancel", new DialogInterface.OnClickListener() {
                                public void onClick(DialogInterface dialog, int whichButton) {
                                    sollVerbinden = false;
                                    Log.d("arduino", "sollVerbinden = false");
                                }
                            }).show();
                            
                            if(sollVerbinden == true) 
                            {
                                mmDevice = device;
                                break;
                            }
                    }
                }

BlueCobold

Community-Fossil

Beiträge: 10 738

Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer

  • Private Nachricht senden

4

05.10.2014, 12:58

Wie gesagt, die Schleife wird ERST KOMPLETT ausgeführt und da ist "sollVerbinden" immer false. Erst nach Verlassen der Schleife (bzw. der ganzen Methode) werden die Dialoge angezeigt. Mit dieser Schleife kommst Du jedenfalls nicht weiter.
Teamleiter von Rickety Racquet (ehemals das "Foren-Projekt") und von Marble Theory

Willkommen auf SPPRO, auch dir wird man zu Unity oder zur Unreal-Engine raten, ganz bestimmt.[/Sarkasmus]

MitgliedXYZ

Alter Hase

  • »MitgliedXYZ« ist der Autor dieses Themas

Beiträge: 1 369

Wohnort: Bayern

  • Private Nachricht senden

5

05.10.2014, 13:09

Ok schade, dann muss ich mir einen ganz anderen Ansatz überlegen.
Da lobe ich mir doch bei WindowsForms die MessageBox. Die war wenigstens nicht asynchron, da hat der Rest des Programms noch auf das Ergebnis gewartet.

BlueCobold

Community-Fossil

Beiträge: 10 738

Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer

  • Private Nachricht senden

6

05.10.2014, 13:10

WinForms bietet eine synchrone und eine asynchrone Variante an. Synchron ist allerdings totaler Mist, wenn da noch anderer Kram ist, der aktualisiert werden muss.
Teamleiter von Rickety Racquet (ehemals das "Foren-Projekt") und von Marble Theory

Willkommen auf SPPRO, auch dir wird man zu Unity oder zur Unreal-Engine raten, ganz bestimmt.[/Sarkasmus]

Werbeanzeige

Ähnliche Themen