L'Email de validation en C++

Bon alors je suis en train de faire un programme qui permet à l'utilisateur d'entrer son e-mail. Leur e-mail sera considéré comme valable que si deux conditions sont remplies: A. il doit y avoir un signe "@" quelque part là-bas et B. il doit y avoir une période après le "@". J'ai eu le code en bas pour la plupart, mais j'ai quelques difficultés quand il s'agit de la validation des e-mails qui ont une période avant le signe"@". Si ils ont de la période avant le signe "@" ils sont considérés comme valides, mais ils ne devraient pas l'être. Par exemple, en entrant text.example@randomcom est considéré comme valide.

Quelqu'un peut-il m'aider à comprendre ce que j'ai fait de mal? Je vous remercie à l'avance!

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

25 tampon de caractères pour une adresse email? Je ne suis pas sûr si je devrais pleurer parce que c'est ridiculement court. Votre programme va prendre le train express pour un comportement indéterminé dès que quelqu'un a l'audace de se taper une adresse plus que cela. [email protected] est un modeste longueur de l'adresse qui est trop long par 4 caractères de votre code. C'est pourquoi fixe-longueur de C style tampons sont vraiment de mauvaises nouvelles.
Le titre peut être trompeur, car ce n'est plus la "validation d'un sous-ensemble de e-mails", l'email de validation est étonnamment complexe.
Certaines adresses ne contiennent pas d'un point dans le domaine. Voir RFC2822 pour plus de détails. Adresse Email de validation est un tar-bébé.
Êtes-vous limité à l'utilisation <cstring> ? Ou pouvez-vous utiliser <algorithm> et <regex>?

InformationsquelleAutor ETERNAL OBLIVION | 2016-04-28

c++c-strings email validation

Pourquoi ne pas utiliser des regex?

#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

"Maintenant, vous avez deux problèmes" avec l'aimable autorisation de l'homme lui-même qui a créé un Débordement de Pile! blog.codinghorror.com/...
Vous êtes à la recherche d'une . ou _ avant la @? Ce n'est pas dans les exigences, mais le reste a l'air bon.

InformationsquelleAutor gonjay

1

Le principal problème ici est que c'est censé être un programme en C++, mais, au lieu de cela, il est devenu un programme C. strstr() et strlen() sont des fonctions de la bibliothèque C.

En C++ moderne que nous utilisons std::string, itérateurs, et des algorithmes, ce qui rend la tâche beaucoup plus court et plus facile à analyser. Et il n'y a pas besoin de s'inquiéter à propos de dépassements de la mémoire tampon, soit:
```
#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;
}
```
Maintenant, n'est-ce pas plus court et plus facile à analyser?
- Que if état est au-delà de la portée d'un grand nombre de simples mortels. J'ai dû lire plusieurs fois pour s'assurer qu'elle est correcte. Pas facile à comprendre.
- Mon esprit fonctionne de façon mystérieuse. Je le trouve plus facile à envelopper mon cerveau autour des itérateurs, de garder une trace de l'indice de solutions à utiliser find().
- Le problème n'est pas l'utilisation des itérateurs--les itérateurs sont beaux. Le problème, c'est la double condition dans la if où la première condition modifie les arguments de la seconde. Plus le bonus de comparer le résultat d'une cession à un tiers de la valeur (mais bon, au moins vous avez ajouté l'appropriées parens pour le rendre moins impénétrable).
- J'apprécie hautement l'aide mais je suis d'accord avec John, ce code est un peu trop au dessus de ma tête, afin de bien saisir. Hélas, je suis à peine dans l'Introduction à la CS.
- Même remarque à propos de case à cocher utiliser de getline que j'ai faite en vertu de John réponse s'applique ici....
- À l'aide de strstr() et strlen() n'est pas une C programme.
InformationsquelleAutor Sam Varshavchik
1

Utilisation std::string, pas que le méchant de taille fixe C de la chaîne de trucs.
```
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;
}
```
- Je vous suggère de - if (cin >> input) { ... } else { ...err... } - évite les problèmes avec avant/après/embedded d'espaces (en supposant post-blanc contenu sera validé en quelque sorte par la suite) et les lignes qui ne contiennent pas de tout non-blanc, ou si vous voulez vous assurer que c'est de la très prochaine de la ligne d'entrée qui contient l'e-mail, gardez à l'aide de getline mais vérifiez qu'il réussit et garniture espaces / case vide input par la suite.
InformationsquelleAutor John Zwinck

static bool IsEmailAddress(const std::string& str)
{
    //Locate '@'
    auto at = std::find(str.begin(), str.end(), '@');
    //Locate '.' after '@'
    auto dot = std::find(at, str.end(), '.');
    //make sure both characters are present
    return (at != str.end()) && (dot != str.end());
}

Pouvez-vous fournir une explication de ce code?
1. Essayez de localiser caractère'@'. <br/> 2. Essayez de localiser caractère '.' après le caractère'@'. <br/> 3. Si 1 et 2 sont remplies, la chaîne d'entrée est une adresse email valide. <br/>
L'explication doit être dans la réponse.

InformationsquelleAutor dshvets1

0

Vous avez très restreint, et des règles spécifiques sur les adresses e-mail valides qui ne sont pas le reflet de réelle adresses e-mail. En supposant que c'est intentionnel, le principal problème que je vois est que vous écrivez des boucles lorsque vous n'avez pas besoin. La fonction de la bibliothèque strstr() la boucle pour vous. Vous venez de passer de la chaîne et il sera en boucle à travers la recherche de la char.

Donc, de laisser la fonction de faire la recherche pour vous, vous pouvez diviser et conquérir le problème comme ceci:
```
bool is_valid(char const* email)
{
    auto at_pos = std::strchr(email, '@');

    if(at_pos == nullptr)
        return false; //did not find an '@' (rule A violation)

    auto dot_pos = std::strchr(email, '.');

    if(dot_pos == nullptr)
        return false; //did not find an '.' (rule B violation)

    if(dot_pos < at_pos)
        return false; //'.' found before '@' (rule B violation)

    return true; //all rules followed!
}
```
InformationsquelleAutor Galik

-1

//Program to validate email
#include<iostream>                 //header files
#include<string>
using namespace std;
int strLength(char str[]);
int email_check(char str[])
{                                               //function to check the conditions for email
int size,pos=0,pos1=0,c=0;
size=strLength(str);
if((str[0]>='a'&& str[0]<='z')||(str[0]>='A'&& str[0]<='Z'))  //first char should be an alphabet
{
for(int i=0;i<size;i++)
{
if((str[i]>='a'&& str[i]<='z')||(str[i]>='0' && str[i]<='9') || str[i]=='.'||str[i]=='_'||str[i]=='-'||str[i]=='#'||str[i]=='@')                                         //combination of characters allowed
{
if(str[i]=='.'||str[i]=='_'||str[i]=='-'||str[i]=='#'||str[i]=='@')    //symbol encountered 
{
if((str[i+1]>='a'&&str[i+1]<='z')||(str[i+1]>='0' &&str[i+1]<='9')) //no 2 repeated symbols
{
if(str[i]=='@')                       //@ encountered, so domain part begins
pos=i;                                  //pos is the position where domain begins
}
else
return 0;
}
}
else
return 0;
}
}
else
return 0;
if(pos==0)
return 0;
else
{
for(int j=pos+1;j<size;j++)
{
if(str[pos+1]>='a'&&str[pos+1]<='z')
{
if(str[j]=='.') 
pos1=j;
}
else
return 0;
}
}
if(pos1==0)
return 0;
else
{
for(int k=pos1+1;k<size;k++)
{
if(str[k]>='a'&&str[k]<='z')
c++;
else
return 0;
}
if(c>=2)
return 1;
else
return 0;
}                                           
}                                           //end of function
int main()
{
int c;
char email[100],ch;
do
{
cout<<"\nEnter email: ";
cin.get(email , 100)    ;                                 //accepting email from user
c=email_check(email);
if(c==1)                     //if all criteria matched
{
cout<<"\nemail accepted...\n";
cout<<"\nYour email address is: ";
puts(email);
break;
}
else                               //criteria not matched
{
cout<<"\nInvalid email";
cout<<"\n\nWant to re-enter email(y/n): ";
cin>>ch;
}
}while(ch=='y'||ch=='Y');   //user is asked to enter again until the choice is yes
return 1;
}
int strLength(char str[]) {
int k = 0;
for(k = 0 ; ;k++){
if(str[k] == '\0')
break;
}
return k;
}

Je ne sais par où commencer.

InformationsquelleAutor Siddy Hacks

-2

lors de la recherche du caractère@, puis après ça, au lieu de chercher '.' du début de la chaîne, vous pouvez commencer à partir de la valeur précédente de la variable i.

Remarque:- je ne pense pas beaucoup à ce sujet et tous les autres cas.

InformationsquelleAutor rajshekhar y dodamani
-2

Il devrait être:
```
if(ptr2 != 0 && &ptr2 >&ptr)
```
au lieu de:
```
if(ptr2 != 0 && &ptr2 < &ptr)
```
- Non, les deux sont mauvais. Tout d'abord, la comparaison de pointeurs vers des variables différentes avec > n'est pas bien défini. Deuxièmement, il n'a pas d'avantages à tous dans ce programme. (Le & était probablement pas souhaité)
InformationsquelleAutor user10513642

Vous devez vous connecter pour publier un commentaire.