newbster question Name.size

[ArmlessBastard]

Trying to use Name.size() to find out how many characters the user puts in. Name is a string::std can I not use .size for this? how can I accomplish this feat.

for (i=*Name.size(); i>0; --i)
	{
		//need to bring this to main()	
		*Reverse=*Reverse+*Name[i];
	}

Error

c:\documents and settings\administrator\my documents\visual studio 2005\projects\reverseit\reverseit\function.cpp(21) : error C2228: left of '.size' must have class/struct/union
        type is 'std::string *'
        did you intend to use '->' instead?

what is the '->' that the error is mentioned?

[GCDEF]

Why are you dereferencing it? What error are you getting?

[kolkoo]

if you have a pointer to a std:string

e.g. string * Name

you can either use (*Name).size() or Name->size() to call the function, provided you have already initialized that pointer with a new string object.

[Lindley]

Two things wrong with that code:

1) The * operator's precedence is such that it thinks you're trying to dereference the result of size(), which you aren't. You can do

(*Name).size()
// or, equivalently
Name->size()

Notice this latter is right there in the error message.

2) You aren't looping over right indexes. In particular,

(*Name)[Name->size()]

will always be invalid, because the largest valid index is Name->size() - 1.

Those are the only really *wrong* things in the code. However, consider:
-In many cases it's better to pass by reference rather than passing a pointer, because the syntax is much easier to deal with.
-It will be faster to pre-size Reverse to the same size as Name, and then just assign directly to the indexes, rather than using operator+=.
-The std::reverse function already exists.

[ArmlessBastard]

-The std::reverse function already exists.

I also have an error with in the function I am using, what is this reverse function that exists already?

The error is illegal indirection for the function in the loop listed above.

Also using the instructions above worked but I got a warning for converting the string_t into int should I change that variable, I really don't see how it could effect the program.

[GCDEF]

I also have an error with in the function I am using, what is this reverse function that exists already?

The error is illegal indirection for the function in the loop listed above.

It's what everybody's been telling you. Implement the changes suggested and try again.

[ArmlessBastard]

@GCDEF
I did what you guys said. It fixed the first error the next error happens a few lines below

*Reverse=*Reverse+*Name[i];

The error happens on this line and says that there is illegal indirection.

[ZuK]

Don't know what Reverse is. Let's assume its a pointer to string as well .
Then it would be

*Reverse=*Reverse+(*Name)[i];

or simpler

*Reverse+=(*Name)[i];

Kurt

[ArmlessBastard]

Ah, I tried putting () on both reverse and name. Thanks for the help.