Блог

У меня возникли проблемы с рефакторингом этого кода в функции. Что именно я должен ввести для возврата, чтобы завершить эти последние две функции. Они включают циклы с общим и промежуточным итогом с несколькими вычислениями на основе введенного символа. Я понимаю первые два, просто верните объявленную переменную.

 #include <iostream>
#include <iomanip>

using namespace std; // forgive me for being lazy!

double getPrice();
int getNumber();
double saleTotal(double getPrice, int getNumber);
void retail(double getPrice, int getNumber, double saleTotal);

const double TAX_RATE = .05;

int main()
{
   double myPrice = getPrice();

   int myNumber = getNumber();

   double getTotal = saleTotal(myPrice, myNumber);

   retail;

    return 0;
}

double getPrice()
{
    double price;
    cout << "Enter price $";
    cin >> price;
    return price;
}

int getNumber()
{
   int number;
   cout << "Enter number purchased: ";
   cin >> number;
   return number;
}

double saleTotal(double getPrice, int getNumber)
{
    char saleType;
    double total;
    double subTotal;

    cout << "Type W if this is wholesale purchase. n"
        << "Type R if this is retail purchase. n"
        << "then press return... n";
    cin.ignore();
    cin.get(saleType);

    if ((saleType == 'W') || (saleType == 'w'))
    {
        total = getPrice * getNumber;
        return total;
    }
    else if ((saleType == 'R') || (saleType == 'r'))
    {
        subTotal = getPrice * getNumber;
        total = subTotal   subTotal * TAX_RATE;
        return total;
    }
    else
    {
        cout << "Error in the input...";
    }
}

void retail(double getPrice, int getNumber, double saleTotal)
{
    char saleType;

    cout << setprecision(2) << fixed << showpoint << getNumber << " items at $" << getPrice << endl;
    cout << "Total bill $" << saleTotal;
    if ((saleType == 'R') || (saleType == 'r'))
    {
        cout << " includes sales tax.n";
    }
    return;
}
  

Как следует из названия методов, вам нужно:

 double subTotal()
{
  ...
  return mySubTotal;
}
  

и:

 double total()
{
  ...
  return total;
}
  

Однако в total() методе saleType переменная используется неинициализированной, как прокомментировал @ThomasSablik, что означает, что ваш код вызывает здесь неопределенное поведение (UB):

 if ((saleType == 'R') || (saleType == 'r'))
  

поскольку вы проверяете неинициализированную переменную в условиях этого оператора if .

subTotal() также вызывает UB по той же причине. Инициализируйте переменную.

О ваших функциях subTotal(); и total переменных не знаю myPrice myNumber main() .

Вы объявляете новые переменные с теми же именами, но они НЕ связаны с исходными.

Вам нужно передать эти переменные из main() в другие функции.