Hilfe bei einem Logik Fehler in php

[Martin L. (tempusertempuser)]

Hallo,
Ich hab ein Formular mit 3 Eingabefeldern
     -username
     -passwort
     -userid

Nach absenden soll überprüft werden ob entweder
    username ausgefüllt und Passwort ausgefullt und userid leer ist
Oder
   Username leer und Passwort leer und userid ausgefüllt wurde

Meine Anfrage lautet

If((empty(username) or empty(password) or !empty(userid) )or !(empty(username) or !empty(password) or empty(userid)))
{ 
FEHLER
}
ELSE
{
WEITER IM CODE
}

Ich land aber immer im if zweig d.h. irgendwo stimmt die Logik nicht, 
Bliss wo????


Danke :)

[Peter II (Gast)]

Michael L. schrieb:
> irgendwo stimmt die Logik nicht,

wenn nur oder vorhanden ist, kann das nichts werden.

> username ausgefüllt und Passwort ausgefullt und userid leer ist

hier steht doch auch ein UND - setze es doch einfach im code genauso um.

[Martin L. (tempusertempuser)]

Hab es so gemacht

 


 if (                                                                          
                        (empty($_POST['f']['username']) and empty($_POST['f']['password']) and  !empty($_POST['f']['userid'])) ||  (!empty($_POST['f']['username']) and !empty($_POST['f']['password']) and empty($_POST['f']['userid']))
                ) {
  FEHLER
} Else
{Code}

Ich lande weiterhin im if zweig somit Fehler wird angezeigt???

[Peter II (Gast)]

Michael L. schrieb:
> Ich lande weiterhin im if zweig somit Fehler wird angezeigt???

ja, weil du mit dem IF den gut-Fall ermittelst. Tausche einfach Fehler 
und code aus oder negiere alles.

[Martin L. (tempusertempuser)]

Irgendwie steh ich Grad aufm schlauch komplett :(

 


 if (                                                                          
                        !(empty($_POST['f']['username']) and empty($_POST['f']['password']) and  !empty($_POST['f']['userid'])) ||  !(!empty($_POST['f']['username']) and !empty($_POST['f']['password']) and empty($_POST['f']['userid']))
                ) {
  FEHLER
} Else
{Code}

Ich lande weiterhin im if zweig somit Fehler wird angezeigt???

[Peter II (Gast)]

Michael L. schrieb:
> Irgendwie steh ich Grad aufm schlauch komplett :(

dann mach dir das leben einfach und schreibe es in getrennten Zeilen. 
Wenn du es jetzt nicht hinbekommt, wirst du es auch nie warten können.

if ( empty(userid) && (!empty(passwort) || (!empty(username) ) 
{
   //Fehler 1
   return;
(                                                                          

if ( !empty(userid) && (empty(passwort) || (empty(username) ) 
{
   //Fehler 2
   return;
(                                                                          

// code wenn erfolgreich

[H. E. (hobby_elektroniker)]

Warum so kompliziert? Weil der Username oder die ID optional aber eines 
davon im Zusammenhang mit dem Passwort erforderlich?

Die kürzeste komprimierteste Schreibweise sähe so aus:

if ( !(!empty($_POST['f']['password']) && ( !empty($_POST['f']['username']) || !empty($_POST['f']['userid'])))) {
  echo "FEHLER";
}
else {
  echo "Alles gut!";
}

Nachteil: Schlecht(er) lesbar bzw. nachvollziehbar.

Ggf. kann man den Else-Teil auch ganz weg lassen, je nachdem wie man den 
Code einsetzt.

Alternativ empfehle ich dir, die Bedingungen in Variablen zu "stückeln" 
und dann lesbarer in den If-Teil zu schreiben. Dann müsste es auch nicht 
so stark komprimiert werden. U.u. kannst du dann die Negation auch 
weglassen.

Das ist vermutlich das, was du wahrscheinlich willst.

Was du geforderst hast:

if(
!( ( !empty( $_POST['f']['password'] ) && !empty( $_POST['f']['username'] ) && empty( $_POST['f']['userid'] ) ) ||
  ( empty( $_POST['f']['password'] ) && empty( $_POST['f']['username'] ) && !empty( $_POST['f']['userid'] ) ) )
) {
  echo "Fehler";
}
else {
  echo "Alles gut";
}

Du könntest empty($bla) jeweils in Variablen als boolischen Wert 
speichern und dann einzeln einsetzen.

[Cyblord -. (Gast)]

Gemäß deinen Anforderungen (Pseudocode):

areCredentialsValid = !empty(username) && !empty(password) && empty(user_id)
isUserIdValid = empty(username) && empty(password) && !empty(user_id)

if !areCredentialsValid || !isUserIdValid
  ERROR
else
  CODE
end