ファイルの入出力で困っています(C言語)

質問

はじめまして、nathan3と申します。
昔、さらっとC言語を学んでいたので、仕事場でも活用できればと思い、勉強しなおしています。

以下のプログラムですが、コンパイルはするものの、実行がなされません。

sprintfをつかってファイルを作り、fprintfで読み込み、countで繰り返し別名のファイルを読み込み・作成し…といったプログラムを書いているつもりです。

調べながら書いた稚拙なプログラムですが、ここがちがう！というところをお教えいただけると大変助かります。

#include <stdio.h>

int main(void){
FILE *fp,*fo;
char *fname1;
char *fname2;
char s[100],t[100];
int ret,count;

for(count = 0 ; count < 3 ; count++)
{
sprintf(fname1, "sankasha%d.txt", count);
fp = fopen(fname1, "r");
if (fp == NULL){
printf("%s can't open a file\n", fname1);
return -1;
}
sprintf(fname2, "matome%d.txt", count);
fo = fopen(fname2,"w");
if (fo == NULL){
printf("%s can't open a file\n", fname2);
return -1;
}
printf("--fscanf---");
while( (ret = fscanf(fp,"%[^,],%s", s, t)) != EOF ){
fprintf(fo,"%s ", t);
}
}
fclose(fo);
fclose(fp);
return 0;
}

何度見直しても間違いが見つからず困窮しております。
どうぞ、みなさまのお力をお貸しください！
よろしくお願いいたします。

回答No.2

hanabutako · ベストアンサー

char *fname1;
...
sprintf(fname1, "sankasha%d.txt", count);

fname1はポインターであって、配列ではないですよね。
そこにsprintfしてもアクセス違反などで強制終了でしょう。

char fname1[100];
のように配列をちゃんと確保するか、
glibcやBSD系のlibcが使えるならsprintfではなくasprintfを使うかでしょうね。
asprintf(&fname1, "sankasha%d.txt", count);
fp = fopen(fname1, "r");
if (fp == NULL){
printf("%s can't open a file\n", fname1);
return -1;
}
free(fname1);

あと、バッファが溢れないという自信がよほど無い限りはsprintfは使わないですね。
ISO C99で標準化されたsnprintfを使うのが普通だと思います。

まあ、何かのデータの整理をするプログラムはよほど速度が問題にならない限り、自分だったらPython、Perl、Rubyなどのスクリプト言語でプログラムを書くと思いますが。

お礼

>glibcやBSD系のlibcが使えるならsprintfではなくasprintfを使うかでしょうね。
プログラムはさまざまな書き方があって面白い反面難しいです。
hanabutakoさんから提案いただいたものは正直まだよくわかっていませんが、いずれ使えるよう精進したいと思います。

詳細にご説明いただき、ありがとうございました。

回答No.3

maiko0318

とりあえず、

char fname1[20];
char fname2[20];

にしましょう。

char *fname1;
char *fname2;

ですと、ポインターが作られるだけで文字列の格納エリアは作られません。

お礼

無事、実行できました。
ありがとうございます。
ポインタの使い方がいまいち理解できていませんでした。
再度、ポインタの使い方を確認し勉強しなおしたいと思います。

回答No.1

Tasuke22

本当にコンパイルエラー出ないのかな？　出ると思うんだけど。
ワーニングでも潰す必要あり、ですよ。
また実行出来たとしたらエラーメッセージが出ると思うのですが、
エラーメッセージを無視して先には進めません。
質問する場合、これらは全てアップしないと回答も付きにくいでしょう。

さてコードですが突っ込みどころが多過ぎます。
取り敢えず1点、fopenが成功する前にsprintfの実行は不可能でしょう。

お礼

ご指摘ありがとうございます。

おっしゃるように、質問が漠然としておりました。

>取り敢えず1点、fopenが成功する前にsprintfの実行は不可能でしょう。
sprintfは指定した形式の文字列として格納するという認識で、独立して実行はできると思っていたのですが、fopenしないとsprintfが実行できないというのは書き方の問題でしょうか。