Problem z odczytem pliku

Question

Nie wiem dlaczego, nie działa kod, kopiowalem nawet kod od Pana Mirosława z odc. 7 o odczytach i zapisach pliku.  Mianowicie chodzi mi o to, że plik zostaje odczytany, ale tylko pierwsze trzy linie, a dalej juz jest nieodczytywane.

#include <iostream>
#include <fstream>
#include <cstdlib>

using namespace std;

fstream plik;
string imie, nazwisko;
int nr_tel;

int main()
{
    plik.open("wizytowka.txt", ios::in);  
    if(plik.good()==false)   
    {
        cerr<<"Plik nie istnieje!";
        exit(0);
    }
    string linia;
    int nr_linii=1;

    while(getline(plik,linia))
    {
        switch(nr_linii)
        {
            case 1: imie=linia; break;
            case 2: nazwisko=linia; break;
            case 3: nr_tel=atoi(linia.c_str()); break;  

        nr_linii++;
    }

    plik.close();

    cout<<imie<<endl;
    cout<<nazwisko<<endl;
    cout<<nr_tel<<endl;




    return 0;
}

 

Comments

Dobrze kod skopiowałeś? Bo switch nie jest zamknięty.

---

Racja, źle skopiowałem, w moim kodzie switch jest zamknięty i nie działa.

Answer 1

Plik jest odczytywany od początku do końca, ale tylko pierwsze 3 linie są użyte.
Aby się o tym przekonać zmodyfikuj pętlę

    while(getline(plik,linia))
    {
        switch(nr_linii)
        {
            case 1: imie=linia; break;
            case 2: nazwisko=linia; break;
            case 3: nr_tel=atoi(linia.c_str()); break;
            default: 
                cout << "Linia nr: " << nr_linii <<" - " << linia;
        } // tutaj brakowało zamknięcia
        nr_linii++;
    }

 

Comments

Przekształciłem trochę kod i pokazuje wszystkie linie, ale trzy pierwsze linie w pliku, program pokazuje jako ostatnie. Mógłby ktoś troszeczkę pomóc zrozumieć ten kod?

#include <iostream>
#include <fstream>
#include <cstdlib>

using namespace std;

                    

fstream plik;
string imie, nazwisko;
int nr_tel;

int main()
{
    plik.open("wizytowka.txt", ios::in);   

    if(plik.good()==false)   
    {
        cerr<<"Plik nie istnieje!";
        exit(0);
    }
    string linia;
    int nr_linii=1;

   while(getline(plik,linia))
{
    switch(nr_linii)
    {
        case 1: imie=linia; break;
        case 2: nazwisko=linia; break;
        case 3: nr_tel=atoi(linia.c_str()); break;
        default:
            cout <<  linia<<endl;
    }
    nr_linii++;
}

    plik.close();

    cout<<imie<<endl;
    cout<<nazwisko<<endl;
    cout<<nr_tel<<endl;

 

---

Pierwsze 3 linie pliku zapisujesz do zmiennych.
Pierwsza linia do zmiennej imie.
Druga linia do zmiennej nazwisko.
Trzecia linia do zmiennej nr_tel.
Kolejne linie są drukowane na ekran (przypadek default).
Pod koniec programu wypisujesz na ekran wartość zmiennych imie, nazwiko i nr_tel.

Powiedz co chcesz osiągnąć i czego w tym kodzie nie rozumiesz (konkretna linia kodu)

---

Chcę, żeby pierwsze pokazały się pierwsze linie pliku.

Nie rozumiem tego switcha, przeciez nr_linii ma wartosc 1, wiec najpierw powinna zostac pobrana pierwsza linia pliku.

EDIT: Ok, przebudowałem switcha i działa, ale nadal nie rozumiem, jak program wczytuje te linie 3+.

 while(getline(plik,linia))
{
    switch(nr_linii)
    {
        case 1: imie=linia;
        {cout <<  linia<<endl; break;}
        case 2: nazwisko=linia;
        {cout <<  linia<<endl; break;}
        case 3: nr_tel=atoi(linia.c_str());
        {cout <<  linia<<endl; break;}
        default:
            cout <<  linia<<endl;
    }
    nr_linii++;
}

 

---

#include <iostream>
#include <fstream>
 
using namespace std;

int main()
{
    fstream plik;
    plik.open("wizytowka.txt", ios::in);  
    if(plik.good()==false) {
        cerr << "Plik nie istnieje!" << endl;
        return 1;
    }
    string linia; 
    while(getline(plik,linia)) {
        cout << linia << endl;
    }
    plik.close();
    return 0;
}

Ten kod pokaże wszystkie linie z pliku w odpowiedniej kolejności.

---

Hmmm to fakt, w takim razie po co Pan Mirosław w odc 7 używał funkcji switch?

---

Nie wiem. Nigdy nie oglądałem jego video-kursów.
Podejrzewam, że w ten sposób chciał przedstawić działanie instrukcji switch.

---

Switch było już przedstawione, no ale trudno, problem rozwiązany dziękuję za pomoc.

Tak z czystej ciekawości, w jaki sposób uczyłeś się programowania? 

---

Na początku kursy programowania z CD-Action. Próba zrozumienia Turbo Pascala.
Potem udało mi się znaleźć tutorial Xiona "Od zera do gier kodera". Czytanie o Windows API zupełnie mnie zniechęciło.
W liceum nauczycielka informatyki robiła mi zadania algorytmiczne.
Potem na studiach szybkie przerabianie różnych języków programowania (Turbo Pascal, C, C++, Java, D, Python). Kod który wtedy pisałem przypominał spagetti. Cały czas robiłem kopie kodu i prowadzałem drobne zmiany, kod zazwyczaj sprowadzał się do jednej długiej funkcji, przerabianej milion razy.
Tak naprawdę podstaw programowania nauczyłem się tworząc narzędzia wspomagające pracę w firmie rodziców. Najpierw pisałem w Javie. Potem odkryłem C#. Dopiero przy nim poczułem, że żyję. Bardzo spodobało mi się uzupełnianie składni dostarczane przez środowisko programistyczne.
Kolejną rewolucją było odkrycie biblioteki Qt4. Pozwoliło mi to wrócić do C++ i pisać programy okienkowe podobne do tych w C#.
To właśnie dzięki znajomości Qt znalazłem swoją pierwszą pracę, gdzie szlifowałem swoje umiejętności.
Żaden kurs czy szkolenie nie nauczy cię tak programowania jak prowadzenie jakiegoś projektu.