[kacper635]

Siemanko swiry. Mam problem, mianowicie chce wpisywac do tej tablicy jednowymiarowej jakiś podciąg liczb z pliku i potem sprawdzać czy jest to ciąg rosnący. Kod kompiluje sie ale pojawia sie ze cmd przestał działać( ͡° ʖ̯ ͡°) Pomoże ktoś zlokalizować błąd i ewentualnie sprawdzi czy to bedzie działać git wtedy?

#include
#include
using namespace std;
int main()
{
int licznik=0,k, tab[k], licznik2=0;
ofstream zapisz;
ifstream czytaj("dane.txt");

while(czytaj >> tab[k])
{
cout << tab[k] << endl;
licznik++;
}

for(int i =0; i < licznik; i++)
{
if(tab[i] < tab[i+1])
licznik2++;

else{
zapisz.open("wynik.txt");
zapisz << "Nie";
zapisz.close();
}
}

if(licznik2 = sizeof(tab[k]-1))
{
zapisz.open("wynik.txt");
zapisz << "Tak";
zapisz.close();
}
return 0;
}

#cpp #programowanie

[Kaczus2B]

tab[k],

@kacper635: ta deklaracja jest zla, a reszta to konsekwencja tego. Dlatego w standardzie nie ma mozliwosci takiej deklaracji, bo ludzie nie umieja tego uzywac.

[kacper635]

@Kaczus2B: jak to powinno być dobrze zadeklarowane?

[Kaczus2B]

@kacper635: powinna byc konkretna wartosc, albo jesli korzystasz z VLA to po ustaleniu wartosci k. Ale jesli piszesz w C++ to masz takie typy jak std::vector!

[kacper635]

@Kaczus2B: Dość skomplikowane jest do dla mnie bo raczej tylko w c++ siedze w szkole czy przy zadaniach jakis do niej, a moge po prostu zrobić tab[100]?

[r00ti]

@kacper635: zdeklarowałeś k, ale nie zainicjalizowałeś jej. Także, w zmiennej k jest garbage value, Pamiętaj, że zmienne lokalne trzeba inicjalizować. Zrób sobie np. #define k (i ilosc miejsca w tablicy) albo zmienna globalna statyczną.

[Kaczus2B]

@kacper635: tak mozesz, ale jak bedziesz mial wiecej niz 100 elementow to bedzie niespodzianka. Dlatego uzyj typu std::vector, on jest w standardzie c++

[kacper635]

@Kaczus2B: Póki co chciałbym założyc że nie będzie miała więcej niż 100 elementów, ale dalej mi wywala tego cmd xd

[Kaczus2B]

while(czytaj >> tab[k])

@kacper635: to kolejny blad, nie dosc, ze k nie ma poczatkowej wartosci to sie nie zmienia

[kacper635]

@Kaczus2B: A jak to powinno poprawnie być?

[Kaczus2B]

if(licznik2 = sizeof(tab[k]-1))

@kacper635: to kolejny blad....

co do tego jak powinno byc poprawnie. Zalezy co chcesz uzyskac, jak mniemam chcesz wypelnic tablice, wiec najpierw powinienes zainicjowac k = 0; a w srodku petli zwiekszac wartosc k, np jako k++; ale to moge tylko domniemywac, ze o to Ci chodzi.

kolejny blad to:

if(tab[i] < tab[i+1])

wychodzisz poza tablice bo do i = licznik - 1 vesziesz iterowal, elementow wypelnionych masz

[kacper635]

@Kaczus2B: Troche pozmieniałem i myślisz że teraz jest już okej? Wlaśnie problem jedynie chyba z tym
if(tab[i] < tab[i+1])

Teraz to wygląda tak:
#include
#include
using namespace std;
int main()
{
int licznik=0,k=0, tab[100], licznik2=0;
ofstream zapisz;
ifstream czytaj("dane.txt");

while(czytaj >> tab[k])
{
cout << tab[k] << endl;
licznik++;
k++;
}

for(int i =0; i < licznik; i++)
{
if(tab[i-1] < tab[i])
licznik2++;
}

if(licznik2 == licznik-1)
{
zapisz.open("wynik.txt");
zapisz <<

[Kaczus2B]

@kacper635: na pierwszy rzut oka wyglada poprawnie. Mozesz zastapic 'k' przez licznik i pozbyc sie jednej zmiennej :) Proponuje tez poznac vector, bo sie przydaje.

[kacper635]

@Kaczus2B: A jeśli chodzi o sprawdzanie czy podciąg jest malejący?
bo tutaj mam rosnący: if(tab[i-1] < tab[i])
a potrzebuje i ten i ten

[Kaczus2B]

@kacper635: a zaraz zaraz i tak masz blad zaczynasz od 0 a sprawdzasz element i -1

[kacper635]

@Kaczus2B: wlasnie zastanawiam się jak to tam ma byc, a o dziwo to dziala

[Kaczus2B]

@kacper635: ze dziala to przypadek... po prostu 'i' zacznij od 1 :)