Email-Validierung in C++

Question

Okay, also ich versuche ein Programm zu machen, mit dem Benutzer ihre E-Mails eingeben können. Ihre E-Mail wird als gültig angesehen, wenn zwei Bedingungen erfüllt sind: A. Es muss irgendwo ein "@" Zeichen stehen und B. Es muss eine Zeit nach dem "@" sein. Ich habe den Code größtenteils verloren, aber ich habe einige Schwierigkeiten, wenn es darum geht, E-Mails zu validieren, die vor dem Zeichen "@" stehen. Wenn sie die Zeit vor dem "@" Zeichen haben, gelten sie als gültig, aber sie sollten nicht sein. Beispielsweise wird die Eingabe text.example@randomcom als gültig angesehen.

Kann mir jemand helfen herauszufinden, was ich falsch gemacht habe? Vielen Dank im Voraus!

#include <iostream> 
#include <cctype> 
#include <cstring> 
using namespace std; 

int main() 
{ 
    int x = 25; //random size enough to hold contents of array plus one for    null terminator 
    char input[x]; //array to hold input 
    int sizeOf; //holds length of input array 
    char* ptr = nullptr; //pointer 
    char* ptr2 = nullptr; //pointer 

    cout << "Enter your email address\n"; 
    cin.getline(input,x); 
    sizeOf = strlen(input); 

    for(int i = 0; i < sizeOf; i++) 
    { 
     ptr= strstr(input, "@"); //searches input array for "@" string 
     if(ptr != nullptr) 
     { 
      break; 
     } 
    } 

    for(int i = 0; i < sizeOf; i++) 
    { 
     ptr2 = strstr(input, "."); //searches input array for "." string 
     if(ptr2 != nullptr && &ptr2 > &ptr) 
     { 
      break; 
     } 
    } 

    if(ptr != nullptr) //validates input of "@" sign 
    { 
     if(ptr2 != 0 && &ptr2 < &ptr) 
      { 
       cout << "Email accepted.\n"; 
      } 

     else 
      { 
       cout << "Missing . symbol after @\n"; 
      } 
    } 

    else 
    { 
     cout << "Missing @ symbol\n"; 
    } 



return 0; 
} 

Answer 1

Das Hauptproblem dabei ist, dass dies ein C++ Programm sein sollte, aber es wurde stattdessen ein C-Programm. strstr() und strlen() sind C-Bibliotheksfunktionen.

In modernen C++ verwenden wir std::string, Iteratoren und Algorithmen, die die gesamte Aufgabe viel kürzer und einfacher zu grok machen. Und es gibt keine Notwendigkeit, sich um Pufferüberläufe zu sorgen, entweder:

#include <string> 
#include <algorithm> 

// Your main() declaration here, etc... 

std::string input; 

std::cout << "Enter your email address" << std::endl; 
std::getline(std::cin, input); 

auto b=input.begin(), e=input.end(); 

if ( (b=std::find(b, e, '@')) != e && 
     std::find(b, e, '.') != e) 
{ 
    std::cout << "Email accepted" << std::endl; 
} 
else 
{ 
    std::cout << "Email rejected" << std::endl; 
} 

Nun ist das nicht kürzer und einfacher zu analysieren?

Answer 2

Verwenden Sie std::string, nicht diese böse C-Zeichenfolge Sachen mit fester Größe.

int main() 
{ 
    string input; 
    cout << "Enter your email address\n"; 
    getline(cin, input); 

    size_t at = input.find('@'); 
    if (at == string::npos) 
    { 
     cout << "Missing @ symbol\n"; 
     return 1; 
    } 

    size_t dot = input.find('.', at + 1); 
    if (dot == string::npos) 
    { 
     cout << "Missing . symbol after @\n"; 
     return 2; 
    } 

    cout << "Email accepted.\n"; 
    return 0; 
} 

Answer 3

Warum nicht Regex verwenden?

#include <iostream> 
#include <string> 
#include <regex> 

bool is_email_valid(const std::string& email) 
{ 
    // define a regular expression 
    const std::regex pattern 
     ("(\\w+)(\\.|_)?(\\w*)@(\\w+)(\\.(\\w+))+"); 

    // try to match the string with the regular expression 
    return std::regex_match(email, pattern); 
} 

int main() 
{ 
    std::string email1 = "text.example@randomcom"; 

    std::cout << email1 << " : " << (is_email_valid(email1) ? 
     "valid" : "invalid") << std::endl; 
} 

http://en.cppreference.com/w/cpp/regex