ポインタの配列に代入

Question

char型のポインタを宣言し、scanf_ｓで代入しようとすると、コンパイルもデバックも正常にできるのに、文字を打ってエンターを押すとDebug　Assertion　Failed！と警告されてしまいます。ポインタの解釈がまちがっていますか？関数にしてはいけないのでしょうか？

...
char *gook[10];

void go(int i) {
scanf_s("%s",gook[i],4);
}

int main(void){
...
go(0);
...
go(1);

}

こんな感じです。Ｃ言語で、Microsoft　visual　studioを使っています。

Answer 1

ポインタの中身が空に見えます。省略されているのですか？

Answer 2

どう見ても正しい時は、一旦目を変える（目が慣れちゃってる）必要があります。明日になれば、あーって思うと思います。

char *gook[10];

アスタリスクがいりません。
あと、今回不要なんですが、不要かどうかはともかく

char gook[10] = {0};

を習慣づけると、魔除けになります。

※コードを読み間違いしてました。
コメント欄も見てください。
goの入力値が配列の範囲内かどうかチェックすることも考えておいてください。

Comments

[cateye]

"ポインタの配列に代入"と言っているので“アスタリスクがいりません。”はおかしいです？

[kendji]

言われてみればそうですね。こういう処理は大抵ベクターを使うので見抜けませんでした。Cだとデータ領域に50バイトの連続配列を確保して頭から5バイトづつのトップアドレスをポインター配列に代入するとかですかね。ご指摘ありがとうございました????

[cateye]

こういう場合はcharの２次元配列を用意して、そのアドレスを代入するとかですかね？・・・方法は色々有りますが・・・

[kendji]

ありますよね、方法。
2次元配列ありましたね。使えますね。

char gook[10][5] = {0};

でうまくいきそうですね。
オーバーランもしない感じですし。

Answer 3

よくある間違いです。文字列を格納する領域を確保しないで、ポインタを使っています。
「追記」
ポインタを扱うときには、データの格納域を確保してそのアドレスを代入することを忘れないようにしましょう。

char str[]="abcdefg";
char *ptr= str;

こんな感じかな？

usr ~/Project/test % ./a.out
入力開始
123
2 
3
4
5
6
7
8
9
0
入力終了
123
2
3
4
5
6
7
8
9
0

scanf_s()が使えないのでscanf()にしてますが・・・入力時の文字数には気をつけましょう。

#include <stdio.h>

#define PTR_SIZ (10)
#define STR_SIZ (5)

static void go(char *sp)
{
    scanf("%4s",sp);
}

int main(void)
{
    char  str[PTR_SIZ][STR_SIZ];
    char *gook[PTR_SIZ] = { str[0], str[1], str[2], str[3], str[4],
                            str[5], str[6], str[7], str[8], str[9] };
    //
    puts("入力開始");
    for (int i = 0; i < PTR_SIZ; i++) {
        go(gook[i]);
    }
    puts("入力終了");
    //
    for (int i = 0; i < PTR_SIZ; i++) {
        puts(gook[i]);
    }

    //
    return 0;
}

各関数についてはC言語関数辞典などを参考にして下さい。
当方の環境
clang version 10.0.0 (trunk 367677) on Linux mint 19.2

Comments

[kazuyakazuya]

メモリを確保しないで
つまり存在しないはずのメモリを
ポインタ変数？に格納しているから
エラーになるということですか？

[cateye]

グローバル（関数の外）でchar *gook[10];を確保しているから初期値はNULL（０）です。
そこにアクセスするのでエラーになります。ポインタには、有効な変数(intでも何でも)のアドレスを設定しましょう。
蛇足：関数内で確保したポインタの初期値は不定です。なので、何が起こっても仕方ないですよ。

[kazuyakazuya]

ありがとうございます

[kazuyakazuya]

char *gook[10];
に問題があるということですか？

ポインタ変数に文字列(標準入力キーボード アドレスが確保されていない)を格納しようとしているからエラーになると思ったのですが...

[cateye]

scanf()に渡すのはポインタですがscanf()の方は与えられたポインタの指しているアドレス（この場合は０）が“読み込んだデータの格納先”という判断で動きます。そしてそこに読み込んだデータ（この場合は文字列）の格納先と思って、書き込みに行ってコケるのです。

[kazuyakazuya]

ありがとうございます。理解不足でした

Answer 4

char *gook[10];

グローバル変数であるなら、この宣言の時点で、10個のNULLのポインタが確保されます

scanf_s("%s",gook[i],4);

これで、文字列はどこに格納されるのでしょうか。
よく考えてみよう。

Comments

[mix_angel]

あ！本当ですね^_^ NULLポインタになってました。
*gookに初めから仮の値を代入しておいたらちゃんと入るようになりました！
ありがとうございます。