2017-02-12 171 views
-1
#include <iostream> 
#include <stdlib.h> 
#include <stdio.h> 
#include <string> 
using namespace std; 

string compress_string(string input_string) 
{ 
    string output_string; 
    int j=0; 
    for (int i=0; i < input_string.length();i++,j++) 
    { 
      output_string[j] = input_string[i]; 
      cout<<"output string at "<<j<<"is"<<output_string[j]; 
      int count =1; 
      for (int k = i; k < input_string.length();k++) 
      { 
       if (input_string[i+1] == input_string[i]) 
       { 
        count++; 
        i++; 
       } 
       else 
       { 
        break; 
       } 
      } 
      j++; 
      cout<<"count is"<<count<<"\n"; 
      output_string[j]=count+'0'; 
      cout<<"String till now is "<<output_string<<"\n"; 
    } 
    output_string[j]='\0'; 
    cout<<"Output string:"<<output_string<<"\n"; 
    printf("output string is %s\n",output_string.c_str()); 
    return output_string; 
} 

int main() { 
    // your code goes here 
    string input_string; 
    cout << "Enter the string"; 
    cin >> input_string; 
    cout<<"Compressed output is"<<compress_string(input_string); 
    return 0; 
} 

我试过了上面的程序,发现程序末尾的压缩输出没有打印出来。
我看看,如果我使用c_str(),它打印在printf的,但不知何故COUT不打印在主函数压缩输出和compress_string功能。无法在C++中打印字符串

+1

请比“它不工作”更具体。 – GhostCat

+0

你得到的实际错误是什么? – Shravan40

+1

'output_string'为空。 'output_string [j]'通过缓冲区溢出展示了所有'j> 0'的未定义行为。 –

回答

2
string output_string; 

output_string这里是空的。 访问下面的元素是未定义的行为。

output_string[j] = input_string[i]; 

resize到的input_string长度.The可以访问它的各个元件;

output_string.resize(input_string.length()); 
0

你必须减少字符串长度第一for循环到字符串长度的范围内 - 因为这input_string[i+1] == input_string[i]i+1检查1。

而且output_string是空的,你可以不使用索引来访问它,这是一个不确定的行为。

for (int i = 0; i < input_string.length() - 1; i++,j++) { 
    ..... 
      for (int k = i; k < input_string.length();k++) { 
       .... 
      } 
      .... 
    }