CSV C Program

[DuckCowMooQuack]

Hey guys and gals,

Below is my code, my input file, and the output. I can't figure out why it is adding an extra zero at the end though. Can anyone help me with this?


CODE:

#include <stdio.h>
#include <stdlib.h>

int main(void)

{

 
FILE *ofp;

FILE *ifp;


int input;
int sum = 1;
int output[60];
char filename[50];


//Opens the file being printed to
 
ofp = fopen("output.dat", "w");

if(ofp == NULL)
   {

    printf("Unable to open output.dat for writing\n");
    exit(1);

   }


//Opens up the file with numbers in it

ifp = fopen("pix.dat", "r");


if(ifp == NULL)
   {

    printf("Unable to open pix.dat\n");
    exit(1);

   }



int i = 0;

//fscanf(ifp, "%d", &input);

while ( !feof(ifp) )
{
  fscanf(ifp, "%d", &input);
  fprintf(ofp, "%d,",input );
  //output[i] = input;
  if(sum%20==0)
	{
	  fprintf(ofp, "\n");
	}
  i++;
  sum++;  
}


return 0;

}

INPUT FILE:

2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
2
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
7
0
0
0
6
6
6
6
6
6
6
6
6
6
6
6
6
6
0
0
0
0
0
0
6
6
6
6
6
6
6
6
6
6
6
6
6
6
0
0
0
0
0
0
6
6
6
6
6
6
6
6
6
6
6
6
6
6
0
0
0
0
0
0
6
6
6
6
6
6
6
6
6
6
6
6
6
6
0
0
0
0
0
0
6
6
6
6
6
6
6
6
6
6
6
6
6
6
0
0
0
0
0
0
6
6
6
6
6
6
6
6
6
6
6
6
6
6
0
0
0
0
0
0
6
6
6
6
6
6
6
6
6
6
6
6
6
6
0
0
0
0
0
0
4
4
4
4
4
4
4
4
4
4
4
4
4
4
0
0
0
0
0
0
4
4
4
4
4
4
4
4
4
4
4
4
4
4
0
0
0
0
0
0
4
4
4
4
4
4
4
4
4
4
4
4
4
4
0
0
0
0
0
0
4
4
4
4
4
4
4
4
4
4
4
4
4
4
0
0
0
0
0
0
4
4
4
4
4
4
4
4
4
4
4
4
4
4
0
0
0
0
0
0
4
4
4
4
4
4
4
4
4
4
4
4
4
4
0
0
0
0
0
0
1
1
1
1
1
1
1
1
1
1
1
1
1
1
0
0
0
0
0
0
1
1
1
1
1
1
1
1
1
1
1
1
1
1
0
0
0
0
0
0
1
1
1
1
1
1
1
1
1
1
1
1
1
1
0
0
0

OUTPUT FILE

2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,
2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,
7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,
7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,7,
0,0,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,0,0,0,
0,0,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,0,0,0,
0,0,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,0,0,0,
0,0,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,0,0,0,
0,0,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,0,0,0,
0,0,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,0,0,0,
0,0,0,6,6,6,6,6,6,6,6,6,6,6,6,6,6,0,0,0,
0,0,0,4,4,4,4,4,4,4,4,4,4,4,4,4,4,0,0,0,
0,0,0,4,4,4,4,4,4,4,4,4,4,4,4,4,4,0,0,0,
0,0,0,4,4,4,4,4,4,4,4,4,4,4,4,4,4,0,0,0,
0,0,0,4,4,4,4,4,4,4,4,4,4,4,4,4,4,0,0,0,
0,0,0,4,4,4,4,4,4,4,4,4,4,4,4,4,4,0,0,0,
0,0,0,4,4,4,4,4,4,4,4,4,4,4,4,4,4,0,0,0,
0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,
0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,
0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,
0,

NOTE NOTE NOTE NOTE NOTE NOTE NOTE
------------------------------------------------
I know that it is frowned upon to use feof to control a loop. Please disregard this mistake ;P Thanks!!!

[DuckCowMooQuack]

Wow...So in this case the use of feof() did cause the error! By removing the extra return character at the end it solved the problem of the extra "0" at the end.

[Salem]

> By removing the extra return character at the end it solved the problem of the extra "0" at the end.
No, that just fudged it to work with your program.

What you really need to be doing is paying attention to what people say, and also on what the return result of fscanf() is.

Fumbling around, ignoring us because your code "mostly works" isn't going to help you in the long run.