C++で電子メールの検証

Question

私はユーザーが自分の電子メールを入力できるようにするプログラムを作ろうとしています。彼らの電子メールは、2つの条件が満たされている場合に有効と見なされます。A.そこには「@」記号があり、Bには「@」の後にピリオドが必要です。私は大部分のコードをダウンしましたが、 "@"記号の前にあるピリオドを持つ電子メールの検証にはいくつかの問題があります。 「@」記号の前にピリオドがある場合は有効とみなされますが、そうではありません。たとえば、text.example@randomcomと入力すると有効と見なされます。

誰かが私が間違っていたことを理解するのを助けることができますか？前もって感謝します！

#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

ここでの主な問題は、これはC++プログラムであると考えられていますが、代わりにCプログラムになりました。 strstr（）とstrlen（）はCライブラリ関数です。

最新のC++では、std::string、イテレータ、アルゴリズムを使用しています。これにより、タスク全体が大幅に短縮され、grokが簡単になります。バッファオーバーフローについて心配する必要はありません。

#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; 
} 

ここで、それほど短くはなく、解析が簡単ですか？

Answer 2

std::stringを使用してください。奇妙な固定サイズのC文字列ではありません。

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

なぜ正規表現を使用しないのですか？

#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