ポインターと関数について

Question

配列aの中身を、配列bにコピーする関数copy_data()の内部がわからないです。
#include <stdio.h>

void data_copy ( const int x[ ], int y[ ], int len );

int main(void) {
int a[] = {1,2,3,4,5};
int b[5];
int i, n;

n = sizeof(a)/sizeof(a[0]);
data_copy( a, b, n );

for( i=0; i<n; i++ )
	printf("%3d", b[i]);
printf( "\n" );

return 0;

}

void data_copy ( const int x[ ], int y[ ], int len ){
for(int i =0;i< len;i++){
printf("%d",x[i] );
}
printf("\n");
}
このように、やってみて結果が以下の様になりました
12345
11153805125219 0 01471873632
どうしたら解決するか教えていただきたいです。
(行いたいこと)配列aの中身を、配列bにコピ－して配列ｂを表示させたいです

Comments

[y_waiwai]

どういうふうに解決したいんでしょうか。
問題が見えません

[otasann]

配列aの中身を、配列bにコピ－して配列ｂを表示させたいです

Answer 1

そもそも、配列コピーしたいだけなら関数自作しなくてもmemcpy関数使えばいいと思いますが…。
↓memcpy関数を使って配列コピー

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

int main(void) {
	int a[] = { 1,2,3,4,5 };
	int b[5] = { 0 };

	memcpy(b, a, sizeof(a));//aからbへsizeof(a)バイト分コピー

	printf("b = {");
	for (int i = 0; i < 5; i++){
		printf(" %d,", b[i]);
	}
	printf("\b }\n");

	return 0;
}

まあ、何かの課題で自作しろって事なのかもしれませんが…。
あと、配列コピーする時はコピー先の確保容量が不足してないかチェックするようにした方がいいですよ。
（上のコードはやってません）

Answer 2

配列aの中身を、配列bにコピ－して配列ｂを表示させたいです

void data_copy ( const int x[ ], int y[ ], int len ){
での引き数yですが、
コピーされてないどころか、全くなんの操作もされていません
そこらへんどうにかしましょう

Comments

[otasann]

Xと同じようにプログラムをかけばいいのでしょうか？

[y_waiwai]

コピーしたい、というのであれば、コピーしましょうよ

[otasann]

for( i=0; i<len; i++ )
printf("%3d", b[x]);
printf( "\n" );
と入れて見たら
1
2
3
4
5
9329364805408 0 0681345632
なりました。
どうすればいいのでしょうか？

[y_waiwai]

そのforのしたに y[i]=x[i]; をいれてみよう

[otasann]

1
2
3
4
5
1 2 3 4 5
出力結果がこうなりました。この場合、このままで大丈夫でしょうか？
縦に並んでいるものを横には出来ないですか？

[cateye]

提示されているコードと、出力結果が違います。
最新（↑の出力になる）コードを追記して下さい。

[dodox86]

一行一行のコードの意味を理解されていないように思うのですが、そんなことは無いでしょうか。