[gloriarose]

I have to search a specific string in a file.Here i am using 'strcmp' function but its not working properly.when am compiling it in linux using gcc it does not shows any error.i couldn't catch the mistake.

#include<stdio.h>
#include<string.h>
void main()
{
	FILE * fpi;
	int totallines=0,headerlines;
	char s1[100],s2[100],str1[]="END OF HEADER";
	fpi=fopen("rinexinput.txt","r");
	if(fpi==NULL)
	{
		printf("error in opening file");
	}
	while(!feof(fpi))
	{
		fgets(s1,100,fpi);
		totallines++;
		if(strcmp(str1,s1) == 0)
		{
			headerlines=totallines;
			exit(0);
		}
	}
	printf("%s %d\n",s1,headerlines);
}

[Adak]

A few suggestions:

1) You should be using strstr() to search for a string, inside of other larger strings. It's simpler by far, and much quicker, as well.

2) feof doesn't work the way you expect. Use

while((fgets(yourCharArrayName, sizeof(your arrayName), fp)) != NULL) {

Then use your char * pointer. If strstr() finds your target string, it will return it's address to your pointer (you will assign that). If not, it will return NULL.

So

if(ptrchar) {    //was the target string found?
  yes, process it here
}

[snoopy11]

Hi,

Dont use void main please use int main and return an integer value from your program

Usually this is zero for success.

the variable headerlines is used uninitialised...
ie not set to zero in this case...


this is preventing your program from working correctly.
Also the use of exit(0) should be replaced with ---- break;
Regards

Snoopy.

This post has been edited by snoopy11: 02 January 2013 - 11:38 PM