入力された文字列を分割し、配列に文字列を代入

Question

###前提・実現したいこと
入力された文字列から複数の配列に文字列を入れたいです。
また、それを全て出力したいです。

abc,def,ghi

abc
def
ghi

###発生している問題・エラーメッセージ
ランタイムエラー

###該当のソースコード

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

int main(void){
    char tmp[100];
    int num;
    //出力数を入力
    fgets(tmp, sizeof(tmp), stdin);
    num=atoi(tmp);
    
    char str[1000];
  //文字列を入力
    fgets(str, sizeof(str), stdin);
    char *words[2][20]={""};
    
    //分割
    words[0][0]=strtok(str,",");
    for(int i=0;i<num;i++){
        words[i][0]=strtok(NULL,",");
    }
    
    int cnt=0;
    for(int i=0;i<num;i++){
        while(1){
            if(words[i][cnt]=="")break;
            printf("%c",*words[i][cnt]);
           cnt++;
        }
    printf("\n");
}
    return 0;
}

###試したこと
一次元配列だと一文字しか入らないため、二次元配列にしました。
words[i][]のような形にしましたが、やはり入りませんでした。

###補足情報(言語/FW/ツール等のバージョンなど)
C言語
paiza.io

Answer 1

strtok の挙動がイメージできていないようですね。 strtok は入力された配列、つまり今回の例で言えば str を書き換えます。 カンマを見つけた箇所を文字終端 ('\0') に置き換えた上で、区切られた文字の先頭を指すポインタを返すのです。

なので、結果を保存する配列は文字列を保存する配列ではなく、ポインタを格納する変数を用意しなければならないのです。 メモリの構造を思い浮かべられるかどうかは C を使う上での基本なので、これはもう練習を重ねてくださいとしか言えないのですが、 C の理屈を理解できれば自然に理解できるはずです。

これらを踏まえて記事中のコードを書き換えるならこういう要領になると思います。

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

int main(void){
  char tmp[100];

  //出力数を入力
  fgets(tmp, sizeof(tmp), stdin);
  int num = atoi(tmp);

  char str[1000];
  //文字列を入力
  fgets(str, sizeof(str), stdin);
  char **words = malloc(sizeof(char*)*num);

  //分割
  words[0]=strtok(str,",");

  for(int i=1;i<num;i++)
    words[i]=strtok(NULL,",");

  for(int i=0;i<num;i++)
    printf("%s\n", words[i]);

  free(words);

  return 0;
}

Comments

[tento65]

この短い変更でも曖昧な部分があるのでお聞きしたいことがあります。

mallocの中身でcharポインタ型の*numとありますが、ポインタを扱ってnumを取り出しに行っている、もしくは型のサイズ×numとしてサイズ取得しているのでしょうか？
mallocを調べたところ、恐らく後者だと思うのですがポインタの知識がないので確信が持てませんでした。

[SaitoAtsushi]

後者です。 `malloc` にはバイト数を指定して呼び出します。

[tento65]

初歩的な質問へのご回答ありがとうございました。