работа цикла while

@alex14058 · Регистрация: 24.05.2017

Author24 — интернет-сервис помощи студентам

здравствуйте
написал вот такой код

C++

#include <iostream>
#include <string>
using namespace std;
bool isDelimeter(char c)
{
    const int size = 37;
    char a[size] = { 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z','.',
    '_','1','2','3','4','5','6','7','8','9'}; 
    for (int i = 0; i < size; i++)
    {
        if (a[i] == c)  return false;
    }
    return true;
}
 
int main()
{
    string str = "asdf hsert 2r asr qew@madsadil..ru hsdfg2@cas.com gio24!@#! !@ asdf@mail.net";
    size_t posStart = 0;
    while ((posStart = str.find("@")) != std::string::npos)
    {
    int a=str.find("@");
    //int g = 0;
    int g = 0;
    for (int c = a + 1; c < str.size(); c++)
    {
        if (isDelimeter(str[c]))
        {
            cout << c << endl;
            g=c;
            break;
        }
         
    }
    int b = 0;
    
    for(int i = a-1; i < str.size(); i--)
    {                                                    //////// вырезка емейла
        if(str[i] == ' ')
          {
             b=i;
             break;
             
          }
          
    }
    cout<<b;
    string emeil = "";
    emeil = str.substr(b,g-b);
    cout<<emeil<<endl;
    int f = emeil.find("@");
    int h = emeil.find("."); // вырезка хоста
    string host = emeil.substr(f+1,h-f-1);
    cout<<host<<endl;
    
    if (!host.empty())
    {
        cout<<"not empty"<<endl;
    }
        if (host.empty())       ////// проверка на наличие хоста
    {
        cout<<" empty"<<endl;
    }
    int point = emeil.find(".");
    int point1 = emeil.find(".",point+1);
    //string zone = emeil.substr(point,999);
    //cout<<zone;
    if (point1 == -1)
    {
        cout<<"ok is point";
    }
    if (point1 != -1)              ////////// проверка повторных точек
    {
        cout<<"wrong address";
    }
}
}

он проверяет корректность емейла
через цикл while вот тут

C++

1 2	size_t posStart = 0; while ((posStart = str.find("@")) != std::string::npos)

пытаюсь пройти по всем возможным емейлам и проверить их корректность
на выходе получаю сейчас
бесконечный цикл с выводом ew@madsadil..ru и его проверкой
а должен получить
qew@madsadil..ru
hsdfg2@cas.com gio24!@
!@
asdf@mail.net
с разными результатами проверки.
я так понимаю проблема в цикле while не могли бы подсказать в чем дело?

@Usaga · 09.10.2017, 07:32

Не по теме:

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

я так понимаю проблема в цикле while не могли бы подсказать в чем дело?

У тебя есть отличный подсказыватель - отладчик (Debugger). Попробуй его спросить.

@zss · 09.10.2017, 08:17

Вы каждый раз ищете знак @ с начала строки, а не с позиции, где кончается очередной email

@alex14058 · 14.10.2017, 20:07 **[ТС]**

переделал
вот так

C++

#include <iostream>
#include <string>
using namespace std;
bool isDelimeter(char c)
{
    const int size = 37;
    char a[size] = { 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z','.',
    '_','1','2','3','4','5','6','7','8','9'}; 
    for (int i = 0; i < size; i++)
    {
        if (a[i] == c)  return false;
    }
    return true;
}
 
int main()
{
    string str = "asdf hsert 2r asr qew@madsadil.ru hsdfg2@cas.com gio24!@#! !@ asdf@mail.net";
    int posStart = 0;
    while ((posStart = str.find("@",posStart+1)) != std::string::npos)
        {
        int a=str.find("@");
        //int g = 0;
        int g = 0;
        for (int c = a + 1; c < str.size(); c++)
        {
            if (isDelimeter(str[c]))
            {
                //cout << c << endl;
                g=c;
                break;
            }
             
        }
        int b = 0;
        
        for(int i = a-1; i < str.size(); i--)
        {                                                    //////// вырезка емейла
            if(str[i] == ' ')
              {
                 b=i;
                 break;
                 
              }
              
        }
       // cout<<b;
        string emeil = "";
        emeil = str.substr(b,g-b);
        cout<<emeil<<endl;
        int f = emeil.find("@");
        int h = emeil.find("."); // вырезка хоста
        string host = emeil.substr(f+1,h-f-1);
       // cout<<host<<endl;
        
        if (!host.empty())
        {
            cout<<"not empty"<<endl;
        }
            if (host.empty())       ////// проверка на наличие хоста
        {
            cout<<" empty"<<endl;
        }
        int point = emeil.find(".");
        int point1 = emeil.find(".",point+1);
        //string zone = emeil.substr(point,999);
        //cout<<zone;
        if (point1 == -1)
        {
            cout<<"ok is point";
        }
        if (point1 != -1)              ////////// проверка повторных точек
        {
            cout<<"wrong address";
        }
    }
}

вот тут изменения которые как я думал помогут

C++

1 2	int posStart = 0; while ((posStart = str.find("@",posStart+1)) != std::string::npos)

вывод сейчас вот такой
qew@madsadil.ru
not empty
ok is point qew@madsadil.ru
not empty
ok is point qew@madsadil.ru
not empty
ok is point qew@madsadil.ru
not empty
ok is point qew@madsadil.ru
not empty
ok is point
Exit code: 0 (normal program termination)

@zss · 14.10.2017, 20:55

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

int a=str.find("@");

А тут почему поиск с начала строки?

@alex14058 · 15.10.2017, 00:10 **[ТС]**

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

А тут почему поиск с начала строки?

C++

1
2
3

int a=str.find("@");
        //int g = 0;
        int g = 0;

тут уже идет поиск @ не посредственно в емейли в данном случае qew@madsadil.ru (во всяком случае по задумки. но видимо я ошибаюсь где то)
если я a заменю на posStart вывод программы не измениться

Nishen · 15.10.2017, 00:19

Вы написали функцию isDelimetr, но почему бы не воспользоваться готовыми средствами? isalpha, isdigit?

@alex14058 · 15.10.2017, 00:28 **[ТС]**

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

Вы написали функцию isDelimetr, но почему бы не воспользоваться готовыми средствами? isalpha, isdigit?

по условиям задачи это требовалось, и второй момент не хотелось пользоваться чем то уже готовым. и если честно пока не знаю как работает isalpha, isdigit. и не хочется просто бездумно использовать не понимая как это работает
разве из за функции isDelimetr вывод не тот что нужно? просто код коряво написан и с ошибками,и я не пойму где в данный момент ошибка

Nishen · 15.10.2017, 01:15

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

не хотелось пользоваться чем то уже готовым

Страуструп пишет, что черта профессионала - умения повторного использования уже наработанного материала.

Не дословно, но смысл тот.

@alex14058 · 15.10.2017, 02:30 **[ТС]**

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

Страуструп пишет, что черта профессионала - умения повторного использования уже наработанного материала.

Не дословно, но смысл тот.

согласен с вами
но мне бы с моим кодом разобраться)))

работа цикла while

Решение

Решение

@Usaga
	09.10.2017, 07:32 #2
	Не по теме: Сообщение от alex14058 я так понимаю проблема в цикле while не могли бы подсказать в чем дело? У тебя есть отличный подсказыватель - отладчик (Debugger). Попробуй его спросить. 0

@zss Модератор 13498 / 10752 / 6407 Регистрация: 18.12.2011 Сообщений: 28,692
	09.10.2017, 08:17	3
	Сообщение было отмечено alex14058 как решение Решение Вы каждый раз ищете знак @ с начала строки, а не с позиции, где кончается очередной email 0

@zss Модератор 13498 / 10752 / 6407 Регистрация: 18.12.2011 Сообщений: 28,692
	14.10.2017, 20:55	5
	Сообщение от alex14058 int a=str.find("@"); А тут почему поиск с начала строки? 0

Nishen 1352 / 851 / 365 Регистрация: 26.02.2015 Сообщений: 3,799
	15.10.2017, 00:19	7
	Вы написали функцию isDelimetr, но почему бы не воспользоваться готовыми средствами? isalpha, isdigit? 0

@alex14058 0 / 0 / 2 Регистрация: 24.05.2017 Сообщений: 87
	15.10.2017, 00:28 [ТС]	8
	Сообщение от Nishen Вы написали функцию isDelimetr, но почему бы не воспользоваться готовыми средствами? isalpha, isdigit? по условиям задачи это требовалось, и второй момент не хотелось пользоваться чем то уже готовым. и если честно пока не знаю как работает isalpha, isdigit. и не хочется просто бездумно использовать не понимая как это работает разве из за функции isDelimetr вывод не тот что нужно? просто код коряво написан и с ошибками,и я не пойму где в данный момент ошибка 0

Nishen 1352 / 851 / 365 Регистрация: 26.02.2015 Сообщений: 3,799
	15.10.2017, 01:15	9
	Сообщение было отмечено alex14058 как решение Решение Сообщение от alex14058 не хотелось пользоваться чем то уже готовым Страуструп пишет, что черта профессионала - умения повторного использования уже наработанного материала. Не дословно, но смысл тот. 0

@alex14058 0 / 0 / 2 Регистрация: 24.05.2017 Сообщений: 87
	15.10.2017, 02:30 [ТС]	10
	Сообщение от Nishen Страуструп пишет, что черта профессионала - умения повторного использования уже наработанного материала. Не дословно, но смысл тот. согласен с вами но мне бы с моим кодом разобраться))) 0

Опции темы