Wczytywanie z pliku txt

Wczytywanie z pliku txt
Krzysztof Miller
  • Rejestracja:prawie 5 lat
  • Ostatnio:ponad 4 lata
  • Postów:10
0

Mam taką funkcję która ma wczytać linie z pliku, ale jeśli odpalę ją w pętli to zaczyna czytać od początku a nie od miejsca gdzie skończyła. Funkcja by miała wczytać plik potem wypisać w konsoli co wczytała i znowu wczytać plik ale za drugim razem wczytuje od początku. Da się jakoś zrobić żeby ominąć pierwsze 6 linijek w pliku, potem 12 itd czy jakoś tak? Plik wygląda tak:

Kopiuj
1. Ile to 2+2
A. 1
B. 2
C. 3
D. 4
d
1. Ile to 2-2
A. 0
B. 2
C. 3
D. 4
a
1. Ile to 2*2
A. 0
B. 2
C. 3
D. 4
d

itd.

Zależy mi żeby funkcja czytała pojedynczo każde pytanie, 4 odpowiedzi i poprawną a nie cały plik na raz bo tak to wiem jak zrobić. Dzięki za pomoc.

Kopiuj
nr_linii=1;
void Pytanie::wczytaj()
{
    fstream plik;
    string linia;
    plik.open("quiz.txt", ios::in);

    if(plik.good()==false)
    {
        cout<<"Nie ma pliku";
        exit(0);
    }

    while(getline(plik,linia))
    {
        switch(nr_linii)
        {
        case 1: tresc=linia; break;
        case 2: a=linia; break;
        case 3: b=linia; break;
        case 4: c=linia; break;
        case 5: d=linia; break;
        case 6: poprawna=linia; break;
        }
    nr_linii++;
    if(nr_linii==7) {nr_linii=1; break;}
    }
    plik.close();
}

/////////////////////////////////////////////////

void Pytanie::zadaj()
{
    cout<<tresc<<endl<<endl;
    cout<<a<<endl;
    cout<<b<<endl;
    cout<<c<<endl;
    cout<<d<<endl;
    cin>>odp;
}
edytowany 1x, ostatnio: cerrato
_13th_Dragon
  • Rejestracja:prawie 20 lat
  • Ostatnio:20 dni
3
Kopiuj
void Pytanie::wczytaj()
{
    fstream plik("quiz.txt", ios::in);
    if(!plik)
    {
        cout<<"Nie ma pliku"<<endl;
        exit(0);
    }
    string question,answerA,answerB,answerC,answerD,,good
    while(getline(getline(getline(getline(getline(getline(plik>>ws,question)>>ws,answerA)>>ws,answerB)>>ws,answerC)>>ws,answerD)>>ws,good))
    {
        cout<<question<<endl<<endl<<answerA<<endl<<answerb<<endl<<answerC<<endl<<answerD<<endl<<"Wybierz: ";
        string answer;
        if(getline(cin>>ws,answer)&&(answer==good) cout<<"Poprawnie"<<endl;
        else cout<<"Niepoprawnie"<<endl;
    }
    //plik.close(); //nie trzeba sam się zamknie
}

Wykonuję programy na zamówienie, pisać na Priv.
Asm/C/C++/Pascal/Delphi/Java/C#/PHP/JS oraz inne języki.
ZK
z tymi "getline" to zakładasz. że zawsze będzie to 6 wierszy ? A co jeśli ilość wierszy przybędzie ?
PerlMonk
Ale zaszalałeś z tym getline :D
Sunnydev
xDDdD piękne.
_13th_Dragon
@zkubinski, przy takim formacie pliku musi być określona ilość, ponieważ jeżeli wierszy nie będzie równo X to nie wiadomo gdzie się kończy jedno pytanie zaś zaczyna się kolejne. W sumie format pliku trzeba zmienić na: 4@4@1@2@3@4@Ile to 2+2<br/> 4@1@0@2@3@4@Ile to 2-2<br/> 4@4@0@2@3@4@Ile to 2*2<br/> gdzie pierwsza liczba to Ilość odpowiedzi; druga liczba to odpowiedź poprawna; dalej odpowiedzi tyle ile wskazuje pierwsza liczba; na końcu samo pytanie.
ZK
  • Rejestracja:około 19 lat
  • Ostatnio:4 miesiące
0

przyznam, że nie robiłem wczytywania pliku od miejsca w którym program zakończył czytanie (czy to z powodów wysypania się programu czy to przez przerwanie wczytywania na żądanie użytkownika czy to z założenia, że program ma na wstępie wczytać "dowolną ilość" linii.

Osobiście bym to zrobił tak (kroki mogą być bardzo ogólne gdyż piszę to na szybko)

Załóżmy, że założyłem sobie, że program po wskazaniu pliku do otwarcia wczytuje dowolną ilość linii które oddzielone są jakimś znakiem specjalnym, ponieważ nie jestem jednoznacznie stwierdzić czy rzeczywiście zawsze będę wczytywał 5 lub 10 linii (znak specjalny może być to zwykły "enter" po którym nie ma żadnych innych znaków

  1. Wczytuję plik i program wczytuje "dowolną ilość" linii i przerywa wczytywanie w miejscu gdzie ustali użytkownik
  2. Tam gdzie przerwało wczytywanie linii, ilość stanu wczytanych linii zapisuję do pliku (np wczytałem 15 linii to ich stan zapisuję do pliku jako liczba 15)
  3. otwieram ponowne ten sam plik i program wczytuje plik do wczytania i plik "ze stanem linii" i zaczyna wczytywanie od momentu w którym poprzednio wczytywanie zostało przerwane

żadne tam kombinowanie z milionami getlinów, program po prostu musi jakoś pamiętać "ostatni stan" na którym skończył - lepszego sposobu na "zapisanie stanu" nie znam


bla
edytowany 2x, ostatnio: zkubinski
_13th_Dragon
Czy nie prościej nie zamykać pliku?
ZK
teoretycznie pomysł jest dobry - ale co jeśli użytkownik program zamknie lub się wysypie ? Informację o "ostatniej przetwarzanej linii" zostanie utracona
_13th_Dragon
Nawet word nie zapamiętuje zmian po każdym ruchu.
ZK
Widocznie odbywa się to w pamięci i co jakiś czas i tak jest zrzut danych do pliku bo jak program się wykrzaczy to w wordzie odzyskujesz ostatnie zapamiętane dane z pliku który jest ukryty. Czas autozapisu sobie sam użytkownik ustawia
enedil
A co jeśli plik się zmieni w trakcie i numer linijki stanie się bezwartościowy? Może zamiast wymyślać takie usecasy, jak ten mój i ten twój, trzeba się zapytać OPa czego potrzebuje.
enedil
  • Rejestracja:prawie 12 lat
  • Ostatnio:3 dni
  • Postów:1027
1

A nie myślałeś, żeby po prostu uchwyt na plik był częścią klasy? Wówczas to "trzymanie pozycji" masz za darmo.

ZK
tzn jak za darmo ? Możesz jaśniej opisać co masz na myśli ?
enedil
Wystarczy nie zamykać pliku
PaulGilbert
  • Rejestracja:około 7 lat
  • Ostatnio:minuta
  • Postów:932
0

Nie lepiej do tego użyć bazy danych? Jeśli chcesz koniecznie wybierać pytania z pliku tekstowego, to może stwórz w pliku przed każdym pytaniem jakieś wyrażenie regularne połączone z numeracją, oznaczając w ten sposób kolejne pytania i sobie będziesz przeszukiwał plik dowolnie z niego wypisując fragmenty które chcesz, posługując się np. numeracją pytań. Nie wiem czy wystarczająco jasno napisałem co mam na myśli?

Zobacz pozostałe 4 komentarze
MarekR22
Naprawdę pan Z uczy robić aż takie kwiatki?
vpiotr
if (plik.good() == false) - mam do tego odpowiedni podkład muzyczny: https://www.youtube.com/watch?v=teY7V8AqPu4
KHX
Te komentarze pod jego filmami są ciekawe, niby wiele osób się nauczyło programować, nawet w 2020, co z tego że w tym roku dużo kodu z filmiku nie działa. Ale, żeby nie było trzeba docenić za staranie,,podstaw podstaw" się tam można nauczyć, ale nie żeby zostać programistą.
MarekR22
Dla początkujących te filmiki wydają się świetne, bo po prostu nie mają dostatecznej wiedzy, by zrozumieć, iż są technicznie niepoprawne i kierują na manowce. Ludzie, co coś umieją, nie oglądają tych filmów, więc ich nie oceniają, więc komentarze to w zasadzie same zachwyty.
Krzysztof Miller
  • Rejestracja:prawie 5 lat
  • Ostatnio:ponad 4 lata
  • Postów:10
0

Zrobiłem na trochę pokrętny sposób może kiedyś jakiemuś początkującemu się przyda co robi pasje inf:
W skrócie przy 2 pytaniu odejmuje 6 od switcha i switch musi przejść przez te pierwsze 6 lini i nic z nimi nie robi, przy trzecim pytaniu odejmuje 12 itd. (chyba)

Kopiuj
//main.cpp
#include <iostream>
#include "pytanie.h"

using namespace std;

int main()
{
    float suma=0;
    Pytanie p[5];
    for(int i=0;i<5;i++)
    {
        p[i].nr_pyt=i;
        p[i].wczytaj();
        p[i].zadaj();
        p[i].sprawdz();
        suma+=p[i].pkt;
        cout<<endl<<"Twoje punkty wynosza: "<<suma<<"/5";
        cout<<" A procent poprawnych odpowiedzi wynosi: "<<(suma/(i+1))*100<<"%"<<endl<<endl;
    }
    cout<<endl<<endl<<"KONIEC QUIZU"<<endl;

    return 0;
}
Kopiuj
//pytanie.cpp
#include <iostream>
#include "pytanie.h"
#include <fstream>


using namespace std;

void Pytanie::wczytaj()
{
    fstream plik;
    string linia;
    plik.open("quiz.txt", ios::in);

    if(plik.good()==false)
    {
        cout<<"Nie ma pliku";
        exit(0);
    }
    int aktualny_nr=1;
    while(getline(plik,linia))
    {
        switch(aktualny_nr-(nr_pyt)*6)
        {
        case 1:
            tresc=linia;
            break;
        case 2:
            a=linia;
            break;
        case 3:
            b=linia;
            break;
        case 4:
            c=linia;
            break;
        case 5:
            d=linia;
            break;
        case 6:
            poprawna=linia;
            break;
        }
        aktualny_nr++;


    }
    plik.close();
}
void Pytanie::zadaj()
{
    cout<<tresc<<endl<<endl;
    cout<<a<<endl;
    cout<<b<<endl;
    cout<<c<<endl;
    cout<<d<<endl;
    cin>>odp;
}

void Pytanie::sprawdz()
{
    if(odp==poprawna)
    {
        cout<<endl<<"Dobrze"<<endl<<endl;
        pkt=1;
    }
    else
    {
        cout<<endl<<"Zle"<<endl<<endl;
    }
}
Kopiuj
//pytanie.h
#include <iostream>

using namespace std;

class Pytanie
{
    public:

    string tresc;
    string a, b, c, d;
    string odp, poprawna;
    int nr_pyt=0, nr_linii, pkt=0, suma=0;

    void wczytaj(); //wczytuje dane z pliku
    void zadaj(); //zadaje pytanie
    void sprawdz(); //sprawdza

};
edytowany 1x, ostatnio: Krzysztof Miller
06
  • Rejestracja:prawie 20 lat
  • Ostatnio:około rok
  • Postów:2440
2

Zrobiłem na trochę pokrętny sposób

A nie lepiej zrobić to normalnie, bez switcha, otwierając strumień tylko raz?

Kopiuj
std::istream& Pytanie::wczytaj(std::istream& is)
{
	std::getline(is, tresc);
	std::getline(is, a);
	std::getline(is, b);
	std::getline(is, c);
	std::getline(is, d);
	std::getline(is, poprawna);
	return is;
}

...

std::vector<Pytanie> pytania;

std::ifstream plik("quiz.txt");
Pytanie pytanie;

while(pytanie.wczytaj(plik))
	pytania.push_back(std::move(pytanie));
MarekR22
Moderator C/C++
  • Rejestracja:ponad 17 lat
  • Ostatnio:około 2 godziny
3

Ja bym poprawił tak:

Kopiuj
std::istream& Pytanie::wczytaj(std::istream& is)
{
    std::getline(is, tresc);
    std::string odpowiedz;
    while (std::getline(is, odpowiedz) && !odpowiedz.empty()) {
         odpowiedzi.push_back(odpowiedz);
    }
    return is;
}

Gdzie std::vector<string> odpowiedzi; to pole klasy, a pierwszy element jest prawidłową odpowiedzią.
Podczas pokazywania pytania odpowiedzi wyświetlałbym w losowej kolejności. Odpowiedzi może być więcej niż jest potrzebne.
Łatwiej wtedy utrzymuje się dane wejściowe.


Jeśli chcesz pomocy, NIE pisz na priva, ale zadaj dobre pytanie na forum.
nalik
  • Rejestracja:ponad 9 lat
  • Ostatnio:około miesiąc
  • Postów:1039
2
Krzysztof Miller napisał(a):

Mam taką funkcję która ma wczytać linie z pliku, ale jeśli odpalę ją w pętli to zaczyna czytać od początku a nie od miejsca gdzie skończyła. Funkcja by miała wczytać plik potem wypisać w konsoli co wczytała i znowu wczytać plik ale za drugim razem wczytuje od początku. Da się jakoś zrobić żeby ominąć pierwsze 6 linijek w pliku, potem 12 itd czy jakoś tak?

Pomysł chybiony. Wejście/wyjście jest z natury strumieniowe. A otwieranie tego samego pliku za każdym razem to marnotrawstwo.

Design też nie taki jak trzeba. Klasa nie musi mieć zakodowanej na stałe nazwy pliku, a funkcja wczytaj nie musi tworzyć strumienia. Twoja funkcja równie dobrze może przejmować strumień poprzez argument funkcji, ewentualnie argument konstruktora klasy. W ten sposób rozdzielasz odpowiedzialność tworzenie (otwierania) strumienia (pliku) od jego konsumpcji. Postaraj się nie tworzyć klas, które robią za dużo. Zasada pojedynczej odpowiedzialności.

A czym jest strumień - plikiem, standardowym wejście, czy stringiem z testu, jest bez znaczenia, ważne, że dostarcza on mechanizm pozwalający Ci odczytać dane. Zobacz kod @MarekR22 . Tak to powinno wyglądać.

edytowany 4x, ostatnio: nalik

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.