c - 按 C 顺序查找带元音的单词

Question

所以对于 uni 的实验室...我面临着在 usr/share/dict/linux.words 中找到所有单词的挑战 按顺序对每个元音使用 fopen、fgets 等文件一次。

即滑稽的

到目前为止，我有以下代码......但它在某处存在缺陷......

int all_vowels( char *s )
{ 
    const unsigned char *p = (const unsigned char *)s;

    char *v = malloc(sizeof(char *));
    char *vowel = v;

if(*p == '\0') return -1;

while( *p != '\0' )
{    
        if( *p == 'a' || *p =='e' || *p =='i'|| *p =='o' || *p =='u' )
        {
            *v = *p;
            v++;
        }
        p++;
    }


    if ( *vowel == 'a' && (*vowel + 1) == 'e' && (*vowel + 2) == 'i' && (*vowel + 3) ==     'o' && (*vowel + 4) == 'u' ) 
    { 
        return 1; 
    }

    return -1;
}

int main (int argc, char *argv[])
{    
    FILE *file;
    char line[BUFSIZ];

    if (( file = fopen("/usr/share/dict/words", "r") ) == NULL) 
    {
        fprintf(stderr, "cannot open %s\n", "/usr/share/dict/words");
        exit(1);
    } 

    while ( !feof(file) )
    {
        fgets(line, sizeof(line), file);
        if ( all_vowels(line) == 1 )
        {
            printf("%s\n", line);
        }
    }
    fclose(file);
    return 0;

}

任何提示都会很棒!!!

我现在真的很迷茫......

Answer 1

您访问 v 就好像它指向一个包含多个字符的位置，而实际上您只为一个 char * (通常是 4 字节)保留空间在 32 位机器上和 8 字节在 64 位机器上):

char *v = malloc(sizeof(char *));

对于您要在其中存储的内容，这可能不够，也可能不够；在您的情况下，任何给定单词中的元音数量。

只要有可能，您应该避免动态分配；在你的情况下，你不需要它们，你可以声明一个固定大小的数组而不是 char*:

char v[5];

除此之外，您还必须检查是否已经读了 5 个元音字母，以免超出数组大小；如果在 5 个元音后遇到另一个，无论如何都可以停止检查；当前遇到的必须是重复的元音，因此该词不合格。

处理字符的方式也是一个问题。再次检查 * 做了什么:它解引用紧靠右边的表达式。在你的例子中，它总是会取消引用 v，然后向它添加一些东西(这也是合法的，因为取消引用的结果是一个字符)。所以如果 v 指向的第一个字符是 a，第二个字符是 e，那么 *v 将产生 'a'，(*v + 1) 将产生 'b'，(*v +2) 将产生 'c' 等等 - 你看，结果是给定数字对字母 a 的加法；第一个字符之后的内容根本无关紧要，因为您永远不会访问那里的值。要使用指针算法实现您想要的，您必须使用括号:*(v+1) - 即，将 1 添加到指针 v，然后取消引用它.这将产生从 v 开始的 c 字符串中的第二个字符，即 'e'。 请注意，使用上面声明的 v，您可以简单地编写 v[0]、v[1]、v[2] 等处理每个字符。

除此之外，检查 if 条件中的最后一个比较，那里有一个“e”而不是“u”。

顺便说一句，作为旁注，还有一些需要考虑的事情:有一个解决你的问题的方法根本不需要 v/vowel 变量...只有一个整数变量!