計數兩個字符序列

-2

我也是新的C++，但有其他語言的經驗。計數兩個字符序列

我工作過：

書： C++入門第五版。
練習： 5.12

當中計數元音，空格和使用開關結構的運動還要求標籤來跟蹤這兩個字符多少次序列ff，fl，並fi出現在某些輸入的文本。我已經看到了這個問題的其他解決方案，大多數使用布爾標誌類型的結構，但我選擇使用字符串迭代器來跟蹤。不過，對於C++來說，我並不確定是否存在對我的代碼（即指向非有效對象的迭代器）有固有危險的內容。這段代碼看起來好嗎？

#include <iostream> 
#include <string> 

using std::string; 

using std::cin; 
using std::cout; 
using std::endl; 

int main() 
{ 
    string text; 
    unsigned aCnt = 0, eCnt = 0, iCnt = 0, oCnt = 0, uCnt = 0, spaceCnt = 0, nlCnt = 0, tabCnt = 0, ffCnt = 0, flCnt = 0, fiCnt = 0; 
    bool flag = false; 

    while (getline(cin, text)) 
     for (auto it = text.begin(); it != text.end(); ++it) { 

      *it = tolower(*it); 
      switch (*it) { 

      case 'a' : 
       ++aCnt; 
       break; 
      case 'e' : 
       ++eCnt; 
       break; 
      case 'i': 
       if (it != text.begin()) 
        if (*(it - 1) == 'f') 
         ++fiCnt; 
        else 
         ++iCnt; 
       break; 
      case 'o' : 
       ++oCnt; 
       break; 
      case 'u' : 
       ++uCnt; 
       break; 
      case ' ': 
       ++spaceCnt; 
       break; 
      case '\n': 
       ++nlCnt; 
       break; 
      case '\t': 
       ++tabCnt; 
       break; 
      case 'f' : 
       //Control strucutre that checks if the character pointed to previously was an f. 
       if (it != text.begin()) 
        if (*(it - 1) == 'f') 
         ++ffCnt; 
       break; 
      case 'l': 
       if (it != text.begin()) 
        if (*(it - 1) == 'f') 
         ++flCnt; 
       break; 
      } 

     } 

    cout << "The number of 'a' vowels is: \t" << aCnt << endl; 
    cout << "The number of 'e' vowels is: \t" << eCnt << endl; 
    cout << "The number of 'i' vowels is: \t" << iCnt << endl; 
    cout << "The number of 'o' vowels is: \t" << oCnt << endl; 
    cout << "The number of 'u' vowels is: \t" << uCnt << endl; 
    cout << "The number of tabs read is: \t" << tabCnt << endl; 
    cout << "The number of newlines is: \t" << nlCnt << endl; 
    cout << "The number of spaces read is: \t" << spaceCnt << endl; 
    cout << "The number of 'ff' read is: \t" << ffCnt << endl; 
    cout << "The number of 'fl' read is: \t" << flCnt << endl; 
    cout << "The number of 'fi' read is: \t" << fiCnt << endl; 


    return 0; 
}

來源

2017-08-16 MattE93

我認爲這屬於上[codereview.se。 –

我可能會使用'std :: map '來計算出現次數，以避免大量奇怪命名的變量 – user463035818

雖然沒有錯，但我建議在'while'循環中添加'{}'。這是有效的，因爲它只是一個陳述，但它可能導致混亂，因爲'for'循環裏面有多少行。 –

如果您需要隨時修改容器，多次訪問元素或以非線性方式遍歷容器，則應使用正常的for循環或其一個表親。

基於範圍的for適用於您需要訪問容器的每個元素一次。您多次訪問某些元素可能會導致問題。

您應該使用const_iterator作爲@SergeyA提到的。而insted的修改*it做

switch (tolower(*it))

或者其存儲到temp變量。

還不錯的做法是使用map instad計數器，每char或string或什麼的。的

std::map<char,unsigned> charCounter; 
charCounter['a']++;

，而是10printf()的

for(auto const & c : charCounter) 
{ 
    for(auto const & i : charCounter.second) 
    { 
     cout << "The number of " << c << " vowels is: \t" << i << endl; 
    } 
}

來源

2017-08-16 20:03:55

*it = tolower(*it);正在修改字符串。我不認爲這是你想要做的事情，因爲通常情況下，計數會假設未修改的傳入序列。我建議你使用const_iterator來防止你自己處理這些問題。

來源

2017-08-16 19:42:18 SergeyA

計數兩個字符序列

回答

相關問題