用于检查 char 是否为大写字母并计算给定字符串中的大写字母数量的 C++ 函数

Question

我尝试编写一个 C++ 函数来检查 char 在给定字符串中是否为大写字母。

这是我的尝试：

#include<iostream>
#include <conio.h>
#include<string>
using namespace std;
int iscapital(char x)
{
 if (x>='A'&&x<='Z')    return 1;

 else  return 0;
}
main()
{
char a[20];int len; int c=0;
cout<<"enter your line: ";
cin>>a;
len=strlen(a);
for (int i=0;i<=len;i++)
iscapital(a[i]);
if (iscapital)
{
    c++;
}

cout<<"capital letter in string is: "<<c;
}

Answer 1

您的代码应如下所示：

int iscapital(char x)
{
       if (x >='A' && x <= 'Z')    return 1;
       else  return 0;
}

int main()
{
  char a[20];int len; int c=0;
  cout<<"enter your line: ";
  cin.getline(a , 20);      
  // Note : ' getline ' will read the entire line written in the console and will stop only at the end line mark...will include and the white spaces .
  // http://stackoverflow.com/questions/4745858/stdcin-getline-vs-stdcin

  len=strlen(a);
  for (int i = 0;i < len;i++)
  {
    if (iscapital(a[i]))
    {
       c++;
    }
  }
  cout<<"capital letter in string is: "<<c;

  return 0;
 }

Answer 2

你没有正确使用iscapital。

for (int i=0;i<=len;i++)
    iscapital(a[i]); // Call the function, ignore the result
if (iscapital)   // <- This is not valid C++
{
    c++;
}

你想要的是这个

for (int i=0;i<=len;i++)
    if (iscapital(a[i]))
    {
        c++;
    }

正如其他人评论的那样，查找 std::isupper 以了解字母是否为大写字母，并查找 std::count, std::count_if 以计算值的出现次数或条件为真的次数。

此外，main 应该返回 int 和 iscapital 应该返回 bool。使用int 表示真或假值已过时，不应在新代码中使用。最后，考虑使用std::string 而不是char []。使用字符数组来表示字符串是 C 的做事方式。 C++ 使用std::string 有很多微妙的问题。

Answer 3

更正您的代码：

• IsCapital() 应该返回布尔值而不是整数。

• for (int i=0; i<=len; i++) 这也是你使用的 [len] 所以更正它：

for (int i = 0; i <len; i++)

• 这是什么if (iscapital) { c++;}？这不是如何调用函数isCapital来调用它添加()和参数。

• 使if(iscapital) 在循环内部而不是外部，并且您知道，只要您不添加括号，此处的循环只有一个语句。

所以代码看起来像：

bool iscapital(char x)
{
    if (x >= 'A' && x <= 'Z')
        return 1;
    else
        return 0;
}

main()
{
    char a[20];
    int len;
    int c = 0;

    cout << "enter your line: ";
    cin >> a;
    len = strlen(a);

    for(int i = 0; i < len; i++)
    {
        if (iscapital(a[i]))
            c++;
    }

    cout << "capital letter in string is: " << c;

    return 0;
}