文字列のコピーについて

Question

文字列sorceを文字列strにコピーし、入力された文字数をカウントする関数を作っています。たとえば、文字列sorceにhello!と入力したところ、コピー先の文字列にはフフフフフフフフフフフフフフフフhello!と表示されてしまいました。以前にも同じ内容の質問があったのですが、どうしても分からなかったたまえ、質問させていただきました。また、ビルトしたときに、scanfのところにエラーが表示されました。どこが間違っているかも合わせて教えていただけたら幸いです。よろしくお願いい致します。

#include <stdio.h>
#define ARRAY_SIZE 50

int str_copy(char str[], char sorce[]);
int count = 0;

int str_copy(char str[], char sorce[]) { //sorceがコピー元、strがコピー先
	char temp;
	temp = sorce[ARRAY_SIZE];
	str[ARRAY_SIZE] = temp;

	
	for (int i = 0; sorce[i] != '\0'; i++) {
		
		count = i;
	}
	return count;
}

int main(void)
{
	printf("Put sentences.\n");

	char sorce[ARRAY_SIZE];

	scanf("%s", &sorce);

	char str[ARRAY_SIZE - 1];
	str_copy(str,sorce);

	printf("%s\0",str);

	printf(", length %d \n", count);


	return 0;
}

Comments

[otn]

プログラム中に、コピーしている部分がありませんが、コピペミスでしょうか？

[h_proc]

#include <stdio.h>
#define ARRAY_SIZE 50

int str_copy(char str[], char sorce[]);
int count = 0;

int str_copy(char str[], char sorce[]) { //sorceがコピー元、strがコピー先

for (int i = 0; sorce[i] != '\0'; i++) {
str[i] = sorce[i];

}
printf("これはもとの\n");
printf("%s\n", sorce);

printf("これはコピー先\n");
printf("%s\n", str);

for (int i = 0; str[i] != '\0'; i++) {

count += 1;
}
return count;
}

int main(void)
{
printf("Input sentences.\n");

char sorce[ARRAY_SIZE];

scanf("%s", sorce);

char str[ARRAY_SIZE];

str_copy(str,sorce);

printf("%s",str);

printf(", length %d \n", count);

return 0;
}
修正しました。よろしくお願いい致します。

Answer 1

scanf("%s", &sorce);

%s の場合、引数は、文字列です。 &sorceは、文字列のアドレスです。 多分、多くの処理系は、そのまま、sorceと見なす(らしい)ので、動きそうですが。

temp = sorce[ARRAY_SIZE];
str[ARRAY_SIZE] = temp;

tempに sorce[ARRAY_SIZE] の文字(一文字)を入れています。しかし、sorceのサイズは、ARRAY_SIZEなので範囲外となっています。 (C では、sorceの配列は、0～(ARRAY_SIZE -1) までの配列)

C言語での文字列の扱いを再確認される事を勧めます。

[追記]
temp = sorce[ARRAY_SIZE]; sorce配列の ARRAY_SIZE + 1 番目(範囲外)の文字を temp に代入。
str[ARRAY_SIZE] = temp; str配列の ARRAY_SIZE + 1 番目(範囲外)に tempの文字を代入。
printf("%s\0",str); コピーは失敗しているが、str, sorce の順にメモリが確保(推測)されているため、"hello!" の文字列が出力されていると思われる。

Comments

[h_proc]

ありがとうございます。scanfのところですが、sorce[ARRAY_SIZE]と書き直せば、文字列として認識されるのでしょうか。文字列についてもう一度確認します。

Answer 2

temp = sorce[ARRAY_SIZE];

str[ARRAY_SIZE] = temp;

これ、意味不明です
よく考えてみましょう

char sorce[ARRAY_SIZE];

scanf("%s", &sorce);

これも間違ってます

Comments

[h_proc]

ありがとうございます。
for (int i = 0; i < ARRAY_SIZE - 1; i++) {
sorce[i] = str[i];
}
のように書き直しました。これで正しくコピーされますか。
文字列に対する理解が不十分であったため、もう一度確認して、考え直してみます。

[y_waiwai]

それでいけるでしょう。
ただし、forの条件分は i<ARRAY_SIZE です

って、文字列のコピーなら
strcpy(sorce,str);
で。

[h_proc]

やはりフフフ、、、と表示されてしまいます。このように書き直したのですが、どこが間違っているのでしょうか。また、strcpは使わないで書いています。よろしくお願いい致します。

#include <stdio.h>
#define ARRAY_SIZE 50

int str_copy(char str[], char sorce[]);
int count = 0;

int str_copy(char str[], char sorce[]) { //sorceがコピー元、strがコピー先

for (int i = 0; sorce[i] != '\0'; i++) {
str[i] = sorce[i];

}
printf("これはもとの\n");
printf("%s\n", sorce);

printf("これはコピー先\n");
printf("%s\n", str);

for (int i = 0; str[i] != '\0'; i++) {

count += 1;
}
return count;
}

int main(void)
{
printf("Input sentences.\n");

char sorce[ARRAY_SIZE];

scanf("%s", sorce);

char str[ARRAY_SIZE];

str_copy(str,sorce);

printf("%s",str);

printf(", length %d \n", count);

return 0;
}

[y_waiwai]

現状の条件では、スペースのときはループを抜ける、というものですが、これだけではだめです。
文字列の終わりの時、も条件に含める必要があります

[h_proc]

ループの条件に
if (sorce[i] == '\0') {
str[i] = '\0';
}
else {
//NR
}
を含めたのですが、結果が変わりませんでした。何度も申し訳ありません。終了条件はどのように書くべきなのでしょうか。

[y_waiwai]

あ、先のコメントは別の質問と間違えました

コピー先に文字列の終端が入ってませんね

int i;
for ( i = 0; sorce[i] != '\0'; i++) {
str[i] = sorce[i];
}
str[i]='\0'; // これを付け加える

としてみよう

[h_proc]

ありがとうございます。フフフの表示が消えました。