ポインタでスワップ関数を作成する際の問題

Question

フロートポインタで自分自身でスワップ関数を作成しようとしていて、うまくいきません。何らかの理由で私は正しい方法で関数へのfloatポインタを渡さないと思う。

#include <stdio.h> 
#include <stdlib.h> 

void swap(float *a, float *b); 

int main() { 
    float num1 = 0.0; 
    float num2 = 0.0; 

    float *px = NULL; 
    float *py = NULL; 

    printf("Please enter a decimal number: "); 
    scanf("%f", &num1); 
    getchar(); 

    printf("Please enter another decimal number: "); 
    scanf("%f", &num2); 

    px = &num1; 
    py = &num2; 

    printf("The numbers before swapping - \nNum1 = %f\nNum2 = %f\n", num1, num2); 

    swap(&px, &py); 

    printf("\nThe numbers after swapping - \nNum1 = %f\nNum2 = %f\n\n", num1, num2); 

    system("PAUSE"); 
    return 0; 
} 

void swap(float *a, float *b) { 
    float temp = *a; 
    *a = *b; 
    *b = temp; 
} 

Answer 1

あなたはfloat値のちょうどアドレスの代わりにfloatへのポインタのアドレスを渡します。コンパイラは、この型の不一致について警告を発しているはず

swap(&num1, &num2); 

：pxとpy変数を除去するために、あなたの呼び出しを簡素化します。これらの警告を無視しないでください。プログラミングエラーを示しています。より良い場合でも、より多くの警告を有効にする：gcc -Wall -Wextra -Werrorまたはclang -Weverythingは、潜在的なエラーについてより多くの警告を生成する。

Answer 2

プロトタイプswapをvoid swap(float** a, float** b);に変更します。

void swap(float** a, float** b) { 
    float temp = **a; 
    **a = **b; 
    **b = temp; 
}