Programiranje u C++-u - pitanja i odgovori

To ti se dogada zato sto program ne moze otvoriti datoteku. A ne moze ju otvoriti zato sto ili ima pogresno ime, ili ju nisi stavio na pravo mjesto. Provjeri zove li se datoteka doista "brojevi.txt" a ne "brojevi" ili "brojevi.txt.txt" (windows explorer prikriva ekstenzije poznatih tipova datoteka, pa mozes imati problema ako si pokusao promijeniti ekstenziju preimenovanjem datoteke).

Btw, malo sam ti uredio kod da ne izgleda onako nakaradno:

#include <stdio.h>

int main()
{
    int niz[20], br=0, i, sum=0, pr;
    FILE *dat;
   
    dat=fopen("brojevi.txt", "r");
    while (!feof(dat))
    {
        fscanf(dat, "%d", &niz[br]);
        br++;
    }
    fclose(dat);
   
    for (i=0; i<br; i++)
        sum=sum+niz[i];
   
    pr=sum/br;
    printf("%d", pr);
    return 0;
}

Po Standardu, svaka funkcija mora nesto vratiti. Ako funkcija nista ne vraca, tada mora biti tipa void. Ako je tipa int, mora vratiti integer, bezobzira vraca li ona stvarno nesto ili ne. O tome ti kao programer moras brinuti. Standard kaze da funkcija main() mora biti tipa int. Prema tome mora vracati integer. Zato return 0;.

Zato.

Aha nisam primjetio da si mi postavio int main (), kod mene je bilo void, ok.

Neće pa neće, sad sam uradio još jedan sličan:
 
Napišite program koji čita sve brojeve iz datoteke brojevi.txt. Ispisati najveći i najmanji broj.
 
 
 
#include <stdio.h>
 
void main()
{
 int niz[20], br=0, i, min, max;
 FILE *dat;
 
 dat=fopen("brojevi.txt","r");
 while(!feof(dat))
 {
  fscanf(dat,"%d", &niz[br]);
  br++;
 }
 fclose(dat);
 
 min=niz[0];
 max=niz[0];
 
 for(i=0;i<br;i++)
 {
  if(niz[i]<min)
  {
   min=niz[i];
  }
 }
 for(i=0;i<br;i++)
 {
  if(niz[i]>max)
  {
   max=niz[i];
  }
 }
 
 printf("%d %d", min, max);
}
 
I opet isto, program radi u redu, barem mi se tako čini ali kad napravim u notpadu brojevi.txt i napunim je brojevima, ništa...datoteku tavim di sam na slici označio

Rekao sam ti da trebaš provjeriti jesi li otvorio datoteku! Ne možeš čitati iz datoteke, ako je nisi uspješno otvorio.

I ponovno - funkcija main vraća vrijednost tipa int - dakle, ne možeš imati void main(), možeš imati samo int main().

#include <iostream>
#include <stdio.h>

using namespace std;

int main()
{
    int niz[20], br = 0, i, min, max;
    FILE * dat;

    dat = fopen("brojevi.txt", "r");

    if (dat != NULL)
    {
        while (!feof(dat))
        {
            fscanf(dat, "%d", & niz[br]);
            br++;
        }
        fclose(dat);

        min = niz[0];
        max = niz[0];

        for (i = 0; i < br; i++)
        {
            if (niz[i] < min) min = niz[i];

            if (niz[i] > max) max = niz[i];
        }

        printf("%d %d", min, max);
    } else cout << "Datoteka ne postoji!";

    return 0;
}

jedini problem kod c++ visual studia vidim samo što jako sporo compajlira... inače je ok

Zato jer ju je spremio u krivu mapu. Tj. otvara nepostojeću datoteku. Zato sam mu i rekao neka programom spremi brojeve u datoteku - onda je siguran da će program tražiti datoteku tamo gdje ju je i kreirao.

Nije bitno kamo je on Notepadom spremio datoteke, nego je bitno odakle njegov program čita datoteke kad se pokrene - dakle, koji je njegov radni direktorij. Očito je ili da mu to nije radni direktorij ili je krivo napisao naziv datoteke. Neka kreira datoteke (ne mora ju ni puniti podacima) pomoću svog programa, istog tog programa kojim pokušava otvoriti datoteku za čitanje pa neka pogleda gdje mu ju je program kreirao.

Debug assert failed je problem specifičan za Visual Studio i milijuni stvari ga mogu uzrokovati. Program ne mora nužno biti krivo napisan. No prvo bi trebalo provjeriti da li se datoteka uspješno otvorila:

if((dat = fopen("brojevi.txt", "r"))==NULL){
    printf("Ne mogu otvoriti datoteku!");
    return -1;
}

i izbjegavati koristiti feof jer nerjetko zbog njega se zadnji podatak u datoteci može pročitati dva puta (iako ne bi trebao!). Umjesto toga napraviti ovakvo nešto:

while (fscanf(dat, "%d", &niz[br++]) != EOF);
fclose(dat);

..a dalje je sve lagano.

Konkretno ovaj assert fail se javlja zbog dijela koda koji sam ovdje napisao, zakljucio bih iz tog isjecka koda zbog datoteke i feof-a.

Pa naravno, to sam i gore isto stavio, da je vracen NULL (samo sam editirao).

Hm, pa najbolje je pogledati otkud potjece neki problem, pa poceti traziti od tog mjesta, u ovom slucaju je to bila ova datoteka feoferr.c na liniji 44 iz cega se najjasnije vidi u cemu je problem (iako smo ajd, vec vise puta ustanovili u cemu je problem :D). Ja ne znam drugi nacin rjesavanja problema, osim procitati sto kompajler/debugger kaze i koji error izbaci, pa onda gledati u kodu u cemu je greska...

Polje se šalje kao referenca (ili, reći će neki, pokazivač). To znači da si mijenjanjem sadržaja polja u funkciji mijenjao originalno polje. To opet znači da nema nikakvog vraćanja polja glavnoj funkciji, nego u glavnoj funkciji jednostavno ispišeš polje.

Napravi ovako, pa ćeš vidjeti da program radi, odnosno u fopen funkciju unesi argument da se može kreirati datoteka - "file access modes".

o fopen funkciji pogledaj ovdje:

http://www.cplusplus.com/reference/clibrary/cstdio/fopen/

#include <iostream>
#include <stdio.h>

using namespace std;

int main()
{
    int niz[20], br = 0, i, min, max;
    FILE * dat;

    dat = fopen("brojevi.txt", "a+");

    if (dat != NULL)
    {
        while (!feof(dat))
        {
            fscanf(dat, "%d", &niz[br]);
            br++;
        }
        fclose(dat);

        min = niz[0];
        max = niz[0];

        for (i = 1; i < br; i++)
        {
            if (niz[i] < min) min = niz[i];

            if (niz[i] > max) max = niz[i];
        }

        printf("%d %d", min, max);
    } else cout << "Datoteka ne postoji!";

    return 0;
}

Dakle , sa ovim access modom si prilikom otvaranja kreirao datoteku ako ona ne postoji, a ako postoji - otvaraš je bez brisanja.

A ovo je dobra fora ako ćemo primjeniti @mbaksin s.w.a.t. (on voli ubacivanje inkrementa u index polja)

while (!feof(dat))
        {
            fscanf(dat, "%d", &niz[br++]);
        }