配列の入れ替えについて

Question

#include <stdio.h>

void move_array(int *s, int *d, int n );
void show_array( int *x, int n );

int main(void)
{
  int n = 0;
  int a[6];
  a[0] = 3;
  a[1] = 5;
  a[2] = 7;
  a[3] = 9;
  a[4] = 11;
  a[5] = 13;
  
  int b[6];
  
  b[0] = 0;
  b[1] = 0;
  b[2] = 0;
  b[3] = 0;
  b[4] = 0;
  b[5] = 0;
  
  move_array(&a[6], &b[6], 6);
  show_array(&a[6], n);
  
  return 0;
}

void move_array(int *s, int *d, int n )
{
    *s = *d;
  return;
}


void show_array( int *x, int n )
{
  printf("A:");
  for (int i = 0; i < n; i++)
  {
    printf("%d",*x);
  }
  putchar('\n');
  
  printf("B:");
  for (int i = 0; i < n; i++)
  {
    printf("%d",n);
  }
  putchar('\n');
  
}

やりたい事

main関数の二つの配列をmobe_arrayに渡し、関数内で入れ替えたのを今度はshow_arrayに渡してshow_array内で入れ替えた配列を表示させたい。

変更点

void move_array(int *s, int *d, int n )
{
  int tmp = 0;
  for (int i=0; i < n; i++)
  {
    tmp = *s;
    *s = *d;
    *d = tmp;
 }
  
  return;
}

void show_array( int *x, int n )
{
  printf("A:");
  printf("B:");
  for (int i = 0; i < n; i++)
  {
    printf("%d",x[i]);
  }
  
}

#include <stdio.h>

void move_array(int *s, int *d, int n );
void show_array( int *x, int n );

int main(void)
{
  int a[6];
  a[0] = 3;
  a[1] = 5;
  a[2] = 7;
  a[3] = 9;
  a[4] = 11;
  a[5] = 13;
  
  int b[6];
  
  b[0] = 0;
  b[1] = 0;
  b[2] = 0;
  b[3] = 0;
  b[4] = 0;
  b[5] = 0;

  
  move_array(&a[6],&b[6], 6);
  show_array(a, 6);
  show_array(b, 6);
  
  return 0;
}

void move_array(int *s, int *d, int n )
{
  int tmp = 0;
  for (int i=0; i < n; i++)
  {
    tmp = s[i];
    s[i] = d[i];
    s[i] = tmp;
 }
  return;
}


void show_array( int *x, int n )
{
  printf("A:");
  printf("B:");
  for (int i = 0; i < n; i++)
  {
    printf("%d",x[i]);
  }
  
}

Comments

[mather]

変更した現段階のコードを質問に追記すると何が問題か判断しやすいです。追記するときは元のコードを書き換えてしまうのではなく、「## 追記 (2018-05-24 11:30)」のように一区切り入れて追加していくと後で見直した時にどんな修正をしたのかわかりやすいですよ。

Answer 1

配列の初期化について

バグというわけではないですが、非常に冗長に見えるので。
次のように初期化可能です。

int a[] = {3, 5, 7, 9, 11, 13};
int b[] = {0, 0, 0, 0, 0, 0};

なお、0で初期化する場合は次のように書くことも可能です。

int b[6] = {0};

これは別に特殊な記法ではなく、足りない部分が0で補完されるだけです。
(註：必ず0です。{1}で初期化したら{1, 1, ...}になるわけではありません。)

関数へ配列を渡す方法について

実引数の文脈では、配列名は先頭要素のポインタに置き換えられます。
よって、次のように書けます。

move_array(a, b, 6);
// move_array(&a[0], &b[0], 6);  明示的に書いても可

ご提示のコードのように書くと、『6番目の要素のアドレス』を渡そうとしてしまいます。
要素は0番目から5番目までしか存在せず、範囲外アクセスに当たるのでまずいです。

move_array(&a[6], &b[6], 6);

move_arrayの実装について

あくまで仮引数aとbは配列の先頭要素のアドレスを指しています。
ご提示のコードの場合、最初の要素だけしか書き換わらないはずです。

せっかく要素数を一緒に渡しているので、forでぶん回しましょう。

for(int i = 0; i < n; ++i) {
    a[i] と b[i] をスワップ;
}

スワップ処理に関しては、検索すればすぐ見つかるはずです。
べた書きしても良いですし、swap関数を用意してもかまいません。マクロは取扱注意です。

show_arrayの使い方について

インターフェースを見る限り、どうやら一つの配列の値を出力する関数のように見えます。
配列a、配列b、それぞれを出力したい場合、二回呼び出す必要があります。

show_arrayの実装について

for (int i = 0; i < n; i++)
{
printf("%d",*x);
}

*xはこの場合やはり先頭要素の値を解決します。
書き方はいろいろありますが、x[i]を利用するのが素直でしょう。

要素数の渡し方について

これもバグではありませんが、sizeof演算子を上手に利用すると簡潔に書けます。
関心があるようでしたら調べてみてください。

変数nの取り扱いについて

int n = 0;

...

move_array(&a[6], &b[6], 6);

show_array(&a[6], n);

これだとnが初期値0のまま変わりませんね。
move_arrayの第3仮引数の名前がnですが、これはmain関数スコープの変数nとは無関係です。

Comments

[退会済みユーザー]

スワップさせて見ましたが入れ変われていませんでした。
関数内のコードがいけないのでしょうか？

[LouiS0616]

ご提示のコードでは*sと*dは常に先頭要素を指しているので、先頭が何度も入れ替わります。
つまり偶数回ループを回したら元に戻ると言うことです。

*sと*dではなく、s[i]とd[i]を入れ替えるようにしてください。

[退会済みユーザー]

mobe_arrayの関数は解決できましたがどうしてもshow_arrayに値を渡せないのですがどうすればいいのでしょうか？
問題文追記しました。

[LouiS0616]

show_arrayについては、一つの配列の要素を順に表示するように実装し、二回呼び出すのが良いでしょう。
show_array(a, 6);
show_array(b, 6);

[退会済みユーザー]

試して見たのですがshow_arrayにて値が入れ替わっていませんでした。
それと表示させ方が質問欄のようなイメージに出来ないのですが何がいけないんでしょうか？

[LouiS0616]

move_arrayの呼び出し方を誤っていませんか。

[LouiS0616]

あとmove_arrayの実装もちょっと変ですね。
tmp = s[i];
s[i] = d[i];
s[i] = tmp; // ←誤り。d[i] = tmp;