出力結果が@+iみたいな感じにでてしまう

Question

空白ごとに分割して出力したいのですが、
なぜか、"＠+i"みたいな意味不明なものが表示されてしまいます。
どこが悪さしているのでしょうか？

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

void spacePart(char str[], char **num);
// ハッシュ探索を行う関数　引数：なし　戻り値：0
int main()
{
	char str[1000];
	char *num[100];
	puts("100個以内の数値を入力してください。");
	gets(str);
	// 文字列を空白ごと分ける
	spacePart(str, num);
	return 0;
}

// 文字列を空白ごとに分けて配列に格納する関数　引数：入力値、配列　戻り値：なし
void spacePart(char str[], char **num)
{
	// メモリ確保
	num[0] = (char *)malloc(strlen(str) + 1);
	// 空白区切りに文字列を分割
	strcpy(num[0], strtok(str, " "));

	for(int i = 0; num[i] != NULL; i++)
	{	num[i] = (char *)malloc(strlen(str) + 1);
		printf("%s\n", num[i]);
		strcpy(num[i], strtok(NULL, " "));
	}
}

Comments

[episteme]

https://teratail.com/questions/257328
コレ↑にベストアンサー付けておきながら
「実は解決してませんでした(てへぺろ)」ってことですか?

[L4zy]

見てませんでしたてへぺろ

Answer 1

すこーしいぢってみた。

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

char* spacePart(const char* str, char** num);

int main() {
    char* num[100];
    char* buffer = spacePart("1 2 3 4 5 6 7 8 9 10", num);

    // 結果の確認
    int i = 0;
    while ( num[i] != NULL ) {
      printf("[%s] ", num[i]);
      ++i;
    }
    puts("");

    // 後始末
    free(buffer);
    return 0;
}

// 文字列を空白ごとに分けて配列に格納する　
// 引数：入力値、配列
// 戻り値：確保された領域(使い終わったらfreeすべし)
char* spacePart(const char* str, char** num) {
    // メモリ確保
    char* token;
    int i;
    char* buffer = (char*)malloc(strlen(str) + 1);
    strcpy(buffer, str);

    i = 0;
    for ( token = strtok(buffer, " "); token != NULL; token = strtok(NULL," ") ) {
      num[i++] = token;
    }
    num[i] = NULL; // 終わりのシルシ
    return buffer;
}

Answer 2

#include <stdio.h>  // puts, fgets
#include <string.h> // strtok, strdup
#include <stdlib.h> // free

int spacePart(char str[], char **num, int n);

int main(void)
{
    char str[1000];
    char *num[100];
    puts("100個以内の数値を入力してください。");
    if (fgets(str, sizeof str, stdin) == NULL) return 1;
    int n = spacePart(str, num, 100);
	for (int i = 0; i < n; i++) puts(num[i]);
	for (int i = 0; i < n; i++) free(num[i]);
    return 0;
}

int spacePart(char str[], char **num, int n)
{
	char *p = strtok(str, " \n");
	for (int i = 0; i < n; i++) {
		if (p == NULL) return i;
		num[i] = strdup(p);  // strdupの内部で malloc と strcpy
		if (num[i] == NULL) { puts("out of memory"); return -1; }
		p = strtok(NULL, " \n");
	}
	return n;
}

gets よりも fgets をお勧めします。
なお、strdup は非標準関数ですが、多くの処理系には入っています。
このコードについてのコメントをお待ちしております。

Answer 3

自分がなぜそのコードを書いたのか、目的を意識してみましょう。

1. strtokで文字列を分割したい

1-1. strtokは元の文字列を変更してしまう
1-1-A. 事前に複製しよう
1-1-1. そのために、元の文字列の長さ+1文字分の領域を確保して、strcpyでコピーしよう

char* dup = (char*)malloc(strlen(str) + 1);
strcpy(dup, str);
strtok(dup, " ");

1. strtokで各文字列を取り出してコピーをとろう

2-1. そのように書こう

int i = 0;
for(char* next = dup; next != NULL; i++){
    num[i] = (char *)malloc(strlen(next) + 1);
    strcpy(num[i], next);
    // printfを書くならここ
    next = strtok(NULL, " ");
}
num[i] = NULL;

1. 使い終わった領域は解放しよう

free(dup);

みたいな感じで。

Comments

[L4zy]

// 文字列を空白ごとに分けて配列に格納する関数　引数：入力値、配列　戻り値：なし
void spacePart(char str[], char **num)
{
// メモリ確保
*num = (char *)malloc(strlen(str) + 1);
// 空白区切りに文字列を分割
strcpy(num[0], strtok(str, " "));

for(int i = 1; ; i++)
{
num[i] = strtok(NULL, " ");
if(num[i] == NULL) break;
}
free(num);
}
このように変更してみたのですが、どうでしょうか？

[jimbe]

> どこが悪さしているのでしょうか
> どうでしょうか
ご自分で確認する術をお考えになった方が良いのではないでしょうか.

[L4zy]

影分身の術覚えます！

Answer 4

for(int i = 0; num[i] != NULL; i++)

num[i]は初期化されていません

Comments

[L4zy]

配列の初期化はどのように書けば良いのでしょうか？

[pepperleaf]

ついでに、 i = 0 の時の最初の malloc()で得られた領域は、ポインタが上書きされるため、メモリーリーク。
... さきほど、こちら気づかず、別のに回答しましたが、一連の質問ですね。

[kazuma-s]

配列の初期化は、char *num[100] = { 0 }; でできます。
これだと 100個のポインタがすべて NULL で初期化されます。
すると spacePart の中の for文で num[i] が NULL でなければループを回るとなっているので、ループを回りません。