Thread: Problem in file handling.

You need to indent your code, what you have there is a bad indentation style.

Code:

	void main()

this is wrong.

no problem with your logic i guess..
but,
1) as invariantloop pointed out, your indentation is horrible.
2) main returns an int. wait till salem posts, watch his avataar, you'l know why. it should be int main() and not void main()
3)strcmp returns 0 and not 1 if the strings are the same. i guess thats where your problem lies. check for

Code:

strcmp()==0

If you want help you should really learn to listen. According to the C standard, the return type for main() cannot be void. If it works for you on your compiler well then lucky you. But around here we stick to standards as much as possible. There's an FAQ for it on this site. Here's a excerpt in particular that talks about it:

Under regular function calling/returning in C and C++, if your don't ever want to return anything from a function, you define it's return type as void. For example, a function that takes no arguments, and returns nothing can be prototyped as:

void foo(void);

A common misconception is that the same logic can be applied to main(). Well, it can't, main() is special, you should always follow the standard and define the return type as int.

Originally Posted by Abhid

Hello,
I thank Ping and invariantloop for their suggestions.I made the 1's to 0's in the if condition,but the code still doesn't work.Since main can be void, thus I have kept it void. Do provide me with the solution if you can.Thanks.

Like everyone has said, you do not declare main as being void. itsme86 did not post a link to the relevant FAQ, but I will - go read it, and understand that you ought not to be using void main(). Now then, on to your various problems. You need to take a look at what is being written to your file 'Rec'. It will not work as you wish - make some modifications so it spaces things out properly. You need to change your if statements to make it possible for your flag variable to be set to 1. Otherwise, it is always 0, and that is somewhat useless. Also, have a look at your scanf() statements. In the way you are using them, you are inviting a buffer overflow problem, similar to what you could create by using the gets() function. If you must use scanf(), be sure to use a field width specifier, and also check your return value, to be sure the amount of conversions you expect are being done. Or, you could just use fgets() and sscanf() and have a much nicer solution to getting user input.

That should get you started at least...

another point is the use of conio.h which is highly non portable
and yah, make some changes in your code..
1) clrscr()==bad.. check the faq to clear the screen
2)

Code:

 fprintf(fp,"%s %s\n",p->usr,p->pass);

this will ensure that all of the things are on a new line and that helps.. the reason is that fscanf that you are using will take in characters till it encounters a white space. now, you can either do what i said, or after the last %s put a space, otherwise the letters will be stored continuously.
3) also, specifying the type is a good practise.. "rec.txt" is much preferred over just "rec"
i guess thats all thats required...and yes, main is int and not void. if it works for you, good enuf, but it wont work for all, stick to the standards and avoid bad remarks..just some advice from a fellow indian..chao