执行问题流

if语句之后的半隆。另外，if/else逻辑没有仔细排除并处理所有可能的答案选择。最后，我使用了std :: getline（）进行std :: cin的阅读，当您期望从cin中读取多个输入时，它会更好地工作。

#include <iostream>
#include <fstream>
#include <cstdlib>
int main()
{
    char stat, answ;
    float rate;
    std::string fname, lname, phone, address, partial_address;
    std::ifstream potentials("C:\Users\Brandon\Desktop\C++Projects\Lab6\potentials.txt");
    std::ofstream results("C:\Users\Brandon\Desktop\C++Projects\lab6\results.txt");
    if (!potentials) {
        std::cerr << "Unable to open file ";
        return 1;
    }
    while ( true)
    {
        // This 'if' statement removes the while(!potentials.eof()) which created an extra read error
        // There are other ways to handle this, but this was a quick solution.  The read itself will return
        // a false value, but as we are trying to read 4 variables, this was one way of doing it.
        if(!(potentials >> stat &&
        potentials >> fname &&
        potentials >> lname &&
        potentials >>  phone)) {
            std::cout << "End of file." << std::endl;
            return -1;
        }
        // if you get into the habit of putting the constant on the left side of an equality,
        // you won't accidentally confuse '=' with '=='
        if ( 'X' == stat)
        {
            std::cout << "customer is preferredn";
            rate = .079;
        } else if('Y' == stat) {  // I usually only check for the value you are looking for, but this is OK, if you create the catch-all else at the end.
            std::cout << "Customer is not preferredn";
            rate = .129;
        } else {
            std::cout << "File corrupted.  First word in each line must be 'Y' or 'X'" << std::endl;
            return -1;  // exit the program as the file is corrupted
        }
        std::cout << fname << " " << lname << " " << phone << " " << rate << std::endl;
        std::cout << "Is this customer interested in this card? Y or N" << std::endl;
        std::cin >> answ;

        if ('Y' == answ || 'y' == answ)  // Just has to check for lower case as well  You can chuck that if you like
        {
            std::cin.ignore(); // cin has a tendency to skip the next input based on a prior "return" or "enter".  Gobble the new line
            std::cout << "Thank the customer and ask for their current address.n";
            std::getline(std::cin, address);
            std::cout << "Thank the customer for accepting the offer, to expect a card in the mail soon, and happy charging!n";
            results << fname << " " << lname << " " << stat << " " << address << " r " << std::endl;
        } else if('N' == answ || 'n' == answ) {
            // AT this point, we already know that stat is either 'Y' or 'N', so we don't check explicitly on the else
            if ('X' == stat)
            {
                std::cout << "Tempt them with the cash back rewards and to call 18003838383 for a better offer.n";
                results << fname << " " << lname << " " << answ << " " << address << " r" << std::endl;
            } else {
                std::cout << "Tell them thank you anyhow and to have a nice day.n";
                results << fname << " " << lname << " " << answ << " " <<
                        address << " r" << std::endl;
            }
        } else {  // catch all else.  Not my preferred way to do this, but follows your design.
            std::cout << "Invalid response.  Response must be 'Y' or 'X'" << std::endl;
            return -1;  // exit the program as the response was not corrupted
        }
    }
}