Домашнее задание с использованием strchr дает мне неожиданный вывод - PullRequest
Новая
       50

Домашнее задание с использованием strchr дает мне неожиданный вывод

1 голос
/ 11 апреля 2020

Мне нужно написать программу C, которая выводит количество слов в тексте, а на следующей строке слова, начинающиеся и заканчивающиеся либо гласной, либо согласной.

Для пример: 

input
La bacalaureat la proba de Informatica a fost un subiect cu un sir de caractere

output
15     
bacalaureat Informatica a fost subiect sir

Вот моя программа: 

#include <iostream>
#include <string.h>

using namespace std;

int main()
{
    int c = 0;
    char sir[100], copie[100];
    cin.get(sir, 100);
    strcpy(copie, sir);
    char* pch = strtok(sir, " ");
    while (pch != NULL)
    {
        c++;
        pch = strtok(NULL, " ");
    }
    cout << c << endl;
    char* pch2 = strtok(copie, " ");
    while (pch2 != NULL)
    {
        if ((strchr("aeiouAEIOU", pch2[0]) && strchr("aeiouAEIOU", pch2[strlen(pch2)-1])) || (strchr("aeiouAEIOU", pch2[0] == NULL) && strchr("aeiouAEIOU", pch2[strlen(pch2)-1] == NULL)))
        {
            cout << pch2<<" ";
        }
        pch2 = strtok(NULL, " ");
    }
}

Но это дает мне неожиданный вывод. Я получаю: 

15
La bacalaureat la proba de Informatica a fost un subiect cu un sir de caractere

и не могу понять почему. Это, очевидно, из-за этого, если, но это выглядит хорошо для меня.

Можете ли вы помочь мне исправить мою программу? Я не понимаю, что происходит. Спасибо.

Ответы [ 2 ]

2 голосов
/ 11 апреля 2020

Это подвыражение, используемое в операторе if 

strchr("aeiouAEIOU", pch2[0] == NULL) && strchr("aeiouAEIOU", pch2[strlen(pch2)-1] == NULL)

неверно.

Должно быть 

strchr("aeiouAEIOU", pch2[0]  ) == NULL && strchr("aeiouAEIOU", pch2[strlen(pch2)-1] ) == NULL

Вы можете написать оператор if намного проще следующим образом 

if ( ( strchr( "aeiouAEIOU", pch2[0] ) == NULL ) == 
     ( strchr( "aeiouAEIOU", pch2[strlen(pch2)-1] ) == NULL ) )

Но в любом случае создание копии исходной строки и использование strtok не очень хороший подход.

Лучше использовать стандартную C функцию strspn и strcspn для извлечения слов в строке без ее изменения.

Вы можете хранить слова, которые удовлетворяют условию, в векторе.

Вот демонстрационная программа. 

#include <iostream>
#include <utility>
#include <vector>
#include <cstring>
#include <cctype>

int main() 
{
    const char *s = "La bacalaureat la proba de Informatica "
                    "a fost un subiect cu un sir de caractere";

    const char *vowels = "aeiou";
    const char *delim = " \t";

    std::vector<std::pair<const char *, const char *>> v;
    size_t n = 0;

    for ( const char *p = s; *p != '\0'; )
    {
        p += std::strspn( p, delim );
        const char *q = p;

        if ( *q )
        {
            ++n;
            p += std::strcspn( p, delim );

            char c1 = std::tolower( ( unsigned char )q[0] );
            char c2 = std::tolower( ( unsigned char )p[-1] );

            if ( ( std::strchr( vowels, c1 ) == nullptr ) == 
                 ( std::strchr( vowels, c2 ) == NULL ) )
            v.emplace_back( q, p );              
        }
    }

    std::cout << n << '\n';
    for ( const auto &p : v ) std::cout.write( p.first, p.second - p.first ) << ' ';
    std::cout << '\n';

    return 0;
}

Его выход 

15
bacalaureat Informatica a fost subiect sir
1 голос
/ 11 апреля 2020

Вот версия вашей программы, использующая strchr, но использующая std::istringstream для разбиения слов: 

#include <iostream>
#include <string>
#include <cstring>
#include <sstream>
#include <vector>

bool isvowel(char ch)
{
    return strchr("aeiouAEIOU",ch)?true:false;
}

bool isconsonant(char ch)
{
    return !isvowel(ch);
}

int main()
{
    std::vector<std::string> sVect;
    int word_count = 0;
    std::string test = "La bacalaureat la proba de Informatica a fost un subiect cu un sir de caractere";
    std::istringstream strm(test);
    std::string word;
    while (strm >> word)
    {
        ++word_count;
        if ( (isvowel(word.front()) && isvowel(word.back())) ||
             (isconsonant(word.front()) && isconsonant(word.back())))
            sVect.push_back(word);     
    }
    std::cout << word_count << "\n";
    for (auto& s : sVect)
        std::cout << s << " ";
}

Вывод: 

15
bacalaureat Informatica a fost subiect sir

Обратите внимание, что тест для гласных или согласных помещены в отдельные функции, что облегчает сравнение.

Также обратите внимание на использование front() и back() класса std::string для простого получения первого и последнего символа каждого слова вместо неоднократного вызова strlen в сравнении.

...