使用功能讀取/寫入文件

Question

嗨我正在一個簡單的工資申請。以下代碼包含使用switch語句的兩個選項菜單。該代碼還鏈接到名爲「shop-account」的文本文件。該文件僅包含值100.

對於選項1，用戶應該能夠從文件中轉移金額。用戶應該能夠進行儘可能多的轉帳，而不用透支帳戶。代碼應該能夠輸出當前餘額。我相信我假設要使用void函數，但我以前從未使用它，並且真的很掙扎。我希望有人可以看看代碼，並告訴我哪裏出錯了。謝謝

int read_balance (void); 
void write_balance (int balance); 

using namespace std; 
int _tmain(int argc, _TCHAR* argv[]) 
{ 

    int selection; 
    int total; 
    int balance; 
    int NewAmount; 

    do { 
      cout << "1. Transfer an amount" <<endl; 
      cout << "2. Quit" << endl; 
      cout << "Please enter menu number"<<endl; 
      cin >> selection; 

      switch(selection) 
      { 
       case 1: 
        cout << "You have choosen to transfer an amount" << endl; 
        cout << "How much do you wish to transfer from the shop account?"<<endl; 
        cin >> NewAmount; 
        balance -= NewAmount; 
        write_balance (balance); 
        cout << balance << endl; 
        break; 

       case 2: 
       return 0; 
       break; 

       default: 
       cout << "Ooops, invalid selection!" << endl; 
       break; 
       } 

     }while(selection != 2); 

    system("pause"); 
    return 0; 
} 


int read_balance (void) 
{ 
    fstream f; 
    f.open("shop-account.txt"); 
    f >> balance; //error: "balance" is unidentified 
    f.close(); 
    return balance; 
} 


void write_balance (int balance) 
{ 
    fstream f; 
    f.open("shop-account.txt"); 
    f << balance; 
    f.close(); 
} 

Answer 1

你的功能沒有平衡變量聲明！你必須在你的函數中添加一個平衡的聲明

int read_balance(void) 
{ 
    fstream f; 
    f.open("shop-account.txt"); 

    int balance; 
    f >> balance; //now it's defined 
    f.close(); 
    return balance; 
} 

Answer 2

1. 與其他人一樣提到的，你沒有申報在正確的範圍內int balance（函數級別範圍，在這種情況下）。

   事實上，它看起來像你忘記打電話read_balance產品總數，讓您的balance計算中使用的不確定的值，這是Undefined Behaviour。

2. 接下來：在你使用變量的地方聲明你的變量，這樣當你決定將代碼片段提取到子函數中時，可以防止這種情況發生，並且可以更容易地看到變量的使用位置。因此，更容易看到代碼是否正確。

3. Next：錯誤處理。如果你沒有，你的程序是毫無價值的。事實上，固定上述問題，即使，

   • 只需輸入無效輸入一次會運行一個循環，並只保留從其減去從賬戶餘額不定值和寫作這些錯誤的價值觀到磁盤。這可能不是你想要的。

   • 使得shop-account.txt只讀足以節目愚弄到

     • 傳輸無限量的，而無需更新具有文件中平衡的錯誤觀念文件
     • （因爲它從來沒有檢查）

這裏是一個清理的版本，進行最低限度的檢查，並添加一個選項來「檢查帳戶餘額」。似乎很有用。

看到它Live On Coliru

我希望一些這有助於。首先，我希望你的老師會提到大部分內容：/

int read_balance(void); 
void write_balance(int balance); 

#include <iostream> 
#include <limits> 

int main() 
{ 
    while(std::cin.good()) 
    { 
     std::cout << "0. Request balance"  << std::endl; 
     std::cout << "1. Transfer an amount" << std::endl; 
     std::cout << "2. Quit"     << std::endl; 
     std::cout << "Please enter menu number" << std::endl; 

     int selection = 2; 
     if(std::cin >> selection) 
     { 
      std::cout << "DEBUG: selection:" << selection << "\n"; 
      switch(selection) 
      { 
      case 0: 
       std::cout << "The current account balance is " << read_balance() << std::endl; 
       break; 

      case 1: 
       { 
        std::cout << "You have choosen to transfer an amount" << std::endl; 
        std::cout << "How much do you wish to transfer from the shop account?"<<std::endl; 

        int amount = 0; 
        if (std::cin >> amount) 
        { 
         std::cout << "DEBUG: amount:" << amount << "\n"; 
         int balance = read_balance(); 
         if(amount<=0) 
         { 
          std::cout << "Amount must be positive\n"; 
         } 
         else if(balance < amount) 
         { 
          std::cout << "Insufficient funds\n"; 
         } 
         else 
         { 
          int new_balance = balance - amount; 
          write_balance(new_balance); 
          std::cout << "New account balance: " << new_balance << std::endl; 
         } 
        } else 
        { 
         // bad input cleared outside the switch 
        } 
       } 
       break; 

      case 2: 
       return 0; 
       break; 

      default: 
       std::cout << "Ooops, invalid selection!" << std::endl; 
       break; 
      } 
     } 

     if(std::cin.eof()) 
     { 
      std::cout << "Bye\n"; 
      return 0; 
     } 
     if(std::cin.fail()) 
     { 
      std::cin.clear(); 
      std::cin.ignore(99999, '\n'); 
      std::cout << "Invalid input\n"; 
      // note eof() can be true here 
     } 
    } 
} 

#include <fstream> 

int read_balance(void) 
{ 
    std::ifstream f; 
    f.exceptions(std::ios::failbit | std::ios::badbit); 
    f.open("shop-account.txt"); 
    int balance; 
    f >> balance; 
    f.close(); 
    return balance; 
} 

void write_balance(int balance) 
{ 
    std::ofstream f; 
    f.exceptions(std::ios::failbit | std::ios::badbit); 
    f.open("shop-account.txt"); 
    f << balance; 
    f.close(); 
}