ポインタで間接参照のレベルが異なりますと表示されます。

Question

前提・実現したいこと

入力した値を変更して表示するプログラムを作っています。
具体手には、
x→z
y→x
z→y
に変更して表示します。

そのためにエラーを取り除きたいですが、できなくて困っています。
お力添えいただけると幸いです。

【追加】
すみません。説明不足でした。
rotateの内部で出力を行う必要があります。

発生している問題・エラーメッセージ

kadai5_2.c(29): warning C4047: '関数': 間接参照のレベルが 'int *' と 'int' で異なっています。
kadai5_2.c(29): warning C4024: 'swap': の型が 1 の仮引数および実引数と異なります。
kadai5_2.c(29): warning C4047: '関数': 間接参照のレベルが 'int *' と 'int' で異なっています。
kadai5_2.c(29): warning C4024: 'swap': の型が 2 の仮引数および実引数と異なります。
kadai5_2.c(30): warning C4047: '関数': 間接参照のレベルが 'int *' と 'int' で異なっています。
kadai5_2.c(30): warning C4024: 'swap': の型が 1 の仮引数および実引数と異なります。
kadai5_2.c(30): warning C4047: '関数': 間接参照のレベルが 'int *' と 'int' で異なっています。
kadai5_2.c(30): warning C4024: 'swap': の型が 2 の仮引数および実引数と異なります。
kadai5_2.c(31): warning C4047: '関数': 間接参照のレベルが 'int *' と 'int' で異なっています。
kadai5_2.c(31): warning C4024: 'swap': の型が 1 の仮引数および実引数と異なります。
kadai5_2.c(31): warning C4047: '関数': 間接参照のレベルが 'int *' と 'int' で異なっています。
kadai5_2.c(31): warning C4024: 'swap': の型が 2 の仮引数および実引数と異なります。

該当のソースコード

#include <stdio.h>

void swap(int *nx,int *ny);
void rotate(int *x,int *y,int *z);

int main(void)
{
	int x,y,z;
	
	printf("3つの整数を入力してください(1マスずつ開けて入力)：");
	scanf("&d %d %d",&x, &y, &z);
	
	rotate(&x,&y,&z);
	
	return 0;
}

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

}

void rotate(int *x,int *y,int *z)
{
	swap(*x,*z);
	swap(*y,*x);
	swap(*z,*y);
	
	printf("出力：%d %d %d",*x, *y, *z);
	

}

試したこと

何をしたらいいのか分からなかったので、試してません。
すみません。

補足情報（FW/ツールのバージョンなど）

サクラエディタ、コマンドプロンプトfor VS 2019を使用しています。

Comments

[dodox86]

[不要なコメントとなったので削除しました]

Answer 1

swap関数の引数はint型のポインタなのに、rotate関数から呼び出す際に実体（値）を渡しているため。

    swap(*x,*z);
    swap(*y,*x);
    swap(*z,*y);
↓
    swap(x,z);
    swap(y,x);
    swap(z,y);

Comments

[mememe0331]

ttyp03様、回答ありがとうございます。
出来ましたが、仕組みが分かりません…。

swapの仮引数がint型のポインタにだったから、＊x＊y＊zを実引数にしました。
何故、＊xだと実体を渡していて、xなら大丈夫なのですか？

申し訳ないのですが教えていただくことは可能でしょうか？

[ttyp03]

rotate関数には各引数int*型で受け取っています。
つまりxという引数にはアドレス値が入っているわけです。
swap関数に*xとして渡してしまうと、xのアドレス値にある実体の値を渡すことになります。
main関数の変数名と引数名が同じだからわかりづらいですが、そういうことです。

余談ですが、実務ではそのような混乱をさけるためにポインタの場合、変数名の頭にpをつけたりします。

void rotate(int *px, int *py, int *pz)
{
　swap(px,pz);
　swap(py,px);
　swap(pz,py);

[hidezzz]

関数の仮引数に付く「*」と、変数を使うときに付ける「*」は意味合いが違います。
仮引数に付いている「*」は変数の型の宣言です。
変数を使うときに「*」を付ける場合それはポインタ演算子です。
引数に「*」が付いてるからといって、変数を使うときにも単純に「*」を付けるというわけではありません。

> void rotate(int *x,int *y,int *z)
「x」という変数が、int型へのポインタ型の変数であることを宣言しています。

変数を使うとき、
「x」はint型のポインタ型で、int*型、
「*x」はポインタ「x」に書かれているint型の数値でint型、
「&x」は[ポインタ「x」のアドレスでint*型のポインタでint**型(今回は出てきません)、
となります。

呼び出そうとしているswap()関数の宣言は「void swap(int* nx, int* ny)」となっているので、
int*の変数を渡す必要があります。
int*型なのは「x」です。

[mememe0331]

お二方ともご回答いただきありがとうございます！
お陰様で理解できました。とても助かりました!!!

Answer 2

#include <stdio.h>

void swap(int* nx, int* ny);
void rotate(int* x, int* y, int* z);

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

    printf("3つの整数を入力してください(1マスずつ開けて入力)：");
    scanf("%d %d %d", &x, &y, &z);
    rotate(&x, &y, &z);
    printf("出力：%d %d %d", x, y, z); /* ここでプリント */

    return 0;
}

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

void rotate(int* x, int* y, int* z) {
    swap(x, z);
    swap(y, x);
//    swap(z, y); /* イラネ */
}