[Pomoc] Odczytywanie danych z pliku, zapis do tablicy.

[maxisst]

Cześć wszystkim, jako iż marny ze mnie programista - dopiero zaczynam (II LO) napotkałem problem w trakcie pisania programu.

Jest to sklejka różnych programów, ponieważ nie posiadam wiedzy na temat otwierania plików i wybierania z nich danych.

Program ma chyba dosyć łatwe zadanie - wczytuje dane z pliku txt (dwie kolumny ze słowami [pierwsza po ang, druga po polsku]) i zapisuje je w dwóch tablicach - nie chcę się bawić w tablice dwuwymiarowe.
Potem losuje jakieś słowo i sprawdza czy użytkownik poprawnie je przetłumaczył.

 

1. #include <fstream>
2. #include <iostream>
3. #include <conio.h>
4. #include <stdlib.h>
5. #include <time.h>
6. using namespace std;
9. int main()
10. {
11. int i=0,x=0,temp, y=0,los,los2;
12. char o;
13. string z,tlum;
14. std::fstream slowka;
15. slowka.open( "slowka.txt", std::ios::in);
16. if( slowka.good() == true )
17. {
18. slowka.seekg (0, slowka.end); // Sprawdzanie
19. x = slowka.tellg(); // Ilosci wersow
20. slowka.seekg (0, slowka.beg); // W pliku
21. }
22. else
23. cout << "Nie mozna odnalezc pliku";
24. string ang[x],pol[x];
25. while (!slowka.eof())
26. {
27. for(int i=0;i<=x;i++)
28. {
29. slowka >> ang[i] >> pol[i];
30. }
31. slowka.close();
32. }
34. for(int i=0;i<=x;i++) // <----- To wykona i się sypie
35. {
36. cout <<"\n"<<ang[i]<<" "<<pol[i];
37. }
39. srand(time(NULL));
40. cout <<"Wdus T, jesli chcesz wylosowac slowo, wdus N, jesli chcesz wyjsc";
41. cin >> o;
42. while (o="T" || "t")
43. {
44. los=rand()%x;
45. los2=rand()%2;
46. if (los2==1)
47. {
48. cout << "Podaj tlumaczenie podanego slowa: " << ang[los];
49. cin >> tlum;
50. if (tlum==pol[los])
51. cout <<"Poprawnie przetlumaczone !";
52. }
53. else
54. {
55. cout << "Podaj tlumaczenie podanego slowa: " << pol[los];
56. cin >> tlum;
57. if (tlum==ang[los])
58. cout <<"Poprawnie przetlumaczone !";
59. }
60. cout <<"Wdus T, jesli chcesz wylosowac slowo, wdus N, jesli chcesz wyjsc";
61. cin >> o;
62. }
63. getch();
64. }

​

 

​Jeśli można prosić o pomoc i wyjaśnienie co dlaczego jest złe, byłbym serdecznie wdzięczny

[maxisst]

@Refff

[Approx]

Sprawdź jak długi jest plik, bo jak rand wylosuje jakąś liczbe z kosmosu to to się wysypie, na dodatek jak porównujesz z charem to:

 

char o = 'T'
NIE char o = "T"

[maxisst]

 

Sprawdź jak długi jest plik, bo jak rand wylosuje jakąś liczbe z kosmosu to to się wysypie, na dodatek jak porównujesz z charem to:

char o = 'T'
NIE char o = "T"

 

 

 

1. {
2. slowka.seekg (0, slowka.end); // Sprawdzanie
3. x = slowka.tellg(); // Ilosci wersow
4. slowka.seekg (0, slowka.beg); // W pliku
5. }

To sprawdza długość pliku.

 

Teraz tak, program wiesza się po wypisaniu tablicy, czyli po tym momencie:

 

 

 

2. for(int i=0;i<=x;i++) // <----- To wykona i się sypie
3. {
4. cout <<"\n"<<ang[i]<<" "<<pol[i];
5. }

[Sopelek997]

Tak możesz co najwyżej uzyskać ilość znaków. Aby policzyć ilość linii musisz policzyć ilość znaków '\n' w pliku.

[maxisst]

Czyli coś w ten deseń :

string z;
while( !slowka.eof() )
      {
            getline( slowka, z );
            slowka << x<<z << "\n";
            x++;
      }

Tylko teraz pozostaje jeszcze kwestia cofnięcia się do początku pliku, z czym sobie poradzić nie mogę

[Sopelek997]

Przecież już to robiłeś wcześniej

slowka.seekg (0, slowka.beg); // W pliku

Nie wiesz co piszesz?

[maxisst]

Przecież już to robiłeś wcześniej

slowka.seekg (0, slowka.beg); // W pliku

Nie wiesz co piszesz?

 

 

 

Pozwolę sobie zacytować samego siebie :

 

Cześć wszystkim, jako iż marny ze mnie programista - dopiero zaczynam (II LO) napotkałem problem w trakcie pisania programu.

Jest to sklejka różnych programów, ponieważ nie posiadam wiedzy na temat otwierania plików i wybierania z nich danych.

 

Ale dzięki za pomoc Jak jakieś problemy po drodze napotkam, zgłoszę się jeszcze, więc nie znacie dnia ani godziny, o której znów was załamię

 

[Sopelek997]

Ale miałeś ten fragment w swoim kodzie, więc nie wymiguj się tym, że jesteś początkujący. Jak kopiujesz jakiś program to powinieneś dociec co on robi i dlaczego.

[maxisst]

Hmm, zauważyłem, gdzie jest problem - otóż program nie widzi końca pliku i nie może wyjść z pętli, pomoc znów potrzebna.

[REPREDATOR]

Powiedz mi jak rozumiesz ten kod co napisales

string z;
while( !slowka.eof() )
      {
            getline( slowka, z );
            slowka << x<<z << "\n";
            x++;
      }

[maxisst]

while( !slowka.eof() ) - wykonuje pętlę, dopóki nie znajdzie końca pliku
{
getline( slowka, z ); - pobiera z "slowka" linię i zapisuje ją w "z"
slowka << x<<z << "\n"; - tutaj mnie możecie skopać, ponieważ nie wiedziałem, a z tego co teraz patrzę to chyba dodaje nową linię w tym tekście przy każdym powtórzeniu pętli ( cout<<, a cin>> - z tego wnioskuję, czyli kierunek strzałek)
x++; - zwiększa x o 1, co każdy obieg
}

Zmieniłem kod i po usunięciu "slowka << x<<z << "\n";" program zatrzymuje się i wisi.

[REPREDATOR]

Najlepiej bedzie jak wlepisz caly kod . (oczywiscie nie ten co w temacie tylko juz ten pozmieniany

[maxisst]

#include <fstream>
#include <iostream>
#include <conio.h>
#include <stdlib.h>
#include <time.h>
using namespace std;
int i=0,x=0,temp, y=0,los,los2;
char o;
string z,tlum;

int main()
{
    
    std::fstream slowka; 
    slowka.open( "slowka.txt", std::ios::in);
    if( slowka.good() == true )
    {
    slowka.seekg (0, slowka.beg);
    while(!slowka.eof())
    {
            getline( slowka, z );
            x++;
            cout << x <<"\n";
    }
	}
	else
	{
		cout << "Nie mozna odnalezc pliku";
	}	
	slowka.seekg (0, slowka.beg);
    string ang[x],pol[x];
    while (!slowka.eof())
    	{
        	for(int i=0;i<=x;i++)
            {
                slowka >> ang[i] >> pol[i];
            }
        slowka.close();
    	}
    for(int i=0;i<=x;i++)
    {
    	cout <<"\n"<<ang[i]<<" "<<pol[i];
	}	
	srand(time(NULL));
	cout <<"Wdus T, jesli chcesz wylosowac slowo, wdus N, jesli chcesz wyjsc";
	cin >> o;
	while (o='T' || 't')
	{
		los=rand()%x;
		los2=rand()%2;
		if (los2==1)
		{
			cout << "Podaj tlumaczenie podanego slowa: " << ang[los];
			cin >> tlum;
			if (tlum==pol[los])
				cout <<"Poprawnie przetlumaczone !";
		}
		else
		{
			cout << "Podaj tlumaczenie podanego slowa: " << pol[los];
			cin >> tlum;
			if (tlum==ang[los])
				cout <<"Poprawnie przetlumaczone !";
		}
	cout <<"Wdus T, jesli chcesz wylosowac slowo, wdus N, jesli chcesz wyjsc";
	cin >> o;
	}
getch();
}

Tak więc mam coś takiego

[koelion]

MAsz prosty blad

 

for(int i=0;i<=x;i++) // TU ZMIEN NA i < X twoja tablica ma X elementow lecz liczac od zera ostatni jej eleent to x-1 wiec dajac i=x wychodzisz poza zakres tablicy
{
cout <<"\n"<<ang[i]<<" "<<pol[i];

 

 

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

 

Po 2 zmien to

while (!slowka.eof())
{
for(int i=0;i<=x;i++)
{
slowka >> ang[i] >> pol[i];
}
slowka.close();

 

 

na

int i = 0;

while (!slowka.eof())
{
slowka >> ang >> pol;
i++;
}