ファイル内の文字列を扱う

Question

ファイルからのデータを配列に保存しようとしていますが、今のところ運がないです。単なる数字の場合にファイルから読み込んだ後にデータを保存するのは簡単ですが、たとえば文字列を保存しようとしている場合など、プログラムは何度も繰り返しクラッシュします。私はfscanf（）を使用しています。関数.txtファイル全体が "姓、名字"と同じ形式で記述されているためです。今度は、私はこの方法でforループを試してみました：

char *firstName = (char*)malloc(sizeof(char)*10240); 
char *lastName = (char*)malloc(sizeof(char)*10240); 
for(int i = 0; i<10; i++){ 
    fscanf(fp, "%s %s", firstName[i],lastName[i]); 
} 

それはクラッシュする場所です。

Answer 1

コードのエラーは次のとおりです。firstName [i]は文字列ではありませんが、％cの代わりに％sを使用して文字列のように使用します。

char *の代わりにchar **を使用する必要があります。

char ** firstName =（char **）malloc（10 * sizeof（char）* 10240）;

10240はfirstNameにはあまりにも多いと思います。 255以下を使用してください。

Answer 2

ピュアCコード：

あなたはそれは大きなサイズの一時文字列に文字列をスキャンし、以降の文字列を複製するのが最善です1 によって各文字列1を割り当て、その後、最初の配列の配列を割り当てる必要があります。それは（確か、またはC99ではない）C準拠したのではなくC++に準拠していないので、私はfor (iにfor (int iを変更した

int i,nb_names = 10; 
char **firstName = malloc(sizeof *firstName * nb_names); 
char **lastName = malloc(sizeof *lastName *nb_names); 
char tempn[1000],templ[1000]; 
for(i = 0; i<nb_names; i++){ 
    fscanf(fp,"%s %s", tempn,templ); 
    firstName[i] = strdup(tempn); 
    lastName[i] = strdup(templ); 

} 

注意。 C++の場合、mallocsを削除し、std::vectorとstd:stringを代わりに使用してください。

できればC++を使用することをお勧めします。私は、2D配列を適切に割り当てることを試みている（そして失敗している）人のC/C++に関する多くの質問に答えました。 C++のライブラリコードを使用するC++ははるかに明確です。 CまたはC++：

#include <vector> 
#include <string> 
#include <iostream> 

using namespace std; 

int main() 
{ 
int nb_names = 10; 
vector<string > firstName(nb_names); 
vector<string > lastName(nb_names); 

for(int i = 0; i<nb_names; i++){ 

    cin >> firstName[i]; 
    cin >> lastName[i]; 

} 
return 0; 
}