出力が変化

Question

私は、次のコードを持っている：

それは正しい出力を与える

#include<iostream> 
#include <string.h> 
using namespace std; 
int main(){ 
     char str[100], str1[100]; 
     cin>>str>>str1; 
     char c[strlen(str1)], flag[strlen(str1)]; 
     int i, j, k = 0; 
     for (i = 0; i < strlen(str1); i++) { 
      flag[i] = 0; 
     } 
     for (i = 0; i < strlen(str); i++) { 
      for (j = 0; j < strlen(str1); j++) { 
       if (str[i] == str1[j] && flag[j] == 0) { 
        c[k] = str[i]; 
        k++; 
        flag[j] = 1; 
        break; 
       } 
      } 
     } 
     if (k != 0) 
      cout<<c; 
     return 0; 
} 

（最初の行が入力された）：

hello world 
lo 

をしかし、それはする必要があり出力

#include<iostream> 
#include <string.h> 
using namespace std; 
char * func(char * str, char * str1){ 
    char c[strlen(str1)], flag[strlen(str1)]; 
    int i, j, k = 0; 
    for (i = 0; i < strlen(str1); i++) { 
     flag[i] = 0; 
    } 
    for (i = 0; i < strlen(str); i++) { 
     for (j = 0; j < strlen(str1); j++) { 
      if (str[i] == str1[j] && flag[j] == 0) { 
       c[k] = str[i]; 
       k++; 
       flag[j] = 1; 
       break; 
      } 
     } 
    } 
    if (k != 0) 
     return c; 
    return NULL; 
} 
int main(){ 
     char str[100], str1[100]; 
     cin>>str>>str1; 
     cout<<func(str,str1); 
     return 0; 
} 

- 私はこのような、関数内のコードを入れて機能func()の内側に置くこと第二の出力ラインは、ASCII文字 ここでのミスは何が含まれていることを

hello world 

ノートのように来ますか？このような文字列配列を返すことはできませんか？

Answer 1

あなたは

return c; 

あなたがスコープの外に出るのローカル変数を返しているときに、それ以外の場合は、func()で、変更

static char c[100]; 
char flag[strlen(str1)]; 

で

char c[strlen(str1)], flag[strlen(str1)]; 

をお試しください。あなたがいるとき

​​

動作は未定義です。

あなたはcを変換する場合はfunc()によって返さchar *はまだfunc()外に生きていると問題なく使用することができ、static変数です。

Answer 2

関数からローカル変数 'c​​'を返しています。プロセッサーが機能から復帰するとすぐに破棄されます。関数内の値を出力するか、宛先文字列を渡してコピーします。

charの代わりに 'string'クラスを使用することをおすすめします。ここ

あなたはまた、staticとして 'C' を宣言することによって解決することができます

#include<iostream> 
#include <string.h> 
using namespace std; 
string func(char * str, char * str1){ 
    char c[strlen(str1)], flag[strlen(str1)]; 
    int i, j, k = 0; 
    for (i = 0; i < strlen(str1); i++) { 
     flag[i] = 0; 
    } 
    for (i = 0; i < strlen(str); i++) { 
     for (j = 0; j < strlen(str1); j++) { 
      if (str[i] == str1[j] && flag[j] == 0) { 
       c[k] = str[i]; 
       k++; 
       flag[j] = 1; 
       break; 
      } 
     } 
    } 
    string s = c; 
    if (k != 0) 
     return s; 
    return NULL; 
} 
int main(){ 
     char str[100], str1[100]; 
     cin>>str>>str1; 
     cout<<func(str,str1); 
     return 0; 
} 

を行きます。しかし、 'c'の長さはハードコード化されるべきです。