Pętla while do podanej liczby

Question

Witam.
Głowię się nad tym kodem i nie wiem jak to ugryźć, a poprzednie próby kończyły się niezamierzonym wynikiem. Muszę zrobić tak, żeby pętla while sprawdzała czy podane liczby są doskonałe, dopóki nie otrzymam ich podanej wartości. Szukałem na różnych stronach i nadal nie mam zielonego pojęcia jak to zrobić. Kod tutaj:

 

#include<iostream>
#include<cmath>
using namespace std;
bool czy_doskonala(int n)
{
int s = 1, p = sqrt(n);
for(int i=2; i<=p; i++)
if(n%i == 0)
s+= i + n/i;
if(n == p*p) s-=p;
if(n == s) return 1;
return 0;	
}
int main()
{
int i,n;
system("chcp 1250 > nul");
cout<<"Podaj ile liczb doskonałych chcesz uzyskać"<<endl;
cout<<"Ile = ";
cin>>i;
while(!czy_doskonala(n)){	
cout<<"Podaj liczbę n: "<<endl;
cout<<"n = ";
cin>>n;
if(czy_doskonala(n))
cout<<"Liczba "<<n<<" jest doskonała.\n";
else
cout<<"Liczba "<<n<<" nie jest doskonała.\n";
}
cout<<"Ilość liczb doskonałych: "<<czy_doskonala(n)<<endl;
return 0;
}

 

Comments

Trochę bałagan masz w tym kodzie. Po co pobierasz i od usera skoro nic z nim nie robisz? Po co tyle razy sprawdzasz czy liczba jest doskonała? Dlaczego za ilość liczb doskonałych bierzesz wartość logiczną? Czy pętla ma kończyć sę w momencie w którym wczytasz pierwszą doskonałą?

Answer 1

Trochę bez sensu. Po co warunek

while(!czy_doskonala(n))

Jeśli dobrze rozumiem, chcesz sprawdzić każdą z podanych liczb. Jednakże Ty sprawdzasz w warunku, czy liczba, która już została sprawdzona jest doskonała. I jeśli nie jest, to kontynuujesz. Ale jeśli jest doskonała, to wychodzisz z pętli nie dając nawet możliwości sprawdzenia innych liczb. Ponadto w Twoim programie można wpisywać tyle liczb, ile się chce, o ile nie są doskonałe. 
Ja bym zamienił to na: 
 

while(i > 0) // lub po prostu na while(i)
{
Twoje instrukcje;
i--;
}

W dodatku widzę, że na koniec podajesz ilość liczb doskonałych odwołując się do funkcji sprawdzającej, czy podana liczba jest doskonała. To nie ma najmniejszego sensu. 
Zainicjalizuj zmienną odpowiadającą za trzymanie informacji o ilości liczb doskonałych i pod koniec ją wyświetl. 

Comments

Jednakże Ty sprawdzasz w warunku, czy liczba, która już została sprawdzona jest doskonała. I jeśli nie jest, to kontynuujesz.

Właśnie o to chodzi w tym programie. I ma to się zapętlać. Dajmy na to, że chcę, żeby mi wyświetliło 3 liczby, które są doskonałe. Określam, że chcę trzy liczby. Wpiszę dwie, które są doskonałe i jedną, która nie jest. Powinno mi się to wtedy zapętlić do momentu, aż nie wpiszę kolejnej liczby doskonałej i wtedy program powinien zakończyć działanie.

---

Okej, nie doczytałem. Tylko w takim razie po co wypisywać na koniec ile liczb doskonałych wpisał użytkownik, skoro już na początku podajemy ile ich chcemy? 
Jeśli chcesz powiedzmy dokładnie 'i' liczb doskonałych, to wystarczy zmienną 'i' zmniejszać w tym if'ie, w którym wypisujesz, że liczba jest doskonała. 

while(i > 0)
{
coś tam coś tam;
if(czy_doskonala(n))
{
    cout<<"Liczba "<<n<<" jest doskonala.\n";
    i--;
}
coś tam coś tam
}

Wtedy tak jakby będziemy odhaczać ile tych liczb doskonałych już otrzymaliśmy. 

---

Dzięki wielkie. W końcu zadziałało.