Utilizzo di puntatori per lo scambio di valori di array int

Question

Si suppone che utilizzi puntatori per scambiare inte in un array. Si compila senza errori o avvertimenti ed esegue ma non scambia gli interi. Qualsiasi suggerimento sarebbe utile !!!

Ecco il tester:

#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; 
} 

Ecco la funzione swap:

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

Questo è ciò che viene visualizzato dopo l'esecuzione:

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

Answer 1

Odio rovinare questo, ma sembra un errore di battitura più di ogni altra cosa.

Nella funzione di scambio:

*ary = temp; 

dovrebbe essere:

*(ary + 1) = temp; 

edit: C'è una ragione non si sta usando la notazione di matrice? Penso che sia un po 'più chiara per cose come questa:

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

Answer 2

esaminare la vostra funzione di scambio con più attenzione :

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

Quando si fa *(ary + 1) vengono assegnati -?

Answer 3

si sposta il secondo valore nel primo posto, e quindi spostare il primo valore di nuovo nel primo spot.

Answer 4

Prova a modificare:

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

Answer 5

solo per divertimento; E 'anche possibile scambiare senza utilizzare un valore temporaneo

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

Come Gman sottolinea, questo codice oscura il vostro intento dal compilatore e il processore, per cui le prestazioni possono essere peggio di utilizzare una variabile temporanea, in particolare su un moderno PROCESSORE.

Answer 6

la funzione di scambio funziona solo per 2-int array, in modo da mostrare al vostro compilatore (che non cambierà nulla, ma fare più pulito il codice)

void swap(int ary[2]) 

Answer 7

È possibile anche scambiare i valori senza alcuna variabile temporanea:

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

quindi chiamare:

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