0

以下のコードは次のことを行う必要があります。モジュールを受講しているすべての学生をリストし、自分の点数と全体の平均点数を示します

想定どおりの動作をしますが、問題は、出力にマークが表示されず、代わりに登録番号が表示されることです。

マイマークテキストファイル

11  IS1S01  25 
11  SE1S02  50 
11  SE2S04  75 
12  CS3S08  15 
12  CS1S03  20
13  CS1S03  25 
14  CS1S03  50

次を表示する場所：

11   IS1S01   25

Average = 25

プログラムを実行すると、代わりに次のように表示されます。

11   IS1S01    11

Average = 1.1

なんらかの理由でマークを読み取れません。なぜそれをしているのかわからないようです

void studentmark()
    {
        float Marks;
        float Average;
        float EnrolmentNo;
        std::string module;

    // Display message asking for the user input
    std::cout << "\nList all students taking a module, showing their marks." << std::endl;

    // List of options
    std::cout << "\nInformation Resource Engineering:   IS1S01" << std::endl;
    std::cout << "C++ Programming:          SE2S552" << std::endl;
    std::cout << "Data Structures and Algorithms:       CS2S504 " << std::endl;
    std::cout << "AI for Game Developers:           CS3S08" << std::endl;
    std::cout << "Cognitive Science:            MS3S28" << std::endl;
    std::cout << "Game Modification:            CS1S03" << std::endl;
    std::cout << "Building:             BE1S01" << std::endl;
    std::cout << "Plumbing:             BE2S01" << std::endl;
    std::cout << "Coaching:             SS1S02" << std::endl;
    std::cout << "Psychology:               CC1S04" << std::endl;
    std::cout << "Mental Health Care:           SS2S01" << std::endl;
    std::cout << "Missing Module:               SE1S02" << std::endl;

    std::cout << "\nEnter your preferred module:";

    // Read in from the user input
    std::cin >> module;

    // Read from text file and Display list of marks by each student in a particular module

    std::ifstream infile;               // enable to open, read in and close a text file

    infile.open("Mark.txt");            // open a text file called Mark

    if (!infile)                 
    {
        std::cout << "List is empty" << std::endl;      // if the file is empty it output the message
    } 
    else 
    {
        std::cout << "\nList of marks for students: " << std::endl;

        std::string moduleCode;

        while (infile) 
        {
            infile >> EnrolmentNo >> moduleCode >> Marks;
            if (infile) 
            { 
                // strings read correctly
                if (moduleCode == module) 
                {
                    //std::cout << " ";
                    std::cout << "\nEnrolmentNo" << "       " << "Module Code" << " " << "Marks" << std::endl;

                    int sum=0;
                    for(int i=0;i<10;i++)
                    {
                        if(infile>>Marks)
                        {
                            sum+=Marks;
                        }
                    }

                    std::cout << EnrolmentNo << "           " << moduleCode << "        " << Marks << std::endl;

                    float Average = (double)sum/10.0;

                    std::cout << "\nThe average mark of the module: " << Average << std::endl;
                }
            }
        }
    }

    infile.close();         // close the text file  
    system ("PAUSE");
}
4

1 に答える 1

2

読み取りコードの流れは奇妙です。EnrolmentNo、moduleCodeおよびを読むことから始めMarksます。次に、それが興味のあるモジュールである場合、非整数が見つかるまでMarks 、ファイル内の同じ場所から読み取りを開始します。

あなたが与えたサンプルファイルで（そしてユーザーがIS1S01として入力すると仮定してmodule）、読み取りは次のように進みます：

  1. EnrolmentNo最初の 11 を取得します
  2. moduleCodeIS1S01を取得
  3. Marks25を取得します

次に、内側のループが開始されます。

  1. Marks次の 11 を取得します (2 行目から)
  2. Marks「SE1S02」を読み込もうとして失敗するため、ループは終了します

あなたが示したファイルを考えると、おそらく内側のforループを取り除き、外側のループで合計/平均を実行する必要があります。

于 2012-12-13T13:30:17.300 に答える