3つの値を昇順に並びかえるプログラム

Question

3つの値を昇順に並び替えたいのですが、うまくいきませんでした。どのようにしたらよいのでしょうか

　　
＃include <stdio.h>

void swap(int *nx, int *ny)
{
int temp = *nx;
*nx = *ny;
*ny = temp;
}

void sort3(int *n1, int *n2, int *n3)
{
if (*n1 > *n2) swap(n1, n2);
if (*n2 > *n3) swap(n2, n3);
if (*n1 > *n2) swap(n1, n2);
}

int main(void)
{
int x,y,z;

printf("x:\n");
scanf("%d",&x);
printf("y:\n");
scanf("%d",&y);
printf("z:\n");
scanf("%d",&z);

printf("x:%d,y:%d,z:%d\n",x,y,z);
printf("昇順に並びかえると\n");
sort(&y,&z);
sort(&x,&y);
sort(&y,&z);
printf("x:%d,y:%d,z:%d\n",x,y,z);
return 0;
}

Comments

[cateye]

sort3()しか無いのに、sort(&y,&z);・・・って、どこ？

[KYUURI]

sort3(&x, &y, &z); だと考えましたが、だめでした。
sort3のところをどのように変更すればよいのでしょうか

[SHOMI]

CとC#は別の言語です。Cの質問であれば、C#のタグを外してください。

[SHOMI]

sort3()はn1<n2<n3になるようにソートしてるんだから、x>y>zになるようソートしたいならsort3(&z, &y, &x);じゃないの？
もしくはsort3()内のifとswapのn1とn3を逆にする。
(追記)すみません、これだと降順ですね…

[cateye]

いや、sort()とsort3()では、関数名が違うでしょう？　引数の数も合っていない・・・そもそも、リンクエラーでは？

[KYUURI]

再度sort(&y,&z);sort(&x,&y);sort(&y,&z);をsort3(&x, &y, &z);にしたらできました

Answer 1

手元のC処理系では、単に、

sort(&y,&z);
sort(&x,&y);
sort(&y,&z);

を

sort3(&x, &y, &z);

とするだけでx, y, zの値が昇順(x <= y <= z)になったけど、どこらへんが「だめでした」なんだろう。

Comments

[KYUURI]

もう一度実行してみたところちゃんとできました。