plez where is the bug?? here is the code

help me to find the bug ..
the code must print the same word that appears in the ..First sentences & Second sentences.. and must have the Max Length.
char First[]="Ann,Bob and Bill Abraham live in Alabama."
char Second[]="Alabama is where alabama Anne Abrahamson lives."
the output= Alabama

Code:

#include<stdio.h>
#include<ctype.h>
#include<string.h>
int WordLength(char *);
int WordCompare(char *,char *);
char *GoNextWord(char *);
char *max_equivalent_words(char *pFirst , char *pSecond){
    char *pTemp1=pFirst,*pTemp2=pSecond,*MaxP;int Len=0,MaxLen=0;
	while(*pTemp1 && isalpha(*pTemp1)==0)
		while(*pTemp2 && isalpha(*pTemp2)==0){
			if(WordCompare(pTemp1,pTemp2)){
				Len=WordLength(pTemp1);
				if(Len>MaxLen){
					MaxP=pTemp1;
					MaxLen=Len;
				}
				pTemp2=GoNextWord(pTemp2);
			}
		pTemp2=pSecond;
		pTemp1=GoNextWord(pTemp1);
	}
	return(MaxP);

}
char *GoNextWord(char *pFirst){
   char *p1;
   p1=pFirst;
   while(isalpha(*p1) && *p1!='\0')
       p1++;
   return (++p1);
}

void PrintWord(char *pWord){
   char *p1=pWord;
   while(*p1 && isalpha(*p1)){
	   printf("%c",*p1);
	   p1++;
   } 
}

int WordCompare(char *pFirst,char *pSecond){
   char *p1=pFirst,*p2=pSecond;int sum=0;
   if(WordLength(p1)!=WordLength(p2))
	   return 0;
   while(isalpha(*p1)){
	   if(*p1==*p2);
		   sum=sum;
	   p1++; p2++;
	   }
   if(sum==WordLength(p1))
		   return 1;
}

int WordLength(char *Word){
   char *p1;int i=0;
   p1=Word;
   while(isalpha(*p1)){
   p1++;
   i=i+1;
   }
   return(i);
}

int main() {
   char *p1,*p2;
   char First[]="Ann,Bob and Bill Abraham live in Alabama.";
   char Second[]="Alabama is where alabama Anne Abrahamson lives.";
   p1=max_equivalent_words(First,Second);
   PrintWord(p1);
   return 0;
}

i want to re wright the

Code:

sum=sum+1;

Line 36 in WordCompare Function

why dont u do this for the problem tokenize the two string ref this artical tokensize the string

you will get all the words seperated. Then comparing with words will become easy for you rather than comparing two literal string searching for a simplier word.

The algorithm is

1. Take string 1 token it and store it
2. Take string 2 tokenize it ans store it
3. Now compare Str1Tokenize[i] = str1tokenize2[i] on print it.
else
i++

hope this will make your life bit easier

ssharish2005

thank you for you advice.... i learned new things from your advice ...
but i am studying CS and i must use these algorithm cuz my homework is about pointers
and they want me to Suffer while doing the question....hard in the battle..
hard in the learning easy in the war..

On a side note take a look at your indentation too. Indent to three or four spaces each time, This can greatly increase program readability and greatly reduce debugging

I think thi should be like:

Code:

if(*p1==*p2)   //;
{
   sum++;
   p1++; 
   p2++;
}
else
{
   break;
}

If you'll use breakets always even for the single line if/else it will be easier to spot such errors... maybe

also in the WordLength function - you don't need i
Just return p1-Word

> while(*pTemp2 && isalpha(*pTemp2)==0)
Your code would probably be easier to understand if this loop were in a separate function.

> if(*p1==*p2);
This trailing ; in wordcompare isn't doing any good either.
But I see vart has already spotted that one
Using lots of braces to make your intentions obvious is a good idea as well.