[JackOfAllTrades]

If you're getting errors, you need to tell use EXACTLY what the errors are. Copy and paste them into the post.

[Salem_c]

But there is no call to strlen() in your first post.

The error message is complaining about how you tried to call the function, not how it was defined.

[janotte]

Repost your code as it is now.

Your 'first post' doesn't mean much now that your post has been split from the old one you started posting in.
When you post your code answer these questions.
( a ) Does your code compile?
( b ) Any errors or warnings? If there are then share them with us.
Copy and paste the errors exactly as they are.
( c ) Is the program producing any output?
( d ) How is the actual output different to what you want / expect?
Give details and, ideally, examples.
If you provided inputs to the program tell us what they were.
( e ) What have you already tried to fix it?

BTW - We are not going to 'correct your code' but we are happy to help you correct your code

[janotte]

Okay debugging 101.
Add this debug output code to your code in the indicated location.
Compile, run and test your program.
What did you learn about your program?

	for(i=0;s[i]!='\0';i++){
		printf("s[%d] = %c || s2[%d] = %c \n",i,s[i],i,s2[i]);
		if (s[i]==s2[i])

BTW
"void main()" is wrong. Always wrong.
Always use "int main()".
Read here the words of the man who invented C++:
http://www.research.....html#void-main
Can I write "void main()"?
The definition void main() { /* ... */ } is not and never has been C++, nor has it ever been C. See the ISO C++ standard 3.6.1[2] or the ISO C standard 5.1.2.2.1.
And read here about some of the problems void main() can cause
http://www.gidnetwork.com/b-66.html

[janotte]

Well at least you are going to pass the first if() test sometimes now, so that's an improvement.

I am using the string "the dog" and the word "dog" as my test inputs.
If you look at your program behaviour in much the same way as before then we see we are still never reaching 'c++' using those test inputs so you need to think about this a lot more.

	for(i=0;s[i]!='\0';i++){
		int j;
		for (j=0; s[j]!='\0' ; j++)
		{
			printf("s[%d] = %c || s2[%d] = %c || c = %d\n",i,s[i],j,s2[j],c);
			if (s2[j]==s[i] )
			{  
				printf("%s\n","if(s2[j]==s[i]) resolved to true");
				i++;
				if (s[i+1]==' ' || s[i]=='.' )
				{	
					printf("%s\n","if(s[i+1]==' ' || s[i]=='.') resolved to true");
					c++;
				}
				else
					break;
			}
			else
			{
				break;
			}
		}
	}

Output

Enter any string : the dog
s = the dog
enter word: dog
s2 = dog
s[0] = t || s2[0] = d || c = 0
s[1] = h || s2[0] = d || c = 0
s[2] = e || s2[0] = d || c = 0
s[3] =   || s2[0] = d || c = 0
s[4] = d || s2[0] = d || c = 0
if(s2[j]==s[i]) resolved to true
s[6] = g || s2[0] = d || c = 0
s[7] = 
 || s2[0] = d || c = 0


occurences are 1

I really can't guess what your solution is. Are you able to describe your plan to yourself in words? Until you can do that then trying to write the code to implement the plan will be next door to impossible. You could write code without a word plan for weeks and if you ever get to the answer it will probably be by luck and there is a very good chance you will never get to an answer. You need to describe a plan to yourself and try it on a piece of paper and see if the plan seems to work. Once you have that plan writing the code will be far easier.


BTW what is this about?

printf("\n\noccurences are %d",c+1);

Why are you adding 1 to c?
You will always get a false result (except by luck) with that in there.

[janotte]

Good code layout makes it far easier to write correct code.
A good cook has a clean and well organised kitchen. You need to have a clean and well organised code layout if you are going to see easily what you have written and where you have made a mistake. You will also find that you may find people refuse to help you if your code is so badly laid out that they have to struggle to read it. You code is bad enough that people will ignore it. Be better than that.

So choose one of the common indentation styles and use it consistently.
http://en.wikipedia....ki/Indent_style

Just like before add some debug outputs to help you see what is happening in your code.
Put this line in where indicated

if (s[y]==' ')
{
	cout << "DEBUG: if (s[y]==' ') resolved to true" << endl;

	y++;

How often do you pass that if() test?

I still have no idea at all what solution you are trying to implement so it is next door to impossible to help you because it looks like you are just throwing random code at the page. You said before you have a plan. Have you tried that plan on a piece of paper with a pencil to see if it can possibly work? It doesn't appear that you have based on what you are presenting to us.

And at some stage your code has changed from C to C++. (Although a 10+ year out of date version of C++ that should not be seen in any civilised society.)

This is just a big bucket of mystery.

BTW
You should never use gets(). Never. Never. Never. I hope I am being clear. NEVER!!
Read this
http://www.gidnetwork.com/b-56.html