Sumowanie liczb podanych na wejściu

Question

Cześć,

Ma kolejny problem z pętlą for (dlaczego nie potrafię jej ogarnąć?!), więc na początek pierwsze pytanie - ktoś ma jakiś fajny artykuł/filmik/obrazek, który w łopatologiczny sposób tłumaczy zastosowanie pętli for? Drugie - gdy wprowadzam liczby w niżej podanym kodzie, to program w konsoli wiesi się. Możecie mnie poinstruować dlaczego?

Pozdrawiam!
 

#include <iostream>

using namespace std;

int suma = 0;
char liczba;

int main()
{
    cout << "Podawaj cyfry oddzielone spacja, a podlicze Ci ich sume: ";
    cin >> liczba;

    while (liczba != '0')
    {
        cin >> liczba;

        for (int i = 0; liczba != '0'; i++)
            suma += liczba;
    }
    cout << "Suma " << suma;
    return 0;
}

 

Answer 1

Tworzysz nieskończoną pętlę for, zły warunek. liczba!=0 oznacza, że pętla wykywać się będzie cały czas ponieważ jak podasz np  w zmiennej liczba 1 to dosłownie to  brzmi dla for.

"Przyjmij  dla i=0  pętlę wykonuj dopóki liczba jest inna od 0 (no a jest bo jest 1 czyli zapętlenie) inkrementuj pętlę."

Po drugie wynik nigdy w takim wypadku ci nie wyjdzie poprawny, bo char zostanie skonwertowany do Ascii i już dla samego warunku pętli do while dostaniesz 48. A dla np 1 i 0  dostaniesz 97, bo znak 0 w ascii to 48 w systemie dziesiątkowym a 1 to 49 w dziesiątkowym->wikipedia.

kolejna sprawa, podawanie zmiennej  na początku by tylko zainicjować pętlę while jest głupotą. Do tego masz pętlę specjalnie do while która wykona się zawsze przynajmniej raz.

 

 

#include <iostream>
#include <string.h>
/*Program: zadanie domowe nr 1 z lekcji: Pêtle do ... while. */
using namespace std;

int suma = 0;
char  liczba; //do zmiany

int main()
{
    cout << "Podawaj cyfry oddzielone spacja, a podlicze Ci ich sume: ";
   do

    {
        cin >> liczba;

      //  for (int i = 0; liczba != '0'; i++) // blad
            suma += liczba;
    }
     while (liczba != '0'); błąd
    cout << "Suma " << suma; // błędny wynik

    return 0;

porpaw błędy

Comments

Nic nie chcę mówić, ale nie lepiej, żeby kolega spróbował sam poprawić kod - więcej zapamięta 

---

Fakt, trochę pobrudziłem ten kod :)

---

Tylko w zadaniu jeszcze mam warunek, że jak użytkownik wprowadzi cyfrę 0, to kończy sumowanie.

---

No to przecież napisałem ci jak masz to zrobić. Zmień typ, albo w ascii znajdź odpowiednik 0 w heksadecymalnym  systemie.

Answer 2

int suma = 0;
char liczba; // radzilbym pobierac jako typ liczbowy (np. int)
 
int main()
{
    cout << "Podawaj cyfry oddzielone spacja, a podlicze Ci ich sume: ";
    cin >> liczba;
 
    while (liczba != '0') /*wtedy nie musisz się bawić w porownywanie znaków. 
                         Dajesz w tej pętli cin >> liczba, 
                         co oznacza "wykonuj dopóki (cin>> liczba) zwraca true,
                         czyli dopóki podaje się dobre dane. */
    {
        cin >> liczba;
 
        for (int i = 0; liczba != '0'; i++)
            suma += liczba; // Chcesz dodać do int znak '0, zamiast liczby-znak '0' == 48
    }
    cout << "Suma " << suma;
    return 0;
}

Spróbuj poprawić kod, jeśli Ci się nie uda, podrzucę Ci do przeanalizowania kawałek kodu 

Answer 3

Problem rozwiązany, dzięki Panowie. :)

Druga sprawa - jakieś fajne rzeczy do nauki pętli i instrukcji warunkowych?

Comments

W google jest tego mnóstwo, na YT to samo, nie ma dobrych.złych tutoriali do tak prostych rzeczy.