Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > comp.lang.c > #5946

Re: looking at binary files with C

From James Waldby <not@valid.invalid>
Newsgroups comp.lang.c
Subject Re: looking at binary files with C
Date 2011-06-13 01:16 +0000
Organization A noiseless patient Spider
Message-ID <it3od3$j3j$1@dont-email.me> (permalink)
References <95j3bgFj2qU1@mid.individual.net> <1bk4crnhkt.fsf@snowball.wb.pfeifferfamily.net> <95kmvoFn2bU1@mid.individual.net>

Show all headers | View raw


On Sun, 12 Jun 2011 14:47:49 -0600, Uno wrote:
...
> #define SIZE 20000000
...
>      for (j = 0; j < SIZE; ++j) {
> 	c = fgetc(fp);
> 	counter++;
> 	if (c != EOF) {
> 	    a[c] = ++a[c];
> 	} else
> 	    break;
>      }
...

That code is quite ugly and limited (value of SIZE is irrelevantly
based on a specific file's size), besides having the undefined-
behavior problem that other posters mentioned, in "a[c] = ++a[c]".
I suggest that you instead use code like the following:

    while (1) {
       c = fgetc(fp);
       if (c == EOF) break;
       ++a[c];
       ++counter;
    }

Some people prefer to do the read and EOF test in one line:
       if ((c = fgetc(fp)) == EOF) break;

-- 
jiw

Back to comp.lang.c | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 00:06 -0600
  Re: looking at binary files with C Ian Collins <ian-news@hotmail.com> - 2011-06-12 18:41 +1200
    Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 00:58 -0600
  Re: looking at binary files with C Barry Schwarz <schwarz45@yahoo.com> - 2011-06-12 00:05 -0700
    Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 01:41 -0600
      Re: looking at binary files with C "Joachim Schmitz" <jojo.nospam@schmitz-digital.de> - 2011-06-12 11:44 +0200
      Re: looking at binary files with C Joe Pfeiffer <pfeiffer@cs.nmsu.edu> - 2011-06-12 10:25 -0600
  Re: looking at binary files with C Ike Naar <ike@sverige.freeshell.org> - 2011-06-12 10:26 +0000
  Re: looking at binary files with C Eric Sosman <esosman@ieee-dot-org.invalid> - 2011-06-12 08:23 -0400
    Re: looking at binary files with C Hans Vlems <hvlems@freenet.de> - 2011-06-13 23:48 -0700
    Re: looking at binary files with C Tim Rentsch <txr@alumni.caltech.edu> - 2011-06-23 00:55 -0700
  Re: looking at binary files with C "BartC" <bc@freeuk.com> - 2011-06-12 14:26 +0100
  Re: looking at binary files with C Joe Pfeiffer <pfeiffer@cs.nmsu.edu> - 2011-06-12 10:24 -0600
    Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 14:47 -0600
      Re: looking at binary files with C "BartC" <bc@freeuk.com> - 2011-06-12 22:23 +0100
        Re: looking at binary files with C Keith Thompson <kst-u@mib.org> - 2011-06-12 14:29 -0700
          Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 16:31 -0600
            Re: looking at binary files with C Keith Thompson <kst-u@mib.org> - 2011-06-12 16:14 -0700
              Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 18:41 -0600
      Re: looking at binary files with C Eric Sosman <esosman@ieee-dot-org.invalid> - 2011-06-12 17:42 -0400
        Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 17:01 -0600
          Re: looking at binary files with C Keith Thompson <kst-u@mib.org> - 2011-06-12 16:23 -0700
            Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 17:24 -0600
              Re: looking at binary files with C Ian Collins <ian-news@hotmail.com> - 2011-06-13 11:28 +1200
                Re: looking at binary files with C Geoff <geoff@invalid.invalid> - 2011-06-12 16:52 -0700
              Re: looking at binary files with C Keith Thompson <kst-u@mib.org> - 2011-06-13 08:51 -0700
            Re: looking at binary files with C Seebs <usenet-nospam@seebs.net> - 2011-06-13 14:39 +0000
              Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-13 20:01 -0600
                Re: looking at binary files with C Keith Thompson <kst-u@mib.org> - 2011-06-13 19:24 -0700
                Re: looking at binary files with C Seebs <usenet-nospam@seebs.net> - 2011-06-14 17:02 +0000
                Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-16 01:32 -0600
          Re: looking at binary files with C Eric Sosman <esosman@ieee-dot-org.invalid> - 2011-06-13 07:57 -0400
            Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-13 20:23 -0600
              Re: looking at binary files with C Eric Sosman <esosman@ieee-dot-org.invalid> - 2011-06-13 23:12 -0400
                Re: looking at binary files with C lovecreatesbeauty <lovecreatesbeauty@gmail.com> - 2011-06-14 01:09 -0700
          Re: looking at binary files with C Oliver Jackson <chexmofo@hotmail.com> - 2011-06-15 20:15 -0700
      Re: looking at binary files with C Joe Pfeiffer <pfeiffer@cs.nmsu.edu> - 2011-06-12 18:41 -0600
      Re: looking at binary files with C James Waldby <not@valid.invalid> - 2011-06-13 01:16 +0000
        Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-12 19:48 -0600
        Re: looking at binary files with C "io_x" <a@b.c.invalid> - 2011-06-14 12:19 +0200
          Re: looking at binary files with C lovecreatesbeauty <lovecreatesbeauty@gmail.com> - 2011-06-14 03:58 -0700
            Re: looking at binary files with C "io_x" <a@b.c.invalid> - 2011-06-14 19:07 +0200
              Re: looking at binary files with C "io_x" <a@b.c.invalid> - 2011-06-14 19:46 +0200
              Re: looking at binary files with C Joe Pfeiffer <pfeiffer@cs.nmsu.edu> - 2011-06-14 16:08 -0600
          Re: looking at binary files with C Ian Collins <ian-news@hotmail.com> - 2011-06-15 08:47 +1200
      Re: looking at binary files with C Hans Vlems <hvlems@freenet.de> - 2011-06-13 23:51 -0700
  Re: looking at binary files with C Michael Press <rubrum@pacbell.net> - 2011-06-15 13:06 -0700
    Re: looking at binary files with C Uno <Uno@example.invalid> - 2011-06-16 01:38 -0600
      Re: looking at binary files with C Michael Press <rubrum@pacbell.net> - 2011-06-16 00:54 -0700
        Re: looking at binary files with C James Waldby <not@valid.invalid> - 2011-06-16 16:59 +0000
      Re: looking at binary files with C "J. J. Farrell" <jjf@bcs.org.uk> - 2011-06-16 18:32 +0100

csiph-web