ポインタ渡しと文字列比較

Question

C言語初心者です。

文字列rawcがリストlistの中に入っていない(登録されていない)とき、
その文字列をリストの登録されている最後に格納する関数regを作成しました。
既に登録されている文字列を上書きする事を防ぐため、
int*型cntをリストに登録された最終番号として関数に渡し、
新しく文字列が登録されたときにはインクリメントするようにしました。
(用いたいプログラムの都合上list[0]には格納しないようにスキップ)
しかしポインタを渡す過程でかプログラムがフリーズしてしまいます。
ポインタ渡しを調べても解決ができませんでした。
どなたかご教授頂きたいです。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void reg(char rawc[50], char list[300][50], int *cnt);

int main(void){
int i;
int cnt;
char raw[300][50], rawc[50];
char list[300][50];
cnt=(int)malloc(sizeof(int));

for(i=0;i<300;i++){
strcpy(list[i],"nothing");
}
strcpy(rawc,"test");
*cnt=0;
reg(rawc,list,cnt);
}
void reg(char rawc[50], char list[300][50], int *cnt){
int i;
int flag=0;
if(*cnt!=0){
for(i=0;i<*cnt;i++){
puts("a");
if(strcmp(rawc,list[i])==0){
flag=1-flag;
break;
}
}
}
if(flag==0){
*cnt++;
strcpy(list[*cnt],rawc);
}
}

Answer 1

フリーズする原因は、*cnt++; の部分ですね。
(*cnt)++; とすれば、フリーズしなくなります。

前者だと、ポインタのアドレスを進めてしまうので、
list[*cnt]でフリーズ。

(*cnt)++; とすれば、ポインタの指し示す値を＋１してくれます。

それから、
この場合、cntは、ポインタ変数で宣言する必要ありませんね。
int cnt;で宣言し、mallocは必要ありません。

cnt=0;
reg(rawc,list,&cnt); としてください。

さらに、reg関数内の処理は、
目的の処理にはなってないんで、フリーズ回避できたら、
勉強のためもう一度見直してみてください。

Comments

[token_regain]

回答ありがとうございます。
int*でなくてよいのですね。
ポインタに限らずあくまでその変数のアドレスを関数に渡してあげれば、
間接的に変数を書き換えることが出来るという理解で正しいでしょうか。
puts()はどこでフリーズするかデバッグしていたもので消し損ねました、
勘違いを生みましたらすみません。
テスト用に省略したデータですので実際に動かしたいデータを掲示いたします。

Answer 2

*cnt++;

この結果はポインタ変数cntに対して＋されますので、参照先の値を＋したい場合は

(*cnt)++;

とします

Comments

[token_regain]

解決しました。
ありがとうございます。
これではポインタを進めることになってしまうのですね。

Answer 3

void reg(char rawc[50], char list[300][50], int *cnt){

ここはシングルポインタとダブルポインタで受け取る形がよいのではないかと思います。
こんな感じ。

void reg(char *rawc, char **list, int *cnt){

テスト中のプログラムのためだと思いますが、

*cnt=0;
reg(rawc,list,cnt);

.
.
.

void reg(char rawc[50], char list[300][50], int *cnt){
int i;
int flag=0;
if(*cnt!=0){

だとif文の判定は常に偽になるのではないかと思います。

Comments

[token_regain]

回答ありがとうございます。
テスト中のプログラムですので、本来試したい形を追加で記入しておきます。

配列についてはダブルポインタを使いこなせないためこのような形になっています。
もし複数の文字列(文字列の長さは50文字以内、文字列数は100以内とします)を
ダブルポインタ**cで管理したいのならば、
int i;
c=(char**)malloc(sizeof(char*)*100);
for(i=0;i<100;i++)
*c=(char*)malloc(sizeof(char)*50);
}
という形で動的にメモリ確保すればよいのでしょうか。
動的なメモリが確保の必要な場合が理解できていないのですが、
ポインタを使用するときは必ず動的メモリ確保をすべきなのでしょうか。
よろしくお願いいたします。

[HachiyaKouya]

ダブルポインタのメモリ動的確保はそれで良いと思います。
配列の変数名に添え字をつけなければ配列の先頭アドレス（ポインタ変数と同じ）になりますので、事前に配列で変数を宣言できると所は配列で宣言するのがソースもシンプルになりますしバグの元も減るので良いと思います。

それと、C言語の文字列は文字列本体の後にヌル文字（"\0"）が入りますので、50文字の文字列を配列に入れる場合は要素数は51必要になります。