Co jest nie tak w tym kodzie?

Question

Witam, mam problem ze zrobieniem programu który został przedstawiony w algorytmie krokowym. Po wprowadzeniu n nic się nie dzieje.

Krok 0: wczytaj N

Krok 1: wyn := 0; d := 2;

Krok 2: Dopóki d ≤ (N div 2) wykonuj kroki 2.1 i 2.2;

Krok 2.1: Jeżeli N mod d = 0, to wyn := wyn +1;

Krok 2.2: d := d + 1;

Krok 3: wypisz na ekranie wyn

#include <cstdlib>
#include <iostream>

using namespace std;

int main(int argc, char *argv[])
{
    int wyn, d, n;
    
    cout<<"Podaj wartość N= ";
    cin>>n;
    
    wyn=0;
    d=2;

    do
    {
    	if(n%d==0)
		{
      		wyn=wyn++;
        }
	d=d++;
	}
    while(d<=n/2);
    
    cout<<"wyn= "<<wyn;
	cout<<endl<<"d= "<<d<<endl;

    system("PAUSE");
    return EXIT_SUCCESS;
}

 

Answer 1

d=d++;

Tutaj masz ciekawą konstrukcję, która uniemożliwia opuszczenie pętli.

Masz tutaj zastosowane dwa operatory - operator przypisania i operator postinkrementacji. Najpierw wykona się ten drugi - czyli zostanie zwiększona wartość 'd' o jeden, po czym zostanie zwrócona wartość sprzed inkrementacji (tak właśnie działa postinkrementacja).

Następnie zadziała operator przypisania i wartość sprzed inkrementacji, zwrócona z operatora ++, zostanie przypisana do zmiennej 'd'.

Jeśli pozbędziesz się operatora przypisania, pętla ruszy z miejsca. :)

Comments

Dziękuję.

---

@draghan, poza tym wynik działania jest niezdefiniowany ze względu na punkt sekwencyjny: https://en.wikipedia.org/wiki/Sequence_point

---

W C++17 wyrażenie jest 'well defined' [paragraf "Rules", punkt 20].

(tutaj kolega pisze, że C++11 [wzwyż] nie definiuje punktów sekwencyjnych)

---

Ok.. spoko...

Answer 2

Niepotrzebnie użyłeś do..while zamiast while.
Druga sprawa:
 

d=d++;

Wiesz co tutaj robisz? Równie dobrze mógłbyś napisać `d=d;`. d++ zwiększa wartość `d` o 1, ale wynikiem wyrażenia d++ jest pierwotna wartość d. Zatem przypisujesz wartość d do d. Samo d++ lub ++d zrobi to co chcesz. To samo w linii 20. Nie jestem nawet pewien czy to co teraz robisz nie jest UB.

"Nic się nie dzieje", bo wpadasz w nieskończoną pętlę.

Comments

Dziękuję.