c言語学習中の学生です。

Question

###前提・実現したいこと
ｃ言語を学んでいる学生です。
非常に低レベルな質問ですがお願いします。
Windows10でmingwを使っています。

###発生している問題・エラーメッセージ
文字列を入力して、長さと各文字が何回出現したかを計算するプログラムです
ですが、実行中にプログラムが停止してしまいます。
ちょうど、文字列を入力したあとくらいです。

エラーメッセージ
kadai13は動作を停止しました

###該当のソースコード

#include<stdio.h>

void analyzeString(char *a,int *len,int *cnt);

int main(){
    char string[256];
    int count[26] = {};
    int *lengh = 0;
    int i;
    fgets(string,256,stdin);
    analyzeString(string, lengh, count);
    printf("lengh = %d\n",*lengh);
    for(i=0;i<26;i++){
        printf("%c : %d\n",'a'+i,*(count+i));
    }
}


void analyzeString(char *a,int *len,int *cnt){
    int i;
    *len = 0;
    while(a[*len] != '\0'){//lenをカウントとして使うが実際の値は＋１になる
        for(i=0;i<26;i++){
            if(a[*len] = 'a'+i){
                cnt[i]++;
            }
        }
        *len++;
    }
    *len++;
}

###試したこと
ソースコードを熟読しましたが間違いを見つけることができませんでした

まだまだ未熟ですがアドバイス等お願いします。

Answer 1

ご参考のプログラムです。

int lengh[1] = {};

ポインタがNULLに０値を代入していましたので配列にしました

if(a[*len] == 'a'+i)

=で代入になっていました。

(*len)++;

アドレス値をインクレメントしていましたので変更

#include<stdio.h>

void analyzeString(char *a,int *len,int *cnt);

int main(){
    char string[256];
    int count[26] = {};
    int lengh[1] = {};
    int i;
    fgets(string,256,stdin);
    analyzeString(string, lengh, count);
    printf("lengh = %d\n",lengh[0]);
    for(i=0;i<26;i++){
        printf("%c : %d\n",'a'+i,*(count+i));
    }
}


void analyzeString(char *a,int *len,int *cnt){
    int i;
    *len = 0;
    while(a[*len] != '\0'){//lenをカウントとして使うが実際の値は＋１になる
        for(i=0;i<26;i++){
            if(a[*len] == 'a'+i){
                cnt[i]++;
                break;
            }
        }
        (*len)++;
    }
}
~

Comments

[hoge01]

回答ありがとうございます。
要点を抑えた回答、非常にわかりやすかったです。
今後ともよろしくお願いします。

[A.Ichi]

printの行を削除してしまったので戻しました。

Answer 2

int *lengh = 0;はヌルポインタなので、*lenghとして書き込もうとすればアクセス違反で死にます。

int length;として、analyzeStringに渡すときに&lengthとしましょう。

Comments

[hoge01]

回答ありがとうございます！
非常にわかりやすく説明していただいて、うれしいです。
原因はアクセス違反だったんですね。。。

Answer 3

１．ポインタlenghに0(NULL)を代入してそのままそのアドレス内容を参照しようとしている
２．後置++は、*より優先度が高いので、*len++は、(*len)++でなく*(len++)と同じ
３．count配列の各要素をゼロで初期化していない
４．代入の=と比較の==を間違えている
くらいでしょうか。
表示される長さは、改行まで含んでとなりますね。改行文字をカウントしないなら、
while(a[*len] != '\0' && a[*len] != '\n')
とかでしょうか。

#include<stdio.h>

void analyzeString(char *a,int *len,int *cnt);

int main(){
    char string[256];
    int count[26];
    int lengh = 0;
    int i;
    for(i=0;i<26;i++){
        count[i]=0;
    }
    fgets(string,256,stdin);
    analyzeString(string, &lengh, count);
    printf("lengh = %d\n",lengh);
    for(i=0;i<26;i++){
        printf("%c : %d\n",'a'+i,*(count+i));
    }
}


void analyzeString(char *a,int *len,int *cnt){
    int i;
    *len = 0;
    while(a[*len] != '\0'){//lenをカウントとして使うが実際の値は＋１になる
        for(i=0;i<26;i++){
            if(a[*len] == 'a'+i){
                cnt[i]++;
            }
        }
        ++*len;
    }
}

Comments

[hoge01]

非常にわかりやすい回答ありがとうございました。
++のほうが優先度が高いなど曖昧になっていたところがはっきりとわかり、気持ちいい気分です！
また機会があればよろしくお願いします。
＊せっかく回答してくださったのに、反抗するようになってしまって申しわけないのですが、調べてみたところ、配列は ={}; ですべて初期化できるようです。

[otn]

あ、そうでしたか。