デリミタで区切った各単語を格納したい

Question

sample.txtを読み込み、デリミタで区切った各単語を配列char **wordsの各要素に格納したいのですが、char.explode関数の中身がおそらく間違えていて実行できません。他の関数内も正直自信がなく手一杯です。どうかご教授お願いいたします。

sample.txtの中身↓

Anyone who has never made a mistake has never tried anything new

実行コード↓

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


void read_text(char *fn, char *text){
  FILE *fp = fopen(fn, "r");
  fgets(text, 9999, fp);
  fclose(fp);
}

int count_words(char *text, const char *delim){
char *p = NULL;
for(int count = 0; ; count++){
p = strstr(text, delim);
if(p == NULL) return count+1;
text = p+1;
 '\0'
   }
}

char **explode(char *text, const char *delim){
  char *p = NULL;
for(char explode  = 0;explode++){
p = strstr(text, delim);
if(p == NULL) return explode;
text = p+1;
 }
}
int main(int argc, char *argv[]){
  char *txt = (char *) calloc(10000, sizeof(char));
  read_text(argv[1], txt);
  printf("入力文字列：%s", txt);
  char *delim = argv[2];
  int num_of_words = count_words(txt, delim);
  printf("単語数：%d\n", num_of_words);
}

エラーメッセージ↓
cpgm/explode.c: In function ‘count_words’:
cpgm/explode.c:20:4: error: expected ‘;’ before ‘}’ token
}
^
cpgm/explode.c: In function ‘explode’:
cpgm/explode.c:25:32: error: expected ‘;’ before ‘)’ token
for(char explode = 0;explode++){
^
cpgm/explode.c:27:22: warning: return makes pointer from integer without a cast [-Wint-conversion]
if(p == NULL) return explode;

---

このような実行結果にしたい
./cpgm/explode data/sample.txt " "
入力文字列：Anyone who has never made a mistake has never tried anything new
単語数：12
1 Anyone
2 who
3 has
4 never
5 made
6 a
7 mistake
8 has
9 never
10 tried
11 anything
12 new

Answer 1

デリミタが 1文字だけなら、strstr でなくて、strchr でいいでしょう。

#include <stdio.h>   // fopen, fclose, fgets, printf
#include <string.h>  // strchr

int read_text(const char *fn, char *text)
{
	FILE *fp = fopen(fn, "r");
	if (!fp || !fgets(text, 1000, fp)) return 1;
	fclose(fp);
	*strchr(text, '\n') = '\0';  // 改行文字を削除
	return 0;
}

int explode(char *text, char **word, char delim)
{
	int n = 0;  char *p;
	for (word[0] = text; p = strchr(word[n], delim); word[++n] = p)
		*p++ = '\0';
	return n + 1;
}

int main(int argc, char *argv[])
{
	char text[1000], *word[500], *p;
	if (argc != 3) return 1;
	if (read_text(argv[1], text)) return 2;
	int n = explode(text, word, argv[2][0]);
	printf("入力文字列：%s\n" "単語数：%d\n", text, n);
	for (int i = 0; i < n; i++) printf("%d %s\n", i+1, word[i]);
}

追記

• read_text の中に 1000 という値が書いてあるのはよくない。
• 改行文字が無い場合を想定していない。
• main で使用しない変数 p が宣言されている。

以上のような不具合を修正します。

#include <stdio.h>   // fopen, fclose, fgets, printf
#include <string.h>  // strchr

int read_text(const char *fn, char *text, int n)
{
	FILE *fp = fopen(fn, "r");
	if (!fp || !fgets(text, n, fp)) return 1;
	fclose(fp);
	char *p = strchr(text, '\n');
	if (p) *p = '\0';
	return 0;
}

int explode(char *text, char **word, char delim)
{
	int n = 0;
	for (char *p = text; p = strchr(word[n++] = p, delim); *p++ = '\0') ;
	return n;
}

int main(int argc, char *argv[])
{
	char text[1000], *word[500];
	if (argc != 3) return 1;
	if (read_text(argv[1], text, sizeof text)) return 2;
	int n = explode(text, word, argv[2][0]);
	printf("入力文字列：%s\n" "単語数：%d\n", text, n);
	for (int i = 0; i < n; i++) printf("%d %s\n", i+1, word[i]);
}

Answer 2

こんなことがやりたいのかな?

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

int split(char* str, char* tokens[], const char* delim) {
  int count = 0;
  for ( char* token = strtok(str, delim); token; token = strtok(NULL, delim)) {
    if ( tokens ) tokens[count] = token;
    ++count;
  }
  return count;
}

int main() {
  char input[] = "きょうは,とても,いい天気";
  char* tokens[10];
  int n = split(input, tokens, ",");
  int i;
  for ( i = 0; i < n; ++i ) {
    printf("[%s] ", tokens[i]);
  }
  printf("\n");
  return 0;
}

[追記] 少なからず小賢しい修正。第二引数:tokensにNULLを渡せば分割数が返ってくるので...:

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

int split(char* str, char* tokens[], const char* delim) {
  char* last = str + strlen(str);
  int count = 0;
  for ( char* token = strtok(str, delim); token; token = strtok(NULL, delim)) {
    if ( tokens ) { tokens[count] = token; }
    else { token[strlen(token)] = *delim; *last = '\0'; }
    ++count;
  }
  return count;
}

int main() {
  char input[] = "きょうは,とても,いい天気";
  char** tokens;
  int n = split(input, NULL, ","); // token数を求め
  tokens = malloc(sizeof(char*)*n); // そんだけの領域を確保して
  split(input, tokens, ","); // もっかい呼ぶ
  int i;
  for ( i = 0; i < n; ++i ) {
    printf("[%s] ", tokens[i]);
  }
  printf("\n");
  free(tokens);
  return 0;
}

Answer 3

まずは、explode関数全体をコメント化して、main関数の中で単語に分解することに挑戦してください。それで、explode関数の呼び出しや返り値にまつわる問題を先送り(ここでは、問題を分割して後で解決するという意味)にできます。

Answer 4

'\0'

これ、意味不明です

for(char explode = 0;explode++){

for文の構文になってません

if(p == NULL) return explode;

戻り値の型があってません。

基本的な文法をまずしっかりマスターする必要がありますねー
おはなしはそれから