所以对于大学的实验室来说...我遇到的挑战是在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;
}任何提示都会很棒!
我现在真的很困惑……
发布于 2012-09-06 17:52:53
您正在访问v,就像它指向一个包含多个字符的位置一样,而实际上您只为一个char *保留了空间(在32位机器上通常是4字节,在64位机器上通常是8字节):
char *v = malloc(sizeof(char *));对于您试图在其中存储的内容,这可能是足够的,也可能是不够的;在您的示例中,是任何给定单词中的元音数量。
只要有可能,您应该避免动态分配;在您的情况下,您不需要它们,您可以声明一个固定大小的数组,而不是char*。
char v[5];除此之外,您必须检查您是否已经读取了5个元音,以便不超过数组大小;如果在5个元音之后遇到另一个元音,则无论如何都可以停止检查;当前遇到的元音必须是重复的元音,因此该单词将不合格。
寻址字符的方式也是一个问题。再次检查*做了什么:它立即取消对右侧表达式的引用。在你的例子中,它总是取消引用v,然后添加一些东西(这也是合法的,因为取消引用的结果是一个char)。因此,如果v指向的第一个字符是a,第二个字符是e,那么*v将生成'a',(*v + 1)将生成'b',(*v +2)将生成'c',依此类推-你看,结果是字母a加上给定的数字;第一个字符后面的内容并不重要,因为您永远不会访问那里的值。要想用指针算法实现你想要的结果,你必须使用括号:*(v+1) --也就是说,将指针v加1,然后解引用它。这将产生c字符串中从v开始的第二个字符,即'e'。请注意,使用上面声明的v,您可以简单地编写v[0]、v[1]、v[2]等来寻址每个字符。
除此之外,检查if条件中的最后一个比较,那里有一个'e‘而不是一个'u’。
顺便说一句,顺便说一句,还有一些需要考虑的事情:你的问题有一个根本不需要v/vowel变量的解决方案……只有一个整型变量!
发布于 2012-09-06 17:51:18
,但它在某些地方有缺陷...
这里可能有错误吗?
if ( *vowel == 'a' &&
(*vowel + 1) == 'e' &&
(*vowel + 2) == 'i' &&
(*vowel + 3) == 'o' &&
(*vowel + 4) == 'e' )
// ^^^ 'u'?还可能有其他错误。我还没有检查你所有的代码。
发布于 2012-09-06 17:52:24
这里有一个很大的缺陷:
char *v = malloc(sizeof(char *));这只分配四个或八个字节(取决于您是在32位还是64位平台上)。我猜你想要的不止这些。
PS。在未来,你可能应该尝试更具体,而不是仅仅说“它有缺陷”。
https://stackoverflow.com/questions/12297360
复制相似问题