字符串代碼不工作

Question

使用C++語言，函數LetterChanges(str)採用str參數傳遞並使用以下算法對其進行修改。

將字符串中的每個字母替換爲 字母表中的後面的字母（即c變成d，z變成a）。然後在這個新的字符串（a，e，i，o，u）中大寫每個元音 ，並最終返回這個修改後的 字符串。

#include <iostream> 
using namespace std; 

string LetterChanges(string str) { 

    // code goes here 
    string str2 = "abcdefghijklmnopqrstuvwxyz"; 
    int j; 
    for (int i = 0; str[i] != '\0'; i++) { 

     for (j = 0; str2[j] != '\0'; j++) { 

      if (str[i] == str2[j]) { 

       str[i] = str2[j + 1]; 

      } 
     } 
    } 
    for (int i = 0; str[i] != '\0'; i++) { 
     if (str[i] == 'a') { 
      str[i] = 'A'; 
     } 
     else if (str[i] == 'e') { 
      str[i] = 'E'; 
     } 
     else if (str[i] == 'i') { 
      str[i] = 'I'; 
     } 
     else if (str[i] == 'o') { 
      str[i] = 'O'; 
     } 
     else if (str[i] == 'u') { 
      str[i] = 'U'; 
     } 
    } 
    return str; 
} 
int main() { 
    // keep this function call here 
    cout << LetterChanges(gets(stdin)); 
    return 0; 
} 

我試圖運行此代碼，但它是不是給的願望輸出請幫助..

Answer 1

的功能

讓我們先從你的第一個循環：

for (int i = 0; str[i] != '\0'; i++) { 
    for (j = 0; str2[j] != '\0'; j++) { 
     if (str[i] == str2[j]) { 
      str[i] = str2[j + 1]; 
     } 
    } 
} 

這裏有幾件事。首先，我們甚至不需要處理str2。用C++人物使用ASCII編碼，這意味着我們實際上可以這樣做str[i]++改變一個「A」到「B」或「E」到「F」，等...

另外，我想建議不要使用str[i] != '\0'。我們使用標準庫字符串而不是C字符串是有原因的，所以我們不妨使我們的生活更輕鬆並使用str.size()。沿着這些相同的路線，我會建議str.at(i)而不是str[i]，因爲前者會爲我們做邊界檢查。

最後，如果您包含cctype，那麼我們可以使用isalpha函數來確保我們只修改字母字符（無數字或空格等）。

因此，你的第一環就呈現：

for (int i = 0; i < str.size(); i++) { 
    if(isalpha(str.at(i)){ 
     if(str.at(i) == 'z') str.at(i) = 'a'; //special case 
     else str.at(i)++; 
    } 
} 

至於你的第二個循環，你甚至都不需要它！我們實際上可以將所有內容直接納入第一個。只要我們確保在之後做元音修改，我們已經更改了單個字母。

也可以用一些ASCII數學轉換從小寫到大寫。小寫字母和大寫字母之間的區別是'A'-'a'，所以如果我們將其添加到任何小寫字母，它會給我們它的大寫版本！

有了這一切，我們可以修改代碼以：

for (int i = 0; i < str.size(); i++) { 
    if(isalpha(str.at(i)){ //Make sure it's a letter! 
     if(str.at(i) == 'z') str.at(i) = 'a'; //special case 
     else str.at(i)++; 

     if(str.at(i) == 'a' | str.at(i) == 'e' | str.at(i) == 'i' 
      | str.at(i) == 'o' | str.at(i) == 'u') { 
      str.at(i) += 'A' - 'a'; 
    } 
} 

你的主要

這裏只有一件事要在這裏解決。請勿使用gets進行輸入。如果您要查找單個單詞，請使用提取操作符>>，或者如果您想要整行，請使用getline。

string word, line; 

getline(cin, line); 
cin >> word; 

cout << LetterChanges(line) << endl; 
cout << LetterChanges(word) << endl;