[C++] [vector] [wysypywanie sie]

Question

Hej,
napisałem program, który ma wczytać serię liczb i zapisać je w wektorze.Następnie, gdy użytkownik skończy wpisywanie liczb, program pyta, ile z nich ma zsumować (zaczynając od początku wektora ofc).

Problem polega na tym, że program w ogóle nie czaka na cina, tylko od razu wykonuje się pętla. Dlaczego? 

//zapisuje podane przez użytownika liczby do wektora tab, a następnie wypisuje sumę n pierwszych z nich

#include <iostream>
#include <vector>


using namespace std;

int main()
{
    vector <double> tab;
    double liczba, suma=0;
    int ile;

    cout<<"Witaj, podaj liczby: "<<endl;

        while(cin>>liczba) tab.push_back(liczba);

    cout<<"Podaj ile liczb wyswietlic: "<<endl;
    cin>>ile;

    for (int i=0; i<tab[ile]; i++) {suma+=tab[i];}
    cout<<"Suma wynosi: "<<suma<<endl;

return 0;
}

 

Comments

Tzn. od razu wykonuje się for z 22. linii?

Btw. dlaczego warunkiem fora jest i<tab[ile] ?

---

Co ma zakończyć wprowadzanie danych ?

Answer 1

A nie tak przypadkiem powinno być:

cout << "Podaj ile liczb: \n";
cin >> ile;

cout <<"Podaj liczby: \n";

while(ile-- && cin >> liczba) tab.push_back(liczba);

czyli najpierw podajesz, ile liczb chcesz wprowadzić, a później podajesz liczby.

 

Można też tak:

cout <<"Podaj liczby (oddzielone spacja): ";
    
string s;
    
if(getline(cin, s))
{
     istringstream iss(move(s));
     while(iss >> liczba) tab.push_back(liczba);
}

 

PS. warunek w for poprawny nie jest...

Comments

No wydaje mi się, że nie.
Na tym polega zaleta wektora, że nie trzeba od początku podawać ile będzie elementów (jak w tablicy), tylko przerywa się wczytywanie w odpowiednim momencie, wklepaniem elementu innego niż deklarowany. Tutaj wektor ma zawierać double, więc wystarczy wpisać literę i już.
To chyba działa jak należy.

Jakaś wskazówka dot. tej pętli? 

---

Po wpisaniu litery strumień wchodzi w stan błędu i dlatego nie czeka na ile. Wyczyść flagę błędu (clear()) i usuń zalegające znaki w linii (ignore()).

---

Ok, chyba mamy to! 
Oczywiście poprawiłem też pętlę i dodałem małą kontrolę błędów. Teraz wygląda tak:

//zapisuje podane przez u¿ytownika liczby do wektora tab, a nastêpnie wypisuje sumê n pierwszych z nich

#include <iostream>
#include <vector>


using namespace std;

int main()
{
    vector <double> tab;
    double liczba, suma=0;
    int ile;

    cout<<"Witaj, podaj liczby: "<<endl;

        while(cin>>liczba) tab.push_back(liczba);

        cin.clear();
        cin.ignore();

    cout<<"Podaj ile liczb wyswietlic: "<<endl;
    cin>>ile;

    if (ile>tab.size())                                                  //sprawdzenie zakresu
    {cerr<<"Wektor nie zawiera tylu liczb!";
    return 0;}

    for (int i=0; i<ile; i++) {suma+=tab[i];}
    cout<<"Suma wynosi: "<<suma<<endl;

return 0;
}

 

---

cin.ignore(numeric_limits<streamsize>::max(), '\n');

Tak daj, będziesz miał pewność, że wszystko, co użytkownik wpisał, zostało usunięte i że cin będzie czekać na podanie ile.