wcześniej użyty znak nowego wiersza, a funkcja "getline()"

Question

Cześć, robiłem właśnie zadanie 7. z załącznika i natrafiłem na problem. Mianowicie, po podaniu liczby aut do skatalogowania, program przeskakiwał mi opcje podania maki i przechodził od razu do roku produkcji. Było to spowodowane tym, że znak nowego wiersza zostawał w strumieniu wejściowym. Rozwiązałem ten problem, dodając linie "cin.get()" na początku pętli i teraz moje pytanie: czy można się jakoś obejść bez tego "cin.get()" lub zastąpić funkcję "getline()" inną, aby to sprawnie działało?

 

Kod programu:

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <string>

using namespace std;
struct car
{
	string brand;
	int yop;


};

int main()
{
	int n, i;
	cout << "how many cars do you want to catalogue? ";
	cin >> n;

	car *tab = new car[n];
	
	for (i = 0; i < n; i++)
	{
		cin.get();

		
		cout << endl << "car " << i + 1 << "." << endl;

		cout << "car brand: ";
		getline(cin, tab[i].brand);

		cout << "year of production: ";
		cin >> tab[i].yop;

		
	}


	for (i = 0; i < n; i++)
	{
		cout << endl << "car " << i + 1 << "." << endl;
		cout << tab[i].brand << endl << tab[i].yop << endl;

		


	}

	delete [] tab;

	_getch();
	return 0;
}

 

​

Answer 1

Metoda clear w tym przypadku nie jest potrzebna. Zamiast metody sync użyj ignore. Czemu? Ponieważ metoda sync nie służy do czyszczenia bufora a takie zachowanie występuje tylko na porcie gcc na Windowsa - MinGW. Na VS czy Linuxie kod z sync po prostu nie zadziała.

Comments

 Jeszcze dorzuciłbym coś o wyrzuceniu nieprzenośnego getch().

---

Dokladnie zamiast getch() cin.get()

---

Wiem bo sam używam Linuksa i inaczej czyszczę bufor. Ale W kodzie powyżej widać że użyto bibliotek windowsowych dlatego zaproponowałem sync.

Cytuje ze źródła cpp0x.pl

"Jeśli chcemy mieć większą kontrolę nad strumieniem wejściowym to powinniśmy czyścić jego zawartość przed każdym wczytaniem danych. Aby to zrobić musimy wywołać dwie metody strumienia std::cin. Pierwszą z nich jest std::cin.clear(), która czyści flagi błędu. Drugą metodą jest std::cin.sync(), która czyści bufor strumienia. "

 

---

Polecam opierać się na pewniejszych źródłach.

it is implementation-defined whether this function does anything with library-supplied streams. 

[źródło: http://en.cppreference.com/w/cpp/io/basic_istream/sync]

 

---

Ale W kodzie powyżej widać że użyto bibliotek windowsowych dlatego zaproponowałem sync.

Czyli według Ciebie pisanie kodu który jest nieprzenośny i zależny od implementacji jest dobre?

---

Kod autorski powyżej właśnie taki jest - nieprzenośny. Dlatego zaproponowane rozwiązanie przeze mnie jest jak najbardziej OK.

Jakby nie było kodu to bym podał inne. Nie rozumiem tego bicia piany, skoro moje rozwiązanie jest prawidłowe i autorowi rozwiązuje problem.

---

Nie rozumiem tego bicia piany, skoro moje rozwiązanie jest prawidłowe i autorowi rozwiązuje problem.

Skąd wiesz? Autor używa chyba VS, więc raczej nie.

---

 Autor używa chyba VS,

Masz szklaną kulę?

Już widzę jak ktoś kto uczy się programowania inwestuje w płatne VS.

---

Już widzę jak ktoś kto uczy się programowania inwestuje w płatne VS.

Eee, VS jest darmowe.

Masz szklaną kulę?

Nie, ale używa stdafx.

---

Autor używa chyba VS

Tak, używam VS Community 2015. Z tego co zrozumiałem, nie powinienem używać funkcji getch(), bo nie wszędzie będzie to działało i z tego samego powodu powinienem też użyć cin.ignore(), tak?

---

Tak, getch jest w conio.h a biblioteki z .h sa dla windowsa, dlatego tez ja w swoich projektach stosuje biblioteki wieloplatformowe i ty tez tak zrob. Zamiast math.h uzyj cmath ctime itd...A zamiast Sleep(); uzyj tego: _sleep() bo dziala na kazdy system a jak uzylem Sleep to cos nie chcialo zadzialac

---

biblioteki z .h sa dla windowsa

Nie, tak się dołącza biblioteki napisane w C

A zamiast Sleep(); uzyj tego: _sleep() bo dziala na kazdy system a jak uzylem Sleep to cos nie chcialo zadzialac

Nie nie zadziała. Powinien użyć this_thread::sleep_for: http://www.cplusplus.com/reference/thread/this_thread/sleep_for/

---

Dzięki za rade. Tylko mam kolejny problem, bo żeby cin.get() działało tak jak getch(), tzn żeby program kończył swoją pracę dopiero po naciśnięciu klawisza muszę przed tym wstawić  to cin.ignore(). Nie da się tego jakoś obejść?

---

Tak, nie używaj strumienia wejścia. Jedyny sposób.

---

Rozumiem. Dzięki za pomoc.

Answer 2

Wrzuć to przed getline

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