success! I think

[Bill Cunningham]

I began to think about some excercises I could with files after reading
turtorials today on arithmetic operators and confusing myself and decided to
do this.
Take the mv command from linux that moves and renames files read it as
binary and write it as text. After some compiler qipes it compiled and I
believe it did want I wanted it to.

#include <stdio.h>

int main(void) {
FILE *ifp, *ofp;
int a;
ifp=fopen("mv", "rb");
ofp=fopen("m"," wt");
a=fgetc(ifp);
if (a==EOF) {
fclose(ifp);
}
fputc(a,ofp);
if (a==EOF) {
fclose(ofp);
}
}

Because I didn't know how long the file would be I couldn't say exactly
how many bytes I should read so fread/fwrite were out and I decided on
fgetc/fputc. Is my code up to standards? And also can someone tell me what
the first int parameter of fputc is and does? My references that I look at
do not say. But I think I used it right. I guessed a 'container' for a char.

Bill

[Bartc]

"Bill Cunningham" <no****@nspam.c omwrote in message
news:gtvMj.4699 $bx3.1392@trndd c02...

>
"Barry Schwarz" <sc******@dqel. comwrote in message
news:j3******** *************** *********@4ax.c om...

>> Because I didn't know how long the file would be I couldn't say
exactly
how many bytes I should read so fread/fwrite were out and I decided on

Your comment makes no sense. Both fread and fwrite have methods for
specifying how many bytes to read/write and for informing the caller
how many bytes were actually read/written. You don't have to use them
but to say you can't is a limitation on you and not the functions.

Exactly. Fread and fwrite require a buffer size and number of elements.
I just wanted to read the entire file without having to enter an exact
size so I don't think fwrite and fread would be the choice here.

In your original post you gave the impression the code worked and you wanted
comments on the coding standard.

Clearly you hadn't tested the code, unless you tested with a file containing
a single character.

The code below does a file copy byte by byte using fgetc/fputc, using binary
mode, of any type of file.

If you want to translate between text files, mixing binary and text modes on
input and output, that might give some strange effects.

If you want to translate a binary executable file as you indicated in your
post, into text, that is fairly meaningless unless you're talking about some
sort of dump or disassembly, but in that case I'd advise to forget that for
now.

/* Copy input file to output file */
#include <stdio.h>
#include <stdlib.h>

#define inputfile "mv"
#define outputfile "m"

int main(void) {

FILE *ifp, *ofp;
int a;

ifp=fopen(input file,"rb");

if (ifp==0){
printf("Can't open input file %s\n",inputfile );
exit (0);
}

ofp=fopen(outpu tfile,"wb");
if (ofp==0){
printf("Can't open output file %s\n",outputfil e);
fclose(ifp);
exit (0);
}

while ((a=fgetc(ifp)) !=EOF)
fputc(a,ofp);

fclose(ifp);
fclose(ofp);

}

--
Bart

[Bill Cunningham]

"Bartc" <[e-mail] rote in message
news:Sh******** *********@text. news.virginmedi a.com...

>
"Bill Cunningham" <no****@nspam.c omwrote in message
news:gtvMj.4699 $bx3.1392@trndd c02...

>>
"Barry Schwarz" <sc******@dqel. comwrote in message
news:j3******* *************** **********@4ax. com...

>>> Because I didn't know how long the file would be I couldn't say
exactly
how many bytes I should read so fread/fwrite were out and I decided on

Your comment makes no sense. Both fread and fwrite have methods for
specifying how many bytes to read/write and for informing the caller
how many bytes were actually read/written. You don't have to use them
but to say you can't is a limitation on you and not the functions.

Exactly. Fread and fwrite require a buffer size and number of
elements. I just wanted to read the entire file without having to enter
an exact size so I don't think fwrite and fread would be the choice here.

In your original post you gave the impression the code worked and you
wanted comments on the coding standard.

Clearly you hadn't tested the code, unless you tested with a file
containing a single character.

That's what I must've done.

The code below does a file copy byte by byte using fgetc/fputc, using
binary mode, of any type of file.

If you want to translate between text files, mixing binary and text modes
on input and output, that might give some strange effects.

If you want to translate a binary executable file as you indicated in your
post, into text, that is fairly meaningless unless you're talking about
some sort of dump or disassembly, but in that case I'd advise to forget
that for now.

OK

/* Copy input file to output file */
#include <stdio.h>
#include <stdlib.h>

#define inputfile "mv"
#define outputfile "m"

int main(void) {

FILE *ifp, *ofp;
int a;

ifp=fopen(input file,"rb");

if (ifp==0){
printf("Can't open input file %s\n",inputfile );
exit (0);
}

ofp=fopen(outpu tfile,"wb");
if (ofp==0){
printf("Can't open output file %s\n",outputfil e);
fclose(ifp);
exit (0);
}

while ((a=fgetc(ifp)) !=EOF)
fputc(a,ofp);

fclose(ifp);
fclose(ofp);

}

I appreciate your patience. When I first looked at C for some reason I
became transfixed. I know Basic pretty well but I would like to learn C very
much. Some times I sound confused because much of the time I am. I have
lived in the same town all my life and I pretty much gave up for the most
part on driving because I would start out somewhere and end up on the
opposite side of town. This is the confusion I have to deal with with drugs
that stop my panic attacks.

So I'm not just going to catch onto a language like C. But I will
continue to try but I think I may have a hard road to hoe. Thanks again.

Bill

[lawrence.jones]

Bill Cunningham <no****@nspam.c omwrote:

>
Exactly. Fread and fwrite require a buffer size and number of elements.
I just wanted to read the entire file without having to enter an exact size
so I don't think fwrite and fread would be the choice here.

That's because you're not thinking enough. If you just want to read
bytes, use an element size of 1 and make the number of elements the
length of your buffer. fread() returns the number of elements it
actually read, which is what you need to pass to fwrite() to write the
chunk of data that you just read. Just do that in a loop to read and
writes successive chunks of the file until there's nothing left to read
and you're done.

-Larry Jones

I think grown-ups just ACT like they know what they're doing. -- Calvin

[Bill Cunningham]

<la************ @siemens.comwro te in message
news:gp******** ****@jones.home ip.net...

That's because you're not thinking enough. If you just want to read
bytes, use an element size of 1 and make the number of elements the
length of your buffer. fread() returns the number of elements it
actually read, which is what you need to pass to fwrite() to write the
chunk of data that you just read. Just do that in a loop to read and
writes successive chunks of the file until there's nothing left to read
and you're done.

Hum. There is sizeof. This might just work.

fread(buff,size of(int),1,fp);

I could try that. I just bet it would work to.

Bill

[Barry Schwarz]

On Mon, 14 Apr 2008 01:41:05 GMT, "Bill Cunningham" <no****@nspam.c om>
wrote:

>
<la*********** *@siemens.comwr ote in message
news:gp******* *****@jones.hom eip.net...

>That's because you're not thinking enough. If you just want to read
bytes, use an element size of 1 and make the number of elements the
length of your buffer. fread() returns the number of elements it
actually read, which is what you need to pass to fwrite() to write the
chunk of data that you just read. Just do that in a loop to read and
writes successive chunks of the file until there's nothing left to read
and you're done.

Hum. There is sizeof. This might just work.

fread(buff,siz eof(int),1,fp);

I could try that. I just bet it would work to.

What makes you think the size of an integer has any significance to
the code you presented?
Remove del for email

[Nick Keighley]

On 12 Apr, 22:43, "Bill Cunningham" <nos...@nspam.c omwrote:

* * I began to think about some excercises I could with files after reading
turtorials today on arithmetic operators and confusing myself and decided to
do this.
* * Take the mv command from linux that moves and renames files read it as
binary and write it as text. After some compiler qipes it compiled and I
believe it did want I wanted it to.

#include <stdio.h>

int main(void) {
*FILE *ifp, *ofp;
*int a;
*ifp=fopen("mv" ,"rb");
*ofp=fopen("m", "wt");
*a=fgetc(ifp);
*if (a==EOF) {
*fclose(ifp);
*}
*fputc(a,ofp);
*if (a==EOF) {
*fclose(ofp);
*}
* *}

* * Because I didn't know how long the file would be I couldn't say exactly
how many bytes I should read so fread/fwrite were out and I decided on
fgetc/fputc. Is my code up to standards? And also can someone tell me what
the first int parameter of fputc is and does? My references that I look at
do not say. But I think I used it right. I guessed a 'container' for a char.

you posted an almost identical problem in february. I gave
you a general way to copy a file. use google "keighley cunningham"
to find it.

Either learn this stuff or give up learning C.
--
Nick Keighley

[Richard]

"Bill Cunningham" <no****@nspam.c omwrites:

<la************ @siemens.comwro te in message
news:gp******** ****@jones.home ip.net...

>That's because you're not thinking enough. If you just want to read
bytes, use an element size of 1 and make the number of elements the
length of your buffer. fread() returns the number of elements it
actually read, which is what you need to pass to fwrite() to write the
chunk of data that you just read. Just do that in a loop to read and
writes successive chunks of the file until there's nothing left to read
and you're done.

Hum. There is sizeof. This might just work.

fread(buff,size of(int),1,fp);

I could try that. I just bet it would work to.

Bill

Learn to read a man page or a C book or, and I hate to say this, give
up. Personally I think you are trolling.

But if you're not, read your program as a cpu would generally do it -
line by line and statement by statement. Your code and your thinking
seem to bear almost no resemblance to the problem you want to solve.

[Bill Cunningham]

you posted an almost identical problem in february. I gave
you a general way to copy a file. use google "keighley cunningham"
to find it.

Either learn this stuff or give up learning C.

Nick I still have a copy of the file you posted and I am going to keep
it. I think I need to learn more about loops like while. You used an example
of do. Your code was very thorough and precise. I haven't got to do in the
tutorials but I will continue reading and studing your code. Don't think
your post was in vain.

Bill
--
Nick Keighley

[Bill Cunningham]

What makes you think the size of an integer has any significance to

the code you presented?

I was addressing Lawrence's post concerning fread. It really didn't have
much to do with using fgetc and fputc or the first code I posted.

Bill

[Nick Keighley]

On 14 Apr, 22:32, "Bill Cunningham" <nos...@nspam.c omwrote:

you posted an almost identical problem in february. I gave
you a general way to copy a file. use google "keighley cunningham"
to find it.

Either learn this stuff or give up learning C.

* * Nick I still have a copy of the file you posted and I am going to keep
it. I think I need to learn more about loops like while. You used an example
of do. Your code was very thorough and precise. I haven't got to do in the
tutorials but I will continue reading and studing your code. Don't think
your post was in vain.

your current problem seems almost identical to one I
gave a solution for. The only difference is the previous one
used fread/fwrite. I cannot comprehend how you can read my code
and then post code that tries to copy a file ***without using a
loop construct***
--
Nick Keighley