0

ナル文字を使って、数字文字の出現回数を出力するプログラムを書きたいのですがうまくいきません。自分の見解では、間違っているところが見つけられないのですが、どこが間違っているのでしょうか。

ちなみに、自分は以下のようにプログラムしました。

void str_dcount(const char s[],int cnt[])
{
    int i;
    i=0;
    while(s[i]!='\0'){
        if('0'<=s[i] && s[i]<='9'){
            cnt[s[i]-'0']++;
        }
        i++;
    }
}

int main(void)
{
    char str[128];
    int dcnt[10]={0};
    int i;
    printf("文字列? ");
    scanf("%s",str);
    str_dcount(str,dcnt);
    puts("数字文字の出現回数");
    for(i=0;i<10;i++){
        putchar('\'');
        putchar('0'+i);
        putchar('\'');
        putchar(":%d\n",dcnt[i]);
    }
    return 0;
}
5
  • 1
    putchar(":%d\n", dcnt[i]);printf(":%d\n", dcnt[i]); ではないでしょうか…………
    – user39889
    Commented 2019年12月10日 12:05
  • 1
    最後のputcharprintfの間違いですよね? 思い込みがあると、ちょっとした誤りも見つけにくくなるものです。「自分の見解では、間違っているところが見つけられない」(のに期待通りに動かない)と言う時ほど、丁寧に1行ずつ調べてみてください。
    – OOPer
    Commented 2019年12月10日 12:06
  • 単に「うまくいかない」ではなく、期待だとどう動いてほしくて、現状ではどんな問題があるのかをもう少し具体的に説明してみましょう。ヘルプも参考に。 / できればタイトルも、対面した相手に話しかけるつもりになって整理してみましょう。
    – cubick
    Commented 2019年12月10日 12:38
  • 1
    metropolisさん、OOPerさんコメントの対処を行えば、入力桁数が少ないなら問題なく動作するでしょう。桁数が128以上になると、問題が起こる可能性があります(必ず起こるとは限りません)。こちらの記事を参考に考えてみてください。標準入力から安全に文字列を受け取る方法いろいろ
    – kunif
    Commented 2019年12月11日 1:44
  • お返事ありがとうございます。確かにprintfであるべき場所がputcharになっていました。自分でもわかっていたことなので、これからは慎重かつ丁寧にプログラムしていこうと思います。
    – Acc
    Commented 2019年12月12日 11:47

1 件の回答 1

0

3つの文字列を連結するプログラムの書き方。 の方で使い方に理解が不足しているように見えたので、こちらの対処方法も回答しておきます。

元のscanfでも良いのですが、新しいコンパイラでエラー、警告の出ないこちら scanf_s、_scanf_s_l、wscanf_s、_wscanf_s_l を使います。修正はmain関数だけで、修正箇所にコメントを入れています。

int main(void)
{
    char str[128] = { 0 };/* 初期化しておく */
    int dcnt[10] = { 0 };
    int i;
    printf("文字列? ");
    scanf_s("%127s", str, 128); /* scanf_sに変更し、最大長を指定 */
    str_dcount(str, dcnt);
    puts("数字文字の出現回数");
    for (i = 0; i < 10; i++) {
        putchar('\'');
        putchar('0' + i);
        putchar('\'');
        printf(":%d\n", dcnt[i]); /* @metropolisさん、@OOPerさん指摘 */
    }
    return 0;
}

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。