Thread: Help using fgets()

My friend is having this problem :

Hi , Im having trouble with my code. Can you please look at it and show me ways of using fgets instead of fscanf as I get problems with it .

Code:

int main(int argc, char * argv[])
{
 // Mainline Variable Declarations
FILE * output = stdout;
FILE * input = stdin;

int exp_time;
float aperture_value;


       fprintf(output,"Please enter an exposure time: ");  
       fscanf(input,"%d",&exp_time);
	
	
	fprintf(output, "Please Enter an Aperture Value: ");
       fgets(aperture_value,10,stdin);
	
if 
(aperture_value !=(1.2||1.4||1.8||2||2.8||4||5.6||8||11||16||22||32))
{
	fprintf(output, "The value entered was incorrect.  The accepted values are :\n f1.2, f1.4,  f1.8, f2, f2.8, f4, f5.6, f8, f11, f16, f22, f32.\n\nProgram Closed.\n");
}
else{
(fprintf(output, "Aperture Value accepted."));
}
}

I tried using fgets as shown above in my code, but it didn't work . Thanks , much appreciated.

Well the reason is that fgets does not have the syntax you used.

Its char *fgets(char *Buffer, int num, FILE *fpointer);

This means that fgets has 3 arguments, the first is an array or buffer say it as you want of characters that you have to create before passing it in fgets().

eg. char Buffer[BUF_SZ] = ""; or char Buffer[10] ; or dynamically
char *Buffer = malloc(10); //Remember to free it in the end.

The second argument is the number of characters that fgets will read,
in case of using static declaration of Buffer you can use the sizeof(Buffer) but if you have malloced the Buffer you cant. You have to write down the exact amount.

The third argument is the file pointer that corresponds to the file from which fgets will try to read the data and store it in your buffer.

So a final example is:
char Buffer[256] = "";

fgets(Buffer, sizeof(Buffer), stdin);

Now it will prompt you to enter the line and when you press enter whatever you write will be stored in Buffer.
Now if you want just to take some numbers from the Buffer you must use either atoi, either sscanf or whatever function does the job.

Also fgets has return value but i think you dont need that this time, try to understand the basic part of fgets.

Hope i helped.

Why is it you all use this tech

Code:

FILE * output = stdout;
FILE * input = stdin;

....
....
....

fprintf(output,"Please enter an exposure time: ");  
fscanf(input,"%d",&exp_time);
	
fprintf(output, "Please Enter an Aperture Value: ");
fgets(aperture_value,10,stdin);

When you have standard printf fnction to output values which is by default pointing to stdout. Is this what you teacher thought you?

ssharish

Originally Posted by Salem

Your "other" friend is much more advanced than you.
http://cboard.cprogramming.com/showt...light=aperture

You're both at the same school / college anyway - close enough.

There seems to be a couple of people from our programming class on these boards.

Originally Posted by ssharish2005

Why is it you all use this tech

Code:

FILE * output = stdout;
FILE * input = stdin;

....
....
....

fprintf(output,"Please enter an exposure time: ");  
fscanf(input,"%d",&exp_time);
	
fprintf(output, "Please Enter an Aperture Value: ");
fgets(aperture_value,10,stdin);

When you have standard printf fnction to output values which is by default pointing to stdout. Is this what you teacher thought you?

ssharish

Yes , he insists we use frpintf and fscanf . He said that it will be important in future