fopen（）の問題

Question

私は在庫ファイルをフォーマットするための小さなユーティリティを書いていますので、既存のデータベースにインポートすることができます。私はfscanfをやろうとする大きな問題を抱えています。私は過去数十万のファイルを読んできました。私はここで間違って何をしていますか？

クリストファーの提案に基づく編集はまだNULLになっています。

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

int main() 
{ 

FILE* stream; 
FILE* output; 
char sellercode[200]; 
char asin[15]; 
string sku[15]; 
string fnsku[15]; 
int quality; 

stream = fopen("c:\out\dataextract.txt", "r"); 
output = fopen("c:\out\output.txt", "w"); 

if (stream == NULL) 
{ 
    return 0; 
} 

for(;;) 
{ 
    //if (EOF) break; 
    fscanf(stream, "%s", sku); 
    fprintf(output, "%s %s %s %s %i\n", sku, fnsku, asin, quality); 
} 
return 0; 
} 

Answer 1

問題の多くは、ここで実際にありますする必要があります。クリストファーとマア（コメント中）は、それらのいくつかについて言及しています。しかし、大きな問題は、データを読み書きするための領域を割り当てていないことです。

string sellercode = "";（P.S.、ないあなたはここでしか自分自身を傷つけている、「文字列」型を定義します）を持つことにより、あなたは、charの読み取り専用配列を指しています。実際には、あなたの記憶を破壊する危険にさらされている場合には、あなたのデータを読み込むためにここにいくつかのメモリを残しておく必要があります。

これらの変数を宣言する必要があります。

char sellercode[SOME_REASONABLE_SIZE]; 
char asin[SOME_REASONABLE_SIZE]; 
char sku[SOME_REASONABLE_SIZE]; 
char fnsku[SOME_REASONABLE_SIZE]; 

SOME_REASONABLE_SIZEのいくつかの値を持ちます。

---

常にファイルを明示的に閉じる必要があります。絶対に開かないでください。

---

最後に、パスでバックスラッシュを使用しているが、エスケープされていないことに気づいたことが最後に気づきました。あなたはそれらをエスケープする必要があります。

stream = fopen("c:\\out\\dataextract.txt", "r"); 
output = fopen("c:\\out\\output.txt", "w"); 

Answer 2

if (stream = NULL) 
{ 
    return 0; 
} 

if (stream == NULL) 
{ 
    return 0; 
} 

Answer 3

バックスラッシュは文字列内のエスケープ文字です。パス名にスラッシュを使用します。 "C：/foo/bar.txt"