[Not a question] find the longest word in a sentence. If the sentence has more than one word of the same length you should pick the first one.

Question

1.00/5 (3 votes)

See more:

C++

#include <stdio.h>

int main()
{
	char str[100],sav[100],tmp[100];
	int i,count=0,k=0,len,flag,a[10];

	printf("Enter string:");
	gets(str);

	printf("\nYou entered:");
	puts(str);
	printf("\n");
	len=strlen(str);


	while(1)
	{
		for(i=0;str[i]!='\0';i++)
		{
			count++;
			sav[i]=str[i];

			if(str[i] == ' ' || str[i] !='\0')
			{
				if(k<count)>
				{
					printf("No. of words is:%d\n",count);
					k=count;
					printf("Words separated by spaces:");
					puts(sav);
					printf("\n");
				}
				count = 0;
			}
		}
	}

	return 0;

}

Posted 11-Oct-13 12:00pm

Member 10331161

Updated 11-Oct-13 12:28pm

Sergey Alexandrovich Kryukov

v4

Add a Solution

Comments

Valery Possoz 11-Oct-13 18:04pm

And... what do you want? sorry but I don't get it.

Member 10331161 11-Oct-13 18:10pm

suppose ...the input is
I live in the USA....
output should be:
live......
because live has 4 letters!

CHill60 11-Oct-13 18:12pm

What Valery means is what is your problem? You've stated what your homework says and posted some code, but haven't said what the problem is.

Member 10331161 11-Oct-13 18:16pm

im not able to get it......my code doesnot work...i need some modification for this code

Member 10331161 11-Oct-13 18:17pm

there is some problem in while loop....not able to solve it....stuck at this problem.

[no name] 11-Oct-13 20:37pm

Nothing wrong with the while loop - it will do what you have told it - go forever. Design a condition to get out of the loop and apply it.
Get that working properly and go from there.

Sergey Alexandrovich Kryukov 11-Oct-13 18:29pm

Did you use the debugger?
—SA

Member 10331161 11-Oct-13 18:30pm

no....im nt that good in programming...just started my career

Sergey Alexandrovich Kryukov 11-Oct-13 18:43pm

Wait a minute... First, learn to use the debugger, than do programming... good a bad. Don't waste your time working without a debugger.
Look, use the debugger, try to find out and solve the problem by yourself, come back if you still have a problem.
—SA

Member 10331161 11-Oct-13 19:14pm

calm down.....im just 8 years old and my dad gave me this as homework....if you want to help....otherwise its fine....thank you

Sergey Alexandrovich Kryukov 11-Oct-13 19:20pm

You calm down. Why you even talk in this manner? I advised you a real thing, it's in your interest to follow this advice. I did it because I really want to help, and this is the best thing you should do next. If you cannot make such decision, have your dad talk to me.
Wish you the best of luck, by the way.
—SA

Member 10331161 11-Oct-13 19:37pm

simply say you dont know!

Sergey Alexandrovich Kryukov 11-Oct-13 19:44pm

Don't be so ridiculous. Do you think you can provoke anyone here? Be positive, to get any help, and to achieve anything in general. Otherwise, you are not interesting enough to anyone, to talk to you.
—SA

Garth J Lancaster 11-Oct-13 22:40pm

the sad thing(s) - he's 8 and has an attitude already, he hasnt broken the task into meaningful chunks he can test as he goes, he's probably going to migrate to VB if he ever stays in his 'career', and last but not least, he'll probably be our boss in years to come... shoot me, shoot me NOW

Sergey Alexandrovich Kryukov 12-Oct-13 1:16am

Don't worry for me: so far, I never allowed anyone to be my "boss". I would rather sorry for this kid. :-)
But your words about VB and "bossing" point out the phenomenon...
—SA

enhzflep 11-Oct-13 23:32pm

Oh yes, really? I don't believe you. I think you were born in around 1995, not 2005.
Incidently, there's some redundancy in at least one of your if-conditions.

Take a close look at this line:

if(str[i] == ' ' || str[i] !='\0')

That reads in english as: "If the character in str at position i equals space, or if the character in str at position i does not equal NULL" See - there's only 1 instance in which this will return false - when the indexed char is NULL.

Personally, I'd rewrite that little mess as:

if ( str[i] )

or if I really wanted to make the code clear to those unfamiliar with the language, I'd write

if (str[i] != NULL)

or, using your form,

if (str[i] != '\0')

I also notice your second instance of an if statement: "if(k<count)>"
I hope that's not a copy'n'paste representation of your code - I hope you typed that mistake in when submitting this question. That wont even compile, let alone function unexpectedly/undesirably. (The second angled bracket doesn't belong there)

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Answer 1 · 2013-10-11T20:09:00

This is homework, and we don't do homework! :laugh:

But your code doesn't work, because you haven't told it to do what you want it to - it doesn't look at the string and work out any word lengths, which you have to do before you can work out which word is longest.

The easiest way to do this is to create three functions
1) get the user string.
2) find each word.
3) find the longest word.

That way, you code becomes a long simpler, and more obvious to both read, and write.

I'm not sure which way to suggest you do this, but... Do you know how to set up arrays? And what pointers are?