Question

所以现在我正试图在C中编写一个程序，该程序接受一个字符串并检查正确的标点符号（例如，结束＆＃39;。＆＃39;，＆＃39;？＆＃39; ，或者＆＃39;！＆＃39;）。我试图使用strchr函数来测试并查看字符串中的最后一个字符是否是在for循环中使用if循环的标点符号之一。但是当我运行程序时，它似乎一起跳过if循环。

以下是该计划：

#include<stdio.h>
#include<string.h>

int main(void)
{
 char string[1000];
 int i,length,a,p,q,e;

 printf("Please enter a sentence with a valid punctuation.\n\n");


 for(i=0;i<1000;i++)
 {
  fgets(string,2,stdin);
  p=strchr(string,'.');
  q=strchr(string,'?');
  e=strchr(string,'!');
  if(string[sizeof(string)-1]=='.'||'?'||'!')
  {
   printf("\nYay a sentence!");
   break;
  }
  else if((p && q && e)==NULL)
  { 
   printf("You didn't provide any punctuation. Goodbye.");
   exit(a);
  }
 }
 printf("You entered the sentence:\n %s",string);

 return 0;
}

我尝试了很多不同的方法，例如尝试strstr，或者甚至以不同的方式存储它（我通过gcc快速学习，而且一些研究不是可行的方法。）

当我输入没有标点符号的句子时，我完全不知道为什么这不起作用。

很抱歉，如果这很简单，我对此很新。

提前致谢。

Answer 1

您误解了strchr的返回值：它不返回该字符的索引;相反，它会将指针返回给您搜索的字符。

char *p=strchr(string, '.');
char *q=strchr(string, '?');
char *e=strchr(string, '!');

此外，sizeof 不返回字符串的实际长度;它返回1000，这是string数组的大小。您需要改为使用strlen。

最后，string[strlen(string)-1]=='.'||'?'||'!'不会将最后一个字符与三个字符之一进行比较。它始终返回1，因为?和!的字符代码不为零，因此逻辑OR ||运算符将它们视为true值。< / p>

同样适用于(p && q && e)==NULL)条件：它不会检查所有三个值都是NULL;其中一个NULL就足以产生平等，但这不是你想要的。

以下是解决此问题的方法：

char last = string[strlen(string)-1];
// Skip '\n's at the end
while (last != 0 && (string[last] == '\n' || string[last] == '\r')) {
    last--;
}
if (last == '.' || last == '?' || last == '!') {
    ...
}
// Using implicit comparison to NULL is idiomatic in C
if (!p && !q && !e) {
    ...
}

Answer 2

strchr返回指针。
像hoge==a||b||c这样的表达式很奇怪。你必须单独写下来。

固定代码：

#include<stdio.h>
#include<string.h>

int main(void)
{
 char string[1000];
 int i,length,a;
 char *p,*q,*e;

 printf("Please enter a sentence with a valid punctuation.\n\n");


 for(i=0;i<1000;i++)
 {
  fgets(string,2,stdin);
  p=strchr(string,'.');
  q=strchr(string,'?');
  e=strchr(string,'!');
  if(string[sizeof(string)-1]=='.'||string[sizeof(string)-1]=='?'||string[sizeof(string)-1]=='!')
  {
   printf("\nYay a sentence!");
   break;
  }
  else if(p==NULL&&q==NULL&&e==NULL)
  { 
   printf("You didn't provide any punctuation. Goodbye.");
   exit(a);
  }
 }
 printf("You entered the sentence:\n %s",string);

 return 0;
}

注意：

string[sizeof(string)-1]将是不确定的，因为它意味着string[999]并且永远不会被写入，因为您告诉fgets string只有2个元素。
a用作exit的参数是不确定的。请初始化它。
length未使用。

strchr不在C工作

2 个答案: