charが大文字であるかどうかをチェックし、指定された文字列の大文字を計算しないようにする関数

Question

charが指定された文字列の大文字であるかどうかを調べるC++関数を作成しようとしました。

#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を使用していない。

は、ここに私の試みです。他の人がコメントしたように何がしたいことは、文字が値の出現数や回数をカウントする資本とstd::count, std::count_ifであるかどうかを知るためにstd::isupperを調べ、この

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

ある

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

条件が真である回数。

さらに、mainは、intおよびiscapitalを返します。boolを返します。真値または偽値を表すのにintを使用することは時代遅れであり、新しいコードでは使用しないでください。最後に、char []の代わりにstd::stringを使用することを検討してください。文字配列を使って文字列を表現することは、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を呼び出す方法ではなく、()とパラメータを追加する方法ではありません。

• make if(iscapital)ループ内にはありません。ここでのループは、かっこを追加しない限り、1つのステートメントしかありません。

ので、コードは次のようになります。あなたがチェックコードを自分で書くために必要されています

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; 
}