Функция неправильно возвращает строку(С++)

[tolikprankster]

Функция Slovo(str,c) проверяет строку "str" на наличие слов начинающихся на символ 'c'. Затем выводить эти слова прямо в функции.
ниже приведенная программа выдает резултат: "capital".

char *slovo(char *str, char c);
int main()
{	slovo(" london is a capital of great britain",'c');
	//cout<<slovo(" london is a capital of great britain",'c');

getch();
return 0;
}
char *slovo(char *str,char c)
{char ss[20]="";
char ch[2];
	for(;*str;str++)
	{
		if ((*str==c)&&(*(str-1)==' '))
		while (((*str)!=' ')||(!(*str)))
		{	ch[0]=*str;
			ch[1]='\0';
			strcat(ss,ch);
			str++;
		}
	}
	cout<<ss;
	return ss;
}

ОДНАКО если выводить возвращаемое функцией значение:

char *slovo(char *str, char c);
int main()
{	//slovo(" london is a capital of great britain",'c');
	cout<<slovo(" london is a capital of great britain",'c');

getch();
return 0;
}
char *slovo(char *str,char c)
{char ss[20]="";
char ch[2];
	for(;*str;str++)
	{
		if ((*str==c)&&(*(str-1)==' '))
		while (((*str)!=' ')||(!(*str)))
		{	ch[0]=*str;
			ch[1]='\0';
			strcat(ss,ch);
			str++;
		}
	}
	//cout<<ss;
	return ss;
}

Результат работы первой и второй программы:
http://www.programmersforum.ru/attac...1&d=1360457645
Почему так?

[Hemul]

Тут дело с временем жизни локального массива ss. Когда функция завершается, все локальные переменные удаляются. Вы же возвращаете указатель на действующий в функции указатель, но он уже указывает на мусор в функции main. Лучше передавайте в качестве аргумента указатель на другой массив, в который будет записываться результат

[tmtm]

#include <iostream.h>
#include <string.h>
#include <stdlib.h>
#include <conio.h>
char *slovo(char *str, char c);
int main()
{ cout<<slovo("c) london is a capital of great britain, companion",'c')<<endl;
  cout<<slovo("capital",'c')<<endl;
getch();
return 0;
}
char *slovo(char *str,char c)
{char* ss=new char[30];
 ss[0]='\0';
 char *s=&str[0];
 char *v=&ss[0];
       for(;*s;)
	{
	  if (*s==c)
          {if ((s==str) || (*(s-1)==' '))
	     {if (*ss) {ss[strlen(ss)]=' ';  v++;}
               { while ((*s!=' ') && (*s))
	         {*v=*s;
                  s++; v++; }
                *v='\0';
               }
             }
           }
          if (*s) s++;
       }
	return ss;
}

[tolikprankster]

всем спасибо исправил:

void slovo(char *str, char c,char *a);
int main()
{	char a[30]="";
	slovo("cs london is a capital of great britain",'c',a);
	//cout<<slovo(" london is a capital of great britain",'c');
	cout<<a;
getch();
return 0;
}
void slovo(char *str,char c,char *a)
{char ss[20]="";
char ch[2];
	for(;*str;str++)
	{
		if ((*str==c)&&(*(str-1)==' '))
		while (((*str)!=' ')||(!(*str)))
		{	ch[0]=*str;
			ch[1]='\0';
			strcat(a,ch);
			str++;
		}
	}
	cout<<ss;
}

[Hemul]

Сообщение от tmtm

char* ss=new char[30];

Вы динамически выделяете память, но нигде ее не освобождаете. Нехорошо