getline(); zostaje przez program pominięty jeśli jest wywoływany w metodzie C++

Question

Chodzi konkretnie o metodę description_on_start();

Proszę mi powiedzieć dlaczego program przez to przechodzi całkowicie to ignorując. Dziękuję z góry za odpowiedzi.

 

http://wklej.to/tYWzG

Answer 1

Problemem jest to, iż nie czyścisz strumienia wejścia, co przy korzystaniu z innych opcji niż std::cin czy fwrite, jest konieczne. Wystarczy, że przed std::getline() wstawisz std::cin.ignore() 

std::cout << std::endl << std::endl << "Komunikat: ";
std::cin.ignore();
std::getline(std::cin, komunikat);

 

Comments

Działa dokładnie tak jak chciałem. Dziękuję!

---

Dobry kod, ale źle to trochę wytłumaczyłeś, przynajmniej rozumując z wiedzy jaką posiadam.
getline(cin, komunikat); pobiera wszystko co siedzi w buforze klasy streambuf zawartym w obiekcie klasy IStream, cin, a jeżeli jest pusty to prosi o jakieś wejście.
W kodzie który został podany w pytaniu, bufor cina jest czymś zapełniony więc program nie musi pytać o jego uzupełnienie i pobiera go z bufora, a metoda: cin.ignore() czyści tenże bufor.
To nie ma nic wspólnego z strumienie wyjścia o którym wspomniałeś.
Przynajmniej ja to tak rozumiem :\

EDIT: getline pobiera wszystko z bufora do znaku enter. ('\n')

---

Literówka, chodziło mi oczywiście o strumień wejścia.

---

To dlaczego wspominasz o cout i printf?

---

Poprawione. To z rozpędu, zacząłem pisać o strumieniu wyjścia i zamiast std::cin napisałem std::cout :D

---

Moja metoda, która podałem jest o tyle lepsza, bo występuje od razu w funkcji w której mogą nastąpić właśnie takie problemy. Jeśli wpiszemy std::cin.ignore(); do metody void decsribtion_on_start() , a potem będziemy chcieli zamienić w funkcji int main() kolejność metod to problem się powtórzy. Dlatego tak jak podałem w swoim sposobie, lepiej czyścić strumień wejścia od razu w metodzie, która tego wymaga, a nie potem dopisywać takie kombinacje do pozostałych funkcji. Mało praktyczne i kłopotliwe.

 

Powtórzę:

void when_activate()
    {
        cout<<"O ktorej ma byc uruchomiony alarm?"<<endl;
        cout<<endl<<"\tGodzina minuta sekunda: ";
        cin>>godzina_stop>>minuta_stop>>sekunda_stop;
        getchar();
    }

Lepsze i praktyczniejsze zastosowanie czyszczenia strumienia od razu w funkcji, która tego wymaga.

---

Evelek to jest jak najbardziej logiczne, tylko że nie zastanawialiśmy się za bardzo jak działa program, tylko co wywołuje błąd. W ten sposób umieściliśmy czyszczenie bufora przed jego użyciem.

---

Zgadza się, to też jest poprawne, jednak ja podszedłem bardziej praktycznie mi się wydaje i zwróciłem uwagę na ten ważny szczegół.

---

Oczywiście :)
Tylko że można było nie zrozumieć twojego komentarza, ponieważ zamieściłeś kod bez jego wytłumaczenia.
A no i nie jestem do końca pewny czy funkcja getchar() pobiera ten char z bufora cina :?
Nawet jeżeli tak, to jest to tylko pojedynczy znak, a może się okazać tak, że w buforze zostało trochę więcej.

Znaczy to tyle, że najlepiej jest zastąpić getchar() na cin.ignore();

---

Masz rację.

Answer 2

A czy przekazujesz funkcji description_on_start jakiś argument np. 'komunikat' aby mogła wywować funkcję getline? Chyba nie bardzo. Bynajmniej tak mi się wydaje patrząc na kod, bo nie kompilowałem tego.

Comments

W innych metodach nie przekazuję żadnych argumentów i wszystko śmiga jak należy.

---

Po co ma przekazywać go jako argument, skoro jest on częścią klasy, w której jest ta metoda?

PS bynajmniej =/= przynajmniej

---

void when_activate()
    {
        cout<<"O ktorej ma byc uruchomiony alarm?"<<endl;
        cout<<endl<<"\tGodzina minuta sekunda: ";
        cin>>godzina_stop>>minuta_stop>>sekunda_stop;
        getchar();
    }

#include <stdio.h>

I po problemie. :)

---

Od razu mnie minusem zlinczowali.

---

Ok, ale w czym to ma mi pomóc?

---

Ta spacja w w funkcji when_activate powoduje niemożność użycia getline (cin, komunikat), bo zostaje w pamięci.

---

W tym pomaga, bo możesz wpisać coś do zmiennej komunikat kolego?