簡單的C++函數

Question

好了，所以我確定即時通訊做一些愚蠢的事：d

我有一個函數：

int wordFunc(string a){ 
    std::ifstream inp; 
    inp.open(a, std::ios::in); 
    if (inp.is_open()){ 
     std::string word; 
     unsigned long wordCount = 0; 
     while(!inp.eof()){ 
      inp >> word; 
      while(word.length() > 0){ 
       wordCount++; 
      } 
      inp.close(); 
     } 
     return wordCount; 
    } 
} 

字符串是用戶輸入file.txt的 - 其設置爲C：\轉儲\ user.txt現在

當我調用代碼：

int main(){ 
    string file; 
    int words = 0; 
    file = "C:\\Dump\\user.txt"; 

    int a = wordFunc(file, words); 
    cout << "Words: " << a << endl; 

    return 0; 
} 

控制檯剛剛停止 - 我havnt編碼任何我ñC++在很多年，所以即時通訊生鏽 - 任何幫助？

編輯 有了某種幫助高度重視和我結束了會是這樣

unsigned long wordFunc(const std::string& a){ 
    std::ifstream inp(a); 
    system("cls"); 
    unsigned long wordCount = 0; 
    std::string word; 
    while(inp >> word) 
    { 
     wordCount++; 
    } 
    return wordCount; 
} 

對於功能 - 應該已經發布了更新

Answer 1

你的問題是這樣的：

 while(word.length() > 0){ 
      wordCount++; 
     } 

這將永遠循環。你可能的意思是

 if(word.length() > 0){ 
      wordCount++; 
     } 

Answer 2

你有很多問題。

---

至於另一個海報評論說，該行：

while (word.length() > 0) 

將永遠循環下去，你需要將其更改爲：

if (word.length() > 0) 

---

您混用整數和無符號長整型不當。 wordCount和a變量以及來自wordFunc()的返回值應該是相同的。

---

你有inp.close()內讀取循環，而不是它屬於外面的位置。這意味着它將在處理第一行後關閉文件。

---

你也有if聲明意味着你有一個語法錯誤，在一個執行路徑（就是文件無法打開）不返回任何內部return語句。

您需要更換return和後面的右大括號。

這也意味着wordCount變量也必須聲明在if語句之外（在函數的頂層）。

---

我覺得fstream::open()需要char*而不是字符串，所以你應該重新編寫它：

inp.open (a.c_str(), std::ios::in); 

---

您的來電wordFunc()，有兩個參數，不匹配的原型，只有一個。

---

有一些外部變量，如words在main()。

---

最後字被計算兩次因爲一旦你嘗試讀取超越文件末尾的EOF標誌只設置。您可以修復與簡單的修改到if聲明遞增wordCount：

if ((!inp.eof()) && (word.length() > 0)) { 

---

與所有這些進行更改，則結束：

#include <iostream> 
#include <fstream> 

unsigned long wordFunc (std::string str) { 
    unsigned long wordCount = 0; 
    std::ifstream inp; 

    inp.open(str.c_str(), std::ios::in); 
    if (inp.is_open()) { 
     std::string word; 
     while (!inp.eof()) { 
      inp >> word; 
      if ((!inp.eof()) && (word.length() > 0)) { 
       wordCount++; 
      } 
     } 
     inp.close(); 
    } 
    return wordCount; 
} 

int main() { 
    std::string file; 
    file = "user.txt"; 
    unsigned long a = wordFunc(file); 
    std::cout << "Words: " << a << std::endl; 
    return 0; 
} 

Answer 3

C++已經擁有的功能在標準庫做大部分的事情，所以你可以做類似的事情：

#include <fstream> 
#include <iostream> 
#include <algorithm> 
#include <iterator> 
#include <string> 

int main() { 
    std::ifstream in("user.txt"); 

    std::cout << "Words: " 
       << std::distance(std::istream_iterator<std::string>(in), 
           std::istream_iterator<std::string>()); 
    return 0; 
} 

得到三個可執行語句需要五個頭文件的愛代碼！

如果你堅持做自己的計數，我會做一些事情，如：

std::ifstream in("user.txt"); 

std::string word; 

while (in >> word) 
    ++wordCount; 

std::cout << "Words: " << wordCount;