Getline wysypujący całego case

Question

Witam! Ostatnio chciałem sobie podsumować niektóre rzeczy z nauki i chciałem stworzyć program prosty notatnik + kalkulator w jednym w konsoli. Wszystko jest dobrze, ale napotkałem jeden błąd. Otóż kiedy chcę nadpisać plik lub dopisać tekst, używając getline wyrzuca mi całego case w switchu. Cin działa ale zależy mi na spacjach. Mógłby ktoś zerknąć na ten kod i dać mi podpowiedź na czym się skupić? Jeśli zajdzie potrzeba to mogę wysłać cały kod. Notatnik ten zapisałem w voidzie i wybieram 4 opcje. Tworzenie (działa), Wyświelanie (działa), Nadpisywanie (działa częściowo), Dopisywanie (działa częściowo). Skąd wiem, że to coś z getline? Wstawia mi poprawnie endl w pliku.

Oto ten case:

case '3':
    {
        cout << endl << "Podaj nazwe pliku: ";
        cin >> nazwa;
        nazwapliku = nazwa + ".txt";
        plik.open(nazwapliku.c_str(), ios::out);
        cout << "Napisz tekst:" << endl;
        getline(cin,tekst);
        plik << tekst << endl;
        plik.close();
        getchar();getchar();
        notatnik();
        break;
    }

 

A tak przy okazji, Mógł by też mi ktoś wytłumaczyć czemu ten case nie działa bez klamerek? Może to coś jest z tym związane.

 

Pozdrawiam. 

Comments

Case działa bez klamerek, może być problem z zasięgiem zmiennych, albo samą konstrukcją Twojego switch, to zależy...

Pokaż konstruktor fstream.

---

void notatnik()
{
    system("cls");
    char notopt;
    string nazwa;
    string nazwapliku;
    string tekst;
    fstream plik;
    cout << "========================" << endl;
    cout << "|    Menu Notatnika    |" << endl;
    cout << "========================" << endl << endl;
    cout << "[1] Utworz plik" << endl;
    cout << "[2] Wyswietl plik" << endl;
    cout << "[3] Nadpisz plik" << endl;
    cout << "[4] Dopisz do plik" << endl;
    cout << "[5] Wyjdz z programu" << endl;
    cout << "========================" << endl;
    cout << "Wybierz opcje: ";
    notopt = getch();

    switch(notopt)
    {
    case '1':
    {
        cout << endl << "Podaj nazwe pliku: ";
        cin >> nazwa;
        nazwapliku = nazwa + ".txt";
        plik.open(nazwapliku.c_str(), ios::out);
        plik.close();
        cout << "Plik txt o nazwie: " << nazwa << " zostal utworzony";
        getchar();getchar();
        notatnik();
        break;
    }
    case '2':
    {
        cout << endl << "Podaj nazwe pliku: ";
        cin >> nazwa;
        nazwapliku = nazwa + ".txt";
        plik.open(nazwapliku.c_str(), ios::in);
        if(plik.good() == false)
        {
            cerr << "Plik nieistnieje/jest uszkodzony. Brak dostepu!";
            exit(0);
        }
        while(!plik.eof())
        {
            getline(plik,tekst);
            cout << tekst << endl;
        }
        plik.close();
        getchar();getchar();
        notatnik();
        break;
    }
    case '3':
    {
        cout << endl << "Podaj nazwe pliku: ";
        cin >> nazwa;
        nazwapliku = nazwa + ".txt";
        plik.open(nazwapliku.c_str(), ios::out);
        cout << "Napisz tekst:" << endl;
        getline(cin,tekst);
        plik << tekst << endl;
        plik.close();
        getchar();getchar();
        notatnik();
        break;
    }
    case '4':
    {
        cout << endl << "Podaj nazwe pliku: ";
        cin >> nazwa;
        nazwapliku = nazwa + ".txt";
        plik.open(nazwapliku.c_str(), ios::out | ios::app);
        cout << "Napisz tekst:" << endl;
        getline(cin,tekst);
        plik << tekst << endl;
        plik.close();
        getchar();getchar();
        notatnik();
        break;
    }
    case '5':
        exit(0);
        break;
    default:
        cout << "Nie ma takiej opcji w menu!";
        notatnik();
        break;
    }
}

To ten pełen void. I przed chwilą ciekawa rzecz się zdarzyła. Odpaliłem bezpośrednio z exe, wpisuje długi test i najpierw nic a za drugim uruchomieniem automatycznie utworzyło mi plik kis.txt a w nim "tam tekst". To co wpisałem brzmiało "Jakis tam tekst"

---

Nie czyścisz bufora po getline().

---

Chyba że tak. A można się spytać jak to wyczyścić, po co to jest i dlaczego tak trzeba robić. Albo przynajmniej jak. Nie widziałem tego na kursach albo pominąłem to. Ale dziękuję za odpowiedź.

---

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

Przed każdym getlinem. Czyścisz bufor iostream'a z nadmiarowych białych znaków. 

---

Oki. Dziękuję za pomoc.