🚀 go-pugleaf

karel adams wrote:

> #include <stdio.h>
> main ()
> {
> int  fp_bgl            ; /* file pointer to file being read   */

>
> Als ik dit compileer (gcc mijn_prog.c) krijg ik een verwittiging
> Warning: assignment makes integer from pointer without a cast
> op de lijn van de fopen, ik heb dus blijkbaar fp_bgl als verkeerd type
> gedeclareerd
> Heb al wat andere type gezet (char, long...) maar niks helpt.
> Bij uitvoering geeft hij altijd -1 bytes gelezen dus de read lukt
> effectief niet.
> Wat doe ik mis? Hoe verbeter ik dit?

fp_bgl moet van het type FILE * zijn. Dus:

FILE *fp_bgl;

Voor de rest moet je de declaratie van fopen() weghalen. Deze is immers al
in stdio.h gedeclareerd (en ook nog eens op een andere wijze ook). Gebruik
ook fread() i.p.v. read(). Je gebruikt namelijk stream en file functies door
elkaar. En later netjes de file sluiten met fclose().


-- 
www.zenobits.com

KLinZ wrote:

Even een volledige recode:

#include <stdio.h>

int main () {
 FILE *fp_bgl;      /* file pointer to file being read   */
 int rd_cnt;        /* will tell how many *items* were actually read */
 char header[128];  /* first 128 bytes in header */

 fp_bgl=fopen("de_naam_van_mijn_bestand", "r");
 if(fp_bgl!=NULL) {
  printf ("file opened with FP_BGL= %p\n", fp_bgl);     /* op zich weinig
nuttige info */
  rd_cnt=fread(header, sizeof(header), 1, fp_bgl);
  printf ("Header has read %3d items\n", rd_cnt);
  fclose(fp_bgl);
 } else {
  printf ("bestand kon niet geopend worden\n");
 }

 return 0;
}

Merk op dat fread niet het aantal gelezen bytes teruggeeft, maar het aantal
succesvol gelezen 'items'. Je kunt die waarde dus gebruiken om te
controleren of het lezen gelukt is.

-- 
www.zenobits.com

KLinZ <klinz@nospamzenobitsspamno.com> wrote:
> KLinZ wrote:
> Even een volledige recode:

Bijna volledig mee eens ... maar:

>  char header[128];  /* first 128 bytes in header */

Ik zou die buffer 128+1 maken.

 - Philip

-- 
Philip Paeps

  BOFH Excuse #350:
    paradigm shift...without a clutch

Philip Paeps wrote:
>>
>>  char header[128];  /* first 128 bytes in header */
>
> Ik zou die buffer 128+1 maken.
>

Waarom? Hij wil toch maar 128 bytes inlezen? Hij heeft het nergens over een
zero terminated string ofzo.

-- 
www.zenobits.com

"KLinZ" <klinz@NOSPAMzenobitsSPAMNO.com> schreef in bericht
news:hKALa.14197$KF1.277566@amstwist00...
> Philip Paeps wrote:
> >>
> >>  char header[128];  /* first 128 bytes in header */
> >
> > Ik zou die buffer 128+1 maken.
> >
>
> Waarom? Hij wil toch maar 128 bytes inlezen? Hij heeft het nergens
over een
> zero terminated string ofzo.

ja kvroeg me dat ook al af
klinz heeft me goed begrepen: het is binaire data
die ik te lezen heb, g��n zero-terminated string
terwijl we het erover hebben:
als ik ze eenmaal ingelezen krijg,
spreek ik de array dan aan van 1 tot 128 of van 0 tot 127?

zonnige groetjes,
ka (trekt nu ter bbq)





>
> -- 
> www.zenobits.com
>

karel adams wrote:

> 
> "KLinZ" <klinz@NOSPAMzenobitsSPAMNO.com> schreef in bericht
> news:hKALa.14197$KF1.277566@amstwist00...
>> Philip Paeps wrote:
>> >>
>> >>  char header[128];  /* first 128 bytes in header */
>> >
>> > Ik zou die buffer 128+1 maken.
>> >
>>
>> Waarom? Hij wil toch maar 128 bytes inlezen? Hij heeft het nergens
> over een
>> zero terminated string ofzo.
> 
> ja kvroeg me dat ook al af
> klinz heeft me goed begrepen: het is binaire data
> die ik te lezen heb, g��n zero-terminated string
> terwijl we het erover hebben:
> als ik ze eenmaal ingelezen krijg,
> spreek ik de array dan aan van 1 tot 128 of van 0 tot 127?

In C lopen array-indices altijd van 0 tot n-1 met n de lengte van de array.

Dus van 0 tot 127 ja.

[snip]

Mvg,

-- 
Jo Vermeulen
Student Computer Science at the tUL
email: jo@lumumba.luc.ac.be
www: http://lumumba.luc.ac.be/jo

KLinZ <klinz@nospamzenobitsspamno.com> wrote:
> Philip Paeps wrote:
> > >  char header[128];  /* first 128 bytes in header */
> >
> > Ik zou die buffer 128+1 maken.
> 
> Waarom? Hij wil toch maar 128 bytes inlezen? Hij heeft het nergens over een
> zero terminated string ofzo.

Rare gewoonte van mij.  Ik voorzie altijd de mogelijkheid om er een \0 achter
te hangen, 'just in case'.  Zelfs als het geen string is, komt die \0 soms
nogal van pas als ik bizarre dingen met die data wil uitvreten.

 - Philip [of is het een slechte gewoonte? ;-)]

-- 
Philip Paeps

  BOFH Excuse #131:
    telnet: Unable to connect to remote host: Connection refused

Philip Paeps wrote:
> KLinZ <klinz@nospamzenobitsspamno.com> wrote:
>> Philip Paeps wrote:
>>>>  char header[128];  /* first 128 bytes in header */
>>> 
>>> Ik zou die buffer 128+1 maken.
>> 
>> Waarom? Hij wil toch maar 128 bytes inlezen? Hij heeft het nergens
>> over een zero terminated string ofzo.
> 
> Rare gewoonte van mij.  Ik voorzie altijd de mogelijkheid om er een
> \0 achter te hangen, 'just in case'.  Zelfs als het geen string is,
> komt die \0 soms nogal van pas als ik bizarre dingen met die data wil
> uitvreten. 
> 
>  - Philip [of is het een slechte gewoonte? ;-)]

Ja. Maar jij hebt wel meer rare gewoontes, dus ik vergeef het je.

-- 
www.zenobits.com