Usando punteros para intercambiar valores de matriz int

Question

Se supone que debo usar punteros para intercambiar enteros en una matriz. Se compila sin errores ni advertencias y se ejecuta, pero no intercambia las entradas. ¡¡¡Cualquier sugerencia sería útil!!!

Aquí está el probador:

#import <stdio.h> 

void swap(int ary[]); 

int main( int argc, char*argv[]) 
{ 
    int ary[] = { 25, 50 }; 
    printf("The array values are: %i and %i \n", ary[0], ary[1]); 
    swap(ary); 
    printf("After swaping the values are: %i and %i \n", ary[0], ary[1]); 

    return 0; 
} 

Aquí es la función de intercambio:

void swap(int ary[]) 
{ 
    int temp = *ary; 
    *ary = *(ary + 1); 
    *ary = temp; 
} 

Esto es lo que se muestra después de ejecutar:

The array values are: 25 and 50 
After swaping the values are: 25 and 50 

Answer 1

Odio echar a perder esto, pero parece más un error tipográfico que nada.

En su función de intercambio:

*ary = temp; 

debería ser:

*(ary + 1) = temp; 

edición: ¿Hay alguna razón usted no está usando la notación de matriz? Creo que es un poco más claro para cosas como esta:

int temp = ary[0]; 
ary[0] = ary[1]; 
ary[1] = temp; 

Answer 2

examinar su función de intercambio con más cuidado :

void swap(int ary[]) 
{ 
    int temp = *ary; 
    *ary = *(ary + 1); 
    *ary = temp; 
} 

¿Cuándo se asigna *(ary + 1)a?

Answer 3

mueve el segundo valor en el primer lugar, y luego pasar el primer valor de nuevo en el primer lugar.

Answer 4

Tal vez puedas probar:

void swap(int ary[]) 
{ 
    int temp = ary[0]; 
    ary[0] = ary[1]; 
    ary[1] = temp; 
} 

Answer 5

sólo por diversión; También es posible intercambiar sin necesidad de utilizar un valor temporal

void swap(int ary[]) 
{ 
    *ary ^= *(ary + 1); 
    *(ary + 1) ^= *ary; 
    *ary ^= *(ary + 1); 
} 

Como GMan señala, este código oscurece su intención de que el compilador y el procesador, por lo que el rendimiento puede ser peor que el uso de una variable temporal, especialmente en una moderna UPC.

Answer 6

su función de intercambio sólo funcionará durante 2-enteros matriz, por lo que mostrarlo a su compilador (que no va a cambiar nada, pero hacer código más limpio)

void swap(int ary[2]) 

Answer 7

También puede cambiar los valores sin ningún variable temporal:

void swap(int *x, int *y) 
{ 
    *x ^= *y; 
    *y ^= *x; 
    *x ^= *y; 
} 

luego llamar a:

swap(&ary[0], &ary[1]);