需要幫助調試我的代碼，以反轉字符串中的單詞

Question

我需要一些幫助來調試我的代碼。此代碼旨在將字符串中的單詞轉換爲句子形式[假設該字符串沒有「。」。最後]。出於某種原因，我得到的輸出是縮進的輸出，在第一個單詞之後加上一個額外的空格以及縮進的輸出減去第一個單詞。我是編碼方面的初學者;所以如果可能的話，我希望更易於理解的解決方案，或者使用循環，字符串和數組的解決方案。

樣品輸入：

My name is Edward 

預期輸出：

Edward is name My 

輸出接收：

Edward is name 

這裏是我到目前爲止的代碼：

#include <iostream> 
#include <string> 
#include <stdio.h> 

using namespace std; 

int main() { 

string s, n, a; 
getline(cin, s); 

for (int i = s.length(); i >= 0; i--){ 
    if (s[i] != 32) { 
     n += s[i]; 
    } 
    else { 
     for (int j = n.length() -1; j >= 0; j--){ 
      a += n[j]; 
     } 
     cout << a << ' '; 
     n.clear(); 
     a.clear(); 
    } 
} 

cin.ignore(); 
getchar(); 
return 0; 

} 

另外，我剛剛注意到最後還有一個額外的空間。如果有辦法可能取消輸出最後一個空格;請告訴我。

感謝您的閱讀，感謝您的幫助。

Answer 1

編輯︰感謝您的意見和答案的幫助下，我解決了額外的空間問題，並在最後增加了一些東西，輸出最後的單詞。這不是完美的，但它的工作原理。 :)

#include <iostream> 
#include <string> 

using namespace std; 

int main() { 

string s, n; 
getline(cin, s); 

for (int i = s.length() -1; i >= 0; i--){ 
if (s[i] != 32) { 
    n += s[i]; 
} 
else { 
    for (int j = n.length() -1; j >= 0; j--){ 
     cout << n[j]; 
    } 
    cout << ' '; 
    n.clear(); 
} 
} 

for (int k = n.length() -1 ; k >= 0; k--) 
cout << n[k]; 

cin.get(); 
return 0; 

} 

Answer 2

正如我在評論中提及，你倒車的字符整個字符串，但您需要拆分單詞和反向：

#include <iostream> 
#include <string> 
#include <sstream> 
#include <vector> 
#include <algorithm> 

using namespace std; 

int main() { 

    string s, n; 
    getline(cin, s); 
    std::istringstream iss(s); 
    std::vector<string> words; 
    while(iss >> n) { 
     words.push_back(n); 
    } 

    std::reverse(words.begin(),words.end()); 

    for(auto word : words) { 
     std::cout << word << ' '; 
    } 

    getchar(); 
    return 0;  

} 

Live Demo

Answer 3

可以使用strrev();函數，而不是全部您的for區塊。

Answer 4

所以這實際上只是從πάντα ῥεῖ's excellent answer抽象的額外步驟。您可以使用istream_iterator和ostream_iterator來進一步簡化您的代碼。

整個代碼回答你的問題可以歸結爲：

const vector<string> words{ istream_iterator<string>(cin), istream_iterator<string>() }; 

copy(crbegin(words), crend(words), ostream_iterator<string>(cout, " ")); 

Live Example