c++ while循环读取输入

我提交了这个程序，它工作得很好，但是我的老师说我的while循环有问题，即使我得到了正确的答案。有什么建议或帮助吗?

当到达文件结束并且第43行上的read无效时，在while循环中会发生什么?你的程序是结构化的，你看不到问题，但问题确实存在。应该重新构建while循环来考虑这一点。

#include<iostream>
#include<fstream>
using namespace std;
int main()
{
    ifstream inputfile;
    char choice;

    int NumberOfIntegers = 0,
        SumOfIntegers = 0,
        Average = 0 ,
        LargestValue,
        SmallestValue,
        integer;
    inputfile.open("random.txt");
    if(!inputfile)
    {
        cout << "the file could not be open" << endl;
    }

    inputfile >> integer;
    //initialize smallest and largest
    SmallestValue = integer;
    LargestValue = integer;
    while(inputfile)
    {
        NumberOfIntegers++;
        SumOfIntegers = SumOfIntegers + integer;
        inputfile >> integer;
        if( integer > LargestValue || integer < SmallestValue)
        {
            if ( integer > LargestValue)
                LargestValue = integer;
            else 
                SmallestValue = integer;
        }
    }
    if(NumberOfIntegers > 0 )
    {
        Average = SumOfIntegers / NumberOfIntegers;
    }
    //closing input file
    inputfile.close();
    do
    {
        //Display Menu
        cout << "Make a selection from the list" << endl;
        cout << "A.   Get the largest Value" << endl;
        cout << "B.   Get the smallest Value" << endl;
        cout << "C.   Get the sum of the values" << endl;
        cout << "D.   Get the average of the values" << endl;
        cout << "E.   Get the number of values entered" << endl;
        cout << "F.   End this program" << endl << endl;
        cout << "Enter your choice -->  ";
        cin >> choice;
        cout << endl;
        switch (choice)
        {
        case 'a':
        case 'A': cout << "The largest value is " << LargestValue << endl;
            break;
        case 'b':
        case 'B': cout << "The smallest value is " << SmallestValue << endl;
            break;
        case 'c':
        case 'C': cout << "The sum of the values entered is " << SumOfIntegers << endl;
            break;
        case 'd':
        case 'D': cout << "The average of the values entered is " << Average << endl;
            break;
        case 'e':
        case 'E': cout << "The number of values entered is " << NumberOfIntegers << endl;
            break;
        case 'f':
        case 'F': cout << "Program ending" << endl << endl;
                cin.ignore();
                cout << "nnPress Enter to end -->  ";
                cin.ignore();
                return 0;

        default: 
            cout << choice << " is an invalid value. " << endl;
        }
        cout << endl;
    } while( choice != 'f' || choice != 'F');

    return 0;
}