Pobieranie argumentów przez wskaźniki i referencje

Question

Moim zadaniem jest zamienić wartości trzech liczb miejscami. Niestety zamiana wykonuje się nie tak jak powinna. Przypisanywana jest tylko jedna wartość (ostatnia). Po wprowadzeniu 1, 2, 3 otrzymuje w odp. 3, 3 ,3. A powinno być 3 ,1 ,2.

 

int a, b, c;

void swap3_ptr(int *x, int *y, int *z)
{
    *x = c;
    *y = a;
    *z = b;
}
int main()
{
    cin >> a;
    cin >> b;
    cin >> c;

    int *wsk_a = &a;
    int *wsk_b = &b;
    int *wsk_c = &c;

    swap3_ptr(wsk_a, wsk_b, wsk_c);
        cout << a << endl;
        cout << b << endl;
        cout << c << endl;

    system("pause");
    return 0;
}

 

Answer 1

Jeśli zrobisz 

*x = c; 

to oznacza, że przepisałeś `c` do `x` a to oznacza, że przepisałeś także do zmiennej `a` 

jeśli teraz zrobisz

 *y = a;  

To będzie oznaczać, że `a` (a właściwie już zawartość `c`) przepisujesz do `y`  a to oznacza, że przepisujesz też do zmiennej `b`

Ostatecznie więc do `a` oraz `b` przepisujesz zawartość zmiennej `c`

Comments

Teraz już wiem co jest nie tak. Pojawia się nowe pytanie co zrobić, aby to zadziałało w taki sposób jaki chciałem. Dodać zmienną pomocniczą?

---

Będzie potrzebna dodatkowa zmienna ale też trzeba zwrócić uwagę na kolejność przestawień. 

W obecnym przykładzie to co masz w funkcji możesz zapisać także jako 

a = c;
b = a;
c = b;

---

ps. widzę możliwy kolejny problem - gdy przekażesz do funkcji wskaźniki w innej kolejności to może się okazać, że cała zamiana będzie wymagała innej modyfikacji.

Answer 2

Strasznie nieczytelny (jeżeli chodzi o działanie) kod. Działa prawidłowo, tj. podajesz "3 3 3" do swap3_ptr przekazujesz po kolei wskaźniki do zmiennych a, b, c, a następnie:

1. Do zmiennej a przypisujesz wartość z c, czyli 3

2. Do zmiennej b przypisujesz wartość z a, czyli 3,

3. Do zmiennej c przypisujesz wartość z b, czyli 3.

Aby zamienić miejscami te obiekty, użyj zmiennej pomocniczej, gdzie będziesz przechowywać wartość każdej liczby i po kolei przypisuj. A najlepiej to użyj tablic i std::rotate.

Answer 3

Wrzuć kod do znacznika code, tak aby był pokolorowany.