ポインタを使用してこのスワップ関数に問題がありますか？

Question

まだ一般的なプログラミングのnoob。私はこのコードは動作しませんが、なぜですか？

void swap(int num1, int num2) 
{ 
    int* p_first =& num1; 
    int* p_sec =& num2; 
    *p_first = num1; 
    *p_sec = num2; 
} 

Answer 1

関数は、関数のみnum1とnum2の一時的なコピーの上で動作していることを意味し、値で引数を取ります。関数はそれらの値を変更し、それが戻ったときに一時的な値を破棄し、変更されていない実際の値が保持されます。

これを修正するための慣用的なC++の方法は、値の代わりに参照による引数の受け渡しです。

void swap(int &num1, int &num2) { 
    auto temp = num1; 
    num1 = num2; 
    num2 = temp; 
} 

int x = 5, y = 7; 
swap(x, y); 

また、ポインタで引数を渡すこともできます。これはそれを行うための慣用的なCの方法です（Cには参照がありません）。不利な点は、呼び出し側のコードが引数そのものではなく引数のアドレスを渡すことです。もちろん

void swap(int *num1, int *num2) { 
    int temp = *num1; 
    *num1 = *num2; 
    *num2 = temp; 
} 

int x = 5, y = 7; 
swap(&x, &y); 

、2つの数値を交換するために、最善の方法は、車輪の再発明し、代わりにすでにのために書かれstd::swapを、使用しないことです。

int x = 5, y = 7; 
std::swap(x, y); 

Answer 2

あなたの関数は何もせず、出力なしの関数のようです。

理由は次のとおりです。 num1、num2はあなたにも変数をスタックしているp_firstとp_secに自分のアドレスを割り当て、スタック変数です。スタック変数のアドレスは、コードを実行するたびに変更されます。

num1とnum2を交換する理由は、他の2つの変数に値を割り当てた理由（意味がありません）です。あなたはこの

void swap(int num1, int num2) 

のように宣言した場合ので、あなたが使用する必要が

temp = num1 
num1 = num2 
num2 = temp 

をし、C++はちょうどパス値が機能するようになりますので：

少なくとも、あなたはそれらを交換するために、このように考える必要がありますポインタまたは参照を使用します（詳細な実装は簡単に検索できます）。

void swap(int* num1, int* num2) //pointer parameter 
void swap(int& num1, int& num2) //reference parameter 

Answer 3

1 - ここでスワップ関数では値による呼び出しを使用しているため、呼び出し元関数で結果を戻すことはできません。

2 - 下記の機能を実装する - >

void swap(int num1, int num2) { 
    int temp; 
    int* p_first =&num1; 
    int* p_sec =&num2; 
    temp = *p_first; 
    *p_first = *p_sec; 
    *p_sec = temp; 
    printf("\n %d %d \n",num1,num2); 
} 

スワップは現在動作します。