, wie Login-Zugriff für falsche Anmeldung in iOS Swift

Question

Ich verweigern Zugriff entweder für eine falsche userEmail oder userPassword verweigern, bin mir aber nicht sicher, wie Sie in beiden Details hinzufügen. und mein Code, um die Anmeldung für eine falsche E-Mail zu verweigern, hat keinerlei Auswirkungen oder gibt Fehler zurück. Es kann nur jeder einloggen. Wie kann ich diesen Code korrigieren?

Answer 1

Das Problem ist, dass Sie zuerst überprüfen, ob die E-Mail korrekt ist, und dann, wenn Sie das Kennwort in diesem Block überprüfen. Sie haben keinen Code zum Ausführen bereitgestellt, wenn die E-Mail falsch ist.

Statt es wie folgt zu strukturieren:

if userEmailStored == userEmail { 
    if userPasswordStored == userPassword { 
     // Login successful 
    } else { 
     // Login unsuccessful 
    } 
} 

Versuchen Sie es wie folgt strukturiert:

if userEmailStored == userEmail && userPasswordStored == userPassword { 
    // Login successful 
} else if userEmailStored != userEmail { 
    // Login unsuccessful (email incorrect) 
} else /* if userPasswordStored != userPassword */ { 
    // Login unsuccessful (password incorrect) 
} 

Hinweis: Linie 5, ich habe if userPasswordStored != userPassword den Zustand kommentiert, weil es ist nicht wirklich erforderlich .

Sie können jedoch feststellen, dass wenn Sie sich bei den meisten Websites anmelden, wenn Sie Ihre Daten falsch erhalten, sie nicht angeben, ob es Ihr Benutzername oder Ihre E-Mail war, die falsch waren.

Zum Beispiel:

Sie tun es so, weil es sicherer ist. So können Sie einfach zu prüfen, es so zu tun:

if userEmailStored == userEmail && userPasswordStored == userPassword { 
    // Login successful 
} else { 
    // Login unsuccessful 
} 

---

Wichtiger Hinweis:

Denken Sie daran, dass NSUserDefaults ist in keiner Weise verschlüsselt, so dass, wenn Sie jemals planen auf Speichern Sie einen Benutzernamen und ein Passwort in ... nicht. Es ist als XML (glaube ich) gespeichert, so dass jeder es mit minimalem Aufwand ansehen kann. Verwenden Sie stattdessen Schlüsselbund!

---

Eine weniger wichtige Anmerkung: Es ist ein Ort, wo man zwei ganze Zeilen Leerzeichen haben zwischen Ihrem else if und seiner Halterung und einige andere unordentlich Dinge im Rest des Codes. Sie werden wahrscheinlich mehr Leute bekommen, die Ihre Fragen beantworten, und es wird Ihnen und anderen Menschen viel leichter fallen, Ihren Code zu verstehen, wenn Sie etwas Mühe haben, ihn lesbarer zu machen. Zwei Leerstellen vor einer öffnenden Klammer sind ein Schandfleck, und viele davon können Code wirklich schwer verständlich machen.