2017-02-12 154 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++) { 
       .... 
      } 
      .... 
    }