Czmu wywala mi program?

Question

Napisałem coś takiego:

#include <iostream>
#include <fstream>

int SeeFile(std::string name,int line)
{
    std::fstream file;
    std::string iss[line];
    int asd=1;
    int buf;
    file.open(name.c_str(), std::ios::in | std::ios::out);
    if(file.good()==false)return 5;
    for(int i=line; i>0; i--)
    {
        getline(file, iss[asd]);
        asd++;
    }
    buf=asd;
    for(int j=asd; j>0; j--)
    {
        std::cout << iss[buf-j+2] << std::endl;
    }
    return 0;
}

int main()
{
    SeeFile("Nic.txt", 10);
    return 0;
}

ale potem zauważyłem, że nie działa(wywala program bez powodu.) Dla ułatwienia napisałem komentarze, aby łatwiej wam było mi pomóc

 

#include <iostream>
#include <fstream>

int SeeFile(std::string name/** Nazwa pliku*/,int line/**Ile linii wczytac*/)
{
    std::fstream file;/** Plik*/
    std::string iss[line];/**Napis z linii*/
    int asd=1;/**Numer linii z napisem*/
    int buf;/**Potrzeba zapamietac te wartosc (parz linia 20)*/
    file.open(name.c_str(), std::ios::in | std::ios::out);/**Otwieranie pliku*/
    if(file.good()==false)return 5;/**Sprawdzanie czy plik istnieje*/
    for(int i=line; i>0; i--)/**Petla wczytujaca linie (od int line(patrz 4 linijka))*/
    {
        getline(file, iss[asd]);/**Wczytywanie napisu*/
        asd++;/**Kolejna linijka*/
    }
    buf=asd;/**Potrzebne do tablicy(patrz 20 linijka)*/
    for(int j=asd; j>0; j--)/** Petla wypisujaca liczby*/
    {
        std::cout << iss[buf-j+2] << std::endl;/**tu wypisuja sie liczby*/
    }
    return 0;/**Konczenie funkcji*/
}

int main() /**Tu sie wszystko dzieje*/
{
    SeeFile("Nic.txt", 10);/** wywolywanie funkcji*/
    return 0; /** Konczenie programu*/
}

Czemu mnie wywala zamiast wypisać te 10 linijek?

PS. zawartość pliku to:

 

Lubie
Placki
1
2
3
4
5
6
7
Nie ma 8 :(

Answer 1

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <stdio.h>
#include <windows.h>


using namespace std;

int main()
{

    fstream plik;

    plik.open("dane.txt", ios::in);

    if(plik.good()==false)
    {
        cerr<<"Nie ma pliku!";
        return 0;
    }

    int x=1;
    string y;

    while(getline(plik,y))
    {
    x++;
    }
    plik.close();

    plik.open("dane.txt", ios::in);

    string linia;
    string tresc[x];
    int nr_linii=0;
    while(getline(plik, linia))
    {
    tresc[nr_linii]=linia;
    nr_linii++;
    }
    //nr_linii--;

    cin>>x;
    if(x>nr_linii)
    {
        cerr<<"Nie ma tyle linii";
        return 0;
    }
    for(int i=0;i<=x;i++)
    {
      cout<<tresc[i]<<endl;

    }

   plik.close();

   getchar();



    return 0;

}

Chodzi o cos takiego?

Comments

Tak, dokładnie to. Tylko tam masz błąd w pętli

for(int i=0;i<=x;i++)
    {
      cout<<tresc[i]<<endl;
 
    }

a powinno być

for(int i=0;i<x;i++)
    {
      cout<<tresc[i]<<endl;
 
    }

 

Answer 2

Co rozumiesz przez wyrażenie "wywala", btw twój kod jest strasznie nieczytelny i moim zdaniem jest niepotrzebnie skomplikowany, jeżeli chodzi o wczytywanie kompletnych linii to da się to załatwić o wiele czytelniej.

Comments

Dasz przykład kodu, który działa tak samo?

PS. Czemu nie czytelny?

---

Jeżeli będzie działał tak samo to też nie będzie działał, raczej chodzi ci o program, który realizuje określone zadanie (wnioskując po nazwie funkcji to ma wyświetlić zawartość pliku, z czego maksymalnie określoną ilość linii? Coś w rodzaju head z Linuxa?

---

Tak, piszesz nazwie pliku (SeeFile(Nazwa, ile linii)), a potem ile linii odczytać (SeeFile(Nazwa, ile linii)). Taka magia :)