Thread: Help!! How to return a pointer to beginning of string??

I'm sorry- but this is for an assignment and I am stuck.. I am trying to convert a string into uppercase..and then return a pointer to the beginning of the string in main..and then somehow copy the pointer into a string again.. when I try to compile I get the following error message..
error C2440: '=' : cannot convert from 'char *' to 'char [35]'
There are no conversions to array types, although there are conversions to references or pointers to arrays

I'm not going to post the entire program because it's lengthy...and am hoping that somene could point me in the right direction from the pieces below..

Any suggestions or help would be greatly appreciated!!

lms

Code:

//function prototype
char* toUpperString(char[]);

//function call from main
char nameString[35];

cout<<"Enter name: ";
cin.getline(nameString,35);
nameString=toUpperString(nameString);
strncpy(e->name, nameString, 35);

//function to convert to uppercase..

char* toUpperString(char x[])
{
	int length;
	char* string;

	for (int i=0; i<length; i++)
	{
	string[i]=toupper(x[i]);
	}
	
	return string;
}

Here is a function that will modify the string you pass to it.

Code:

void StringToUpper(char * s)
{
  int length = strlen(s);
  for( int i=0 ; i<length ; i++)
  {
      s[i] = toupper( s[i] );
   }
}
//use like so.
char string1[6] = "hello";
StringToUpper(string1);

If you want to save the original string, make a copy of it, and call the function on the copy

Here is a function that will create a new string that you must later free using delete

Code:

char * StringToUpper(const char * s)
{
   int length = strlen(s);
   char * ret = new char[length];
   for( int i=0 ; i<length ; i++)
   {
      ret[i] = toupper(s[i]);
   }
  return ret;
}
//use like so
char string1[6] = "hello";
char * string2 = StringToUpper(string1);
//before the end of main do this
delete [] string2;

This version does NOT modify the original string, but you must delete the memory, before the var goes out of scope. I think this is a little messy, the first version is better.

Originally posted by lms

Code:

//function prototype
char* toUpperString(char[]);

//function call from main
char nameString[35];

cout<<"Enter name: ";
cin.getline(nameString,35);
nameString=toUpperString(nameString);
strncpy(e->name, nameString, 35);

//function to convert to uppercase..

char* toUpperString(char x[])
{
	int length;
	char* string;

	for (int i=0; i<length; i++)
	{
	string[i]=toupper(x[i]);
	}
	
	return string;
}

well, you can do this:

Code:

>>char* toUpperString(char[]);
char toUpperString(char*x);

-edited version of toUpperString:
char toUpperString(char *x)
{
int length;
char* string;
for (int i=0; i<length; i++)
{
string[i]=toupper(x[i]);
}
return string;
}

question: do you want to make a copy of the string?
if not, you can save memory by removing that extra pointer.

Regards,
toaster

I'm still kind of confused..I'll keep trying to figure it out..