isupperを使った文字数カウントが違う。

###実現したいこと
大文字が何回入力されたかを表示したい。

###問題点
大文字の数が違う。
私が考えた方法は
isupperで大文字判定された文字をカウントする
です。
ところどころにprintfをいれて確認しているのですがどこが間違っているのかわかりません。
実行結果

文字列を入力してください
ABChdgD64D
n=10
chara[0]=ABChdgD64D
  cnt=1
chara[2]=ABChdgD64D
  cnt=2
chara[6]=ABChdgD64D
  cnt=3
アルファベット大文字の出現回数は3回です

###コード

C
1//大文字の出現回数
2
3#include<stdio.h>
4#include<ctype.h>
5
6//文字列の長さを求める
7int str_length(const char str[])
8{
9    int len=0;
10    while(str[len]) len++;
11    return len;
12}
13
14int main(void)
15{
16    char chara[99];
17    int cnt=0;
18    printf("文字列を入力してください\n");
19    fgets(chara,99,stdin); //入力
20    unsigned i=0;
21
22    int n=str_length(chara); //文字数  
23    printf("n=%d\n",n);  
24
25    for(i=0;i<n;i++){
26        if(isupper(chara[i])){
27            cnt++;
28            printf("chara[%d]=%s  cnt=%d\n",i,chara,cnt);
29        }
30        i++;
31    }
32    printf("アルファベット大文字の出現回数は%d回です\n",cnt);
33
34    return 0;
35}

行動規範の内容に同意します

回答1件

ベストアンサー

for の中に既に i++ があるのでブロック内の i++; は不要です。

c
1#include <stdio.h>
2#include <string.h>
3#include <ctype.h>
4
5//文字列の長さを求める
6int str_length(const char str[])
7{
8    int len=0;
9    while(str[len]) len++;
10    return len;
11}
12
13int main(void)
14{
15    char chara[99];
16    int cnt=0;
17    printf("文字列を入力してください\n");
18    fgets(chara,99,stdin); //入力
19    unsigned i=0;
20
21    int n=str_length(chara); //文字数  
22    printf("n=%d\n",n);  
23
24    for(i=0;i<n;i++){
25        if(isupper(chara[i])){
26            cnt++;
27            printf("chara[%d]=%c  cnt=%d\n",i,chara[i],cnt);
28        }
29    }
30    printf("アルファベット大文字の出現回数は%d回です\n",cnt);
31
32    return 0;
33}