Pewne ćwiczenie w c++ - problem

Question

Hej, oto treść zadania:

Napisz program obliczający średnią Twoich ocen ze świadectwa(indeksu). Program ma pobierać tak długo liczby, aż podasz liczbę zero. Program ma posiadać funkcję sprawdzającą czy podany dane są liczbami, czyli ma zapobiegać uszkodzenia strumienia wejściowego. 

#include <conio.h>
#include <iostream>
#include <cctype>

using namespace std;

int main()
{
    double ocena;
    double suma=0;
    double i=0;

    do
    {
        cin>>ocena;
        if(cin.good())
        {
            suma+=ocena;
            i++;
        }
    } while(ocena != 0);
    cout<<"twoja srednia to: "<<endl;
    cout<<suma/(i-1);

    return 0;
}

Mam taki problem, nie wiem jak sobie z tym poradzić, bo jeśli wpiszę jakąś literę to cały program stopuje. Wsumie w tręści zadania jest napisane, że program ma sprawdzać czy podane dane są liczbami, a ja mam funkcję, która sprawdza czy udało się wczytać liczbę. Bardzo proszę o pomoc :)

Answer 1

Zamiast tego warunku zrób zmienną bool. Przy każdej interacji pętli przyjmuje:

• true kiedy wartość jest różna od zera, lub nie wczytano liczby
• false kiedy wczytano zero

Dodatkowo if dla !cin.good() i wtedy wyświetlasz ostrzeżenie. W pętli czyść bufor. 

cin.clear();
cin.sync();

 

Answer 2

To może tak:

1. Zamiast sprawdzać czy ze strumieniem jest wszystko dobrze, sprawdź czy strumieniowi coś się nie stało:

if(std::cin.fail()) {
  break;
}

Przerwie to pętlę i pokaże ostateczny wynik. Jeżeli chcesz, aby strumień działał dalej, to użyj funkcji clear i sync:

if(std::cin.fail()) {
  std::cin.clear();
  std::cin.sync();
}

2. Jak dla mnie najlepiej byłoby zrobić tak:

while(std::cin >> ocena && ocena != 0) {
  // Obliczenia...
}

W warunku pętli wczytujemy ocenę, jednocześnie sprawdzając czy ze strumieniem wszystko dobrze oraz sprawdzamy czy ocena jest różna od 0.

Pozdrawiam! :D

Answer 3

Z tego co kojażę pod "cin>>ocena" powinno być cin.clear() i cin.sync()

Ja tak robię i jest git :D

Answer 4

WItam ponownie, wasze wskazówki sporo mi pomogły, bo zapomniałem, o czyszczeniu flagi i bufora funkcjami clear() i sync(), ale to nie rozwiązywało problemu, bo gdy wpisywałem jakąś literę to program i tak przestawał działać. Działo się tak ze względu na ten warunek, że kiedy podam 0 to funkcja ma się zatrzymać. Widocznie podanie litery i użycie funkcji clear() oraz sync() zostawiało w strumieniu 0 i program się zatrzymywał ze względu na ten warunek. 

Zamieszczam poprawnie działający kod, może komuś się kiedyś przyda :)

#include <conio.h>
#include <iostream>
#include <cctype>

using namespace std;

    double ocena;
    double suma=0;
    double i=0;

int main()
{
   do
   {
       cin>>ocena;
       if(cin.good())
       {
            if(ocena == 0) break;
            suma+=ocena;
            i++;
       }
       cin.clear();
       cin.sync();
   }while(true);
   cout<<"Srednia wynosi: "<<endl;
   cout<<suma/i;

    return 0;
}

 

Comments

Na tym forum na serio nie liczą się punkty. Przynajmniej dla mnie, ale po znalezieniu rozwiązania dodawanie (przez siebie) nowej odpowiedzi i wybieranie ją jako najlepszą poraz kolejny zmusza mnie do wklejenia tego 

  

---

To jest kłamstwo co piszesz i chciałbym żebyś wiedział, że zrobiło mi się bardzo przykro z tego, że się ze mnie wyśmiewasz.