Более правильное решение

[ndiezel]

Есть одна задачка:

Разработчикам приложений время от времени приходится собирать регистрационные данные от
пользователей. При этом некоторые данные принято автоматически проверять на корректность. Мы
предлагаем вам проверить на корректность адрес электронной почты.
Будем считать, что корректный адрес устроен так: LOCAL@DOMAIN. Мы считаем, что он состоитиз части LOCAL, которая стоит до знака @, и части DOMAIN, которая стоит после него. Часть LOCAL состоит из одного или более
слов, которые отделяютcя друг от друга ровно одной точкой. Часть DOMAIN состоит из двух или более слов, которые отделяются друг от друга ровно одной точкой.
Каждое слово это последовательность символов, которая может состоять из маленьких и больших латинских букв, десятичных арабских цифр, а также дефиса "-" и знака подчёркивания "_".
Каждое слово состоит хотя бы из одного символа.

Формат входного файла
В единственной строке входного файла дана последовательность символов с кодами от 32 до 127.
В этой последовательности не более 100 символов.

Формат выходного файла
Выведите "YES", если содержимое входного файла является корректным адресом электронной почты. В противном случае выведите "NO".

Пример
postmaster@example.com YES
***@gmail.com NO

Я написал такое "лобовое" решение:

#include <iostream>
#include <string>

using namespace std;

int main()
{
	string a;
	cin >> a;
	bool local(false), point(false);
	int shet(0);
	for(int i = 0; i < a.length(); i++){
		if((a[i] >= 'a' && a[i] <= 'z') || (a[i] >= 'A' && a[i] <= 'Z')  || (a[i] >= '0' && a[i] <= '9') || a[i] == '-' || a[i] == '.' || a[i] == '_' || a[i] == '@'){
			if((i == a.length() - 1 && (a[i] == '.' || a[i] == '@')) || (i == 0 && (a[i] == '.' || a[i] == '@'))){
				cout << "NO";
				return 0;
			}
			if(a[i] == '.' && i != 0 && point == false){
				point = true;
				if(local == true)
					shet++;
				continue;
			}
			if(a[i] == '.' && i == 0 && point == false){
				cout << "NO";
				return 0;
			}
			if(a[i] == '.' && point == true){
				cout << "NO";
				return 0;
			}
			point = false;
			if(a[i] == '@' && local == false){
				if(a[i-1] == '.'){
					cout << "NO";
					return 0;
				}
				if(a[i+1] == '.'){
					cout << "NO";
					return 0;
				}
				local = true;
				continue;
			}
			if(a[i] == '@' && local == true){
				cout << "NO";
				return 0;
			}
		}
		else{
			cout << "NO";
			return 0;
		}
	}
	if(shet > 0)
		cout << "YES";
	else
		cout << "NO";
	return 0;
}

Не подскажите ли более простое решение, ибо это падает на паре тестов, да и эстетично выглядит жутко.

[BDA]

Вроде все учел:

#include <iostream>
#include <string>
#include <cctype>

using namespace std;

int
isword(int c)
{
    return (isalnum(c) || c == '-' || c == '_');
}

int
main()
{
    string a;
    cin >> a;
    bool word(false), domain(false), address(true);
    int count = 0;
    for (auto i: a) {
        switch (word) {
            case false:
                if (!isword(i)) {
                    address = false;
                    break;
                } else {
                    word = true;
                    count += domain;
                };
                break;
            case true:
                if (i == '@') {
                    domain = true;
                    word = false;
                } else if (!isword(i)) {
                    word = false;
                    if (i != '.') {
                        address = false;
                    }
                }
        }
        if (!address) {
            break;
        }
    }
    if (!address || count < 2 || !domain || !word) {
        cout << "NO";
    } else {
        cout << "YES";
    }
}

[Somebody]

Сообщение от BDA

Вроде все учел

eto@ne@e-mail

[BDA]

Сообщение от Somebody

eto@ne@e-mail

Спасибо.
fixed

#include <iostream>
#include <string>
#include <cctype>

using namespace std;

int
isword(int c)
{
    return (isalnum(c) || c == '-' || c == '_');
}

int
main()
{
    string a;
    cin >> a;
    bool word(false), domain(false);
    int count(0);
    for (auto i: a) {
        if (word) {
             if (!isword(i)) {
                word = false;
                if (!domain && i == '@') {
                    domain = true;
                } else if (i != '.') {
                    break;
                }
            }
        } else {
            if (!isword(i)) {
                break;
            } else {
                word = true;
                count += domain;
            }
        }
    }
    if (count < 2 || !domain || !word) {
        cout << "NO";
    } else {
        cout << "YES";
    }
}

[ndiezel]

Сообщение от BDA

Спасибо.
fixed

Этот вариант спотыкается об те же грабли, что и мой выше представленный. Возможно трабла в условии

[BDA]

Замените "cin >> a;" на "getline(cin, a);".
Если поможет, и у Вас возникнет вопрос "почему?", то спрашивайте

UPD

Подозреваю, что там еще были пробелы, которые простой cin не ловил.

Точно

[ndiezel]

Действительно заработало.
Подозреваю, что там еще были пробелы, которые простой cin не ловил.

[Somebody]

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

using namespace std;

int main()
{
	string a;
	getline(cin, a);
	regex regexEmail(R"([\w_-]+(\.[\w_-]+)*@[\w_-]+(\.[\w_-]+)+)");
	cout << (regex_match(a, regexEmail) ? "YES" : "NO");
}

Если настанет тот день, когда компиляторы и их библиотеки это переварят...