"Oder" - Abfrage wird als "Und" gewertet?

KriZza

Heute hab ich´s auch. Bei mir wird die Abfrage in Zeile 18 als "and" gewertet. Es müsste ja nur eins der beiden geprüft werden beim "or"....

Kann das jemand nachvollziehen?

AutoIt

#include <GUIConstantsEx.au3>




    Local $hGUI = GUICreate("Example")
	$input1 = GUICtrlCreateInput("",5,10)
	$input2 = GUICtrlCreateInput("",5,30)
	$testbtn = GUICtrlCreateButton("test",5,80)


    ; Display the GUI.
    GUISetState(@SW_SHOW, $hGUI)




    While 1
        Switch GUIGetMsg()
            Case $GUI_EVENT_CLOSE
                ExitLoop
			Case $testbtn
				If GUICtrlRead($input1) = "" or GUICtrlRead($input2) = "" Then
					MsgBox(0,"","leer")
				Else
					MsgBox(0,"","Inhalt: " & GUICtrlRead($input1) & " - " & GUICtrlRead($input2))
				EndIf




        EndSwitch
    WEnd

Alles anzeigen

BananaJoe

Ersetze das OR durch ein AND, dann hast Du (wahrscheinlich) das Ergebnis, das Du willst.

Du prüfst jetzt, ob eines der Felder leer ist. Wenn das der Fall ist, dann gibt er leer aus (macht also alles richtig)

KriZza

Oh dann hab nur ich das Problem. So ist´s bei mir:

Fall 1 beides Leer --> so brauche ich das auch

Fall 2 Nur eine der Inputboxen ist gefüllt TROTZDEM LEER: das ist mein Problem

Fall 3 Beide sind befüllt geht... ist ja auch logisch:

Runa

Deine Abfrage macht genau was du ihr gesagt hast:

Wenn Input1 leer ist ODER Input2 leer ist, dann sag mir bitte, dass es leer ist.

"Fall 2 Nur eine der Inputboxen ist gefüllt TROTZDEM LEER: das ist mein Problem"

Dann darfst du kein "OR" verwenden, sondern brauchst "AND".

BananaJoe

Bei mir macht er das (mit Deinem Code) exakt so, wie Du es auf den Bildern hast...

Wie sind die Ergebnisse mit dem AND?

gmmg

bei mir ist das auch so wie es BananaJoe schreibt!
was willst du denn erreichen? vielleicht hast du einen denkfehler.
was soll für eine meldung kommen, wenn nur ein feld gefüllt ist? soll dann kommen, bitte alle felder ausfüllen oder so?
prinzipiell kannst du ja 3 zustände haben.

1. alle felder leer
2. eines der beiden felder leer
3. alle felder gefüllt

und diese zustände musst du alle prüfen.

gruß gmmg

KriZza

Also Ziel soll es sein, wenn EINE der beiden Checkbox leer ist, das trotzdem der Inhalt ausgegeben wird:

also "Inhalt: asdf - "
oder "Inhalt: -asdf"

EDIT: hat sich wirklich mit dem "AND" erledigt. Oh man ...

svart

Wie BananaJoe schon schrieb, aus dem OR ein AND machen dann siehts wie folgt aus:

BananaJoe

Details warum das so ist: Boolesche Algebra

gmmg

dann sollte das so passen!

da kannst du theoretisch aber auch nur die MSGBox machen ohne if.
MsgBox(0,"","Inhalt: " & GUICtrlRead($input1) & " - " & GUICtrlRead($input2))

liest dann beide Felder aus und gibt dir den Inhalt zurück.

gruß gmmg

misterspeed

Ähm ja ist doch alles richtig. Was willst du erreichen? Deine Abfrage prüft ob eines der beiden Felder leer ist (OR = mindestens eine der Bedingungen muss erfüllt sein). Ist eines der beiden (oder beide) leer, dann erscheint der Hinweis "leer". Wenn du stattdessen nur den Hinweis "leer" bekommen möchtest wenn beide Felder leer sind, dann musst du AND verwenden (AND = beide Bedingungen müssen erfüllt sein).

Wenn du das genauer differenzieren möchtest, also alle 3 bzw. 4 Fälle abdecken möchtest brauchst du ein anderes Konstrukt deiner Bedingung.

Denkbar wäre z.B. folgende Lösung:

[autoit]

Case $testbtn
If GUICtrlRead($input1) <> "" and GUICtrlRead($input2) <> "" Then
MsgBox(0,"","Inhalt: " & GUICtrlRead($input1) & " - " & GUICtrlRead($input2))
else
If GUICtrlRead($input1) = "" AND GUICtrlRead($input2) = "" Then
MsgBox(0,"","beide leer")
Else
If GUICtrlRead($input1) = "" Then MsgBox(0,"","Inhalt: " & GUICtrlRead($input1) & " - Feld 2 ist leer!")
If GUICtrlRead($input2) = "" Then MsgBox(0,"","Inhalt: " & GUICtrlRead($input2) & " - Feld 1 ist leer!")
endif
EndIf

[/autoit]

EDIT: Sry hatte das Posting vorm Essen angefangen und erst später abgeschickt...

Runa

Alternativ kannst du auch den bedingten Operator verwenden:

AutoIt

#include <GUIConstantsEx.au3>
    Local $hGUI = GUICreate("Example")
	$input1 = GUICtrlCreateInput("",5,10)
	$input2 = GUICtrlCreateInput("",5,30)
	$testbtn = GUICtrlCreateButton("test",5,80)
    ; Display the GUI.
    GUISetState(@SW_SHOW, $hGUI)
    While 1
        Switch GUIGetMsg()
            Case $GUI_EVENT_CLOSE
                ExitLoop
			Case $testbtn
				MsgBox(0,"","Inhalt: " & ((GUICtrlRead($input1) = '') ? 'Leer' : GUICtrlRead($input1)) & " - " & ((GUICtrlRead($input2) = '') ? 'Leer' : GUICtrlRead($input2)))
        EndSwitch
    WEnd

Alles anzeigen

Führt zu den Ausgaben:

Leer - Leer
GUICTRLRead($input1) - Leer
Leer - GUICTRLRead($input2)

Kürzt den Quellcode ungemein, aber wird auch ein wenig unübersichtlicher.