Thread: Error CC296 and C2297

C++ does not support vector operations. You have to manually multiply and add each element. And btw, quantity is not initialized, so you'd get garbage results anyway right now.

You could do something like this, to cheat, if you were so inclined:

Code:

template<typename T>
std::vector<T> operator*(const std::vector<T>& lhs, const std::vector<T>& rhs)
{
  std::vector<T> retVal(lhs.size());
  for (auto itLeft = lhs.begin(), itRight = rhs.begin(), itRes = retVal.begin(); itLeft != lhs.end() && itRight != rhs.end(); itLeft++, itRight++, itRes++)
  {
    *itRes = *itLeft * *itRight;
  }

  return retVal;
}

Originally Posted by nadeera

I can't figure out how to solve this error, please help.

Code:

double price[itemNumber] = {10.50, 13.00, 10.50, 13.00, 8.50, 10.00, 11.50, 13.50};
double quantity[itemNumber];
double amountDue;

amountDue = quantity * price;

I keep getting error C2296 '*': illegal, left operand has type 'double [8]' and error C2297 '*': illegal, right operand has type 'double [8]'. How do I solve this?

std::vector is not a mathematical vector. It serves a purpose of a container storing its elements sequentially in a contiguous chunk of memory (it is a dynamically grown array).
There are some pretty good libraries out there which can help you deal with math stuff, for example, you might want to check out Eigen.

Why didn't you define the quantity elements like you did the price? Also weren't they arrays? You're not specifying which element in the arrays your multiplying.

If you want to initialize all the array elements to zero the syntax is:

Code:

 doublequantity[itemNumber] = { 0 };

If you want to assign the 8th element in the array the value 0 you need to declare the array first then try to address the 8th element.

Code:

 doublequantity[itemNumber] = { 0 };
  
  quantity[itemNumber - 1] = 0;

Originally Posted by nadeera

This is basically what I want to do. If I initialize quantity[itemNumber] =0, I get error C2440: 'initializing': cannot convert from 'int' to 'double[8]'.

Code:

const int itemNumber = 8;
    const string coffeeFlavours[itemNumber]= { "CappucinoR", "CappucinoL" "LatteR", "LatteL", "LongBlackR", "LongBlackL", " IceCoffeeR", "IceCoffeeL"};
    double price[itemNumber] = { 10.50, 13.00, 10.50, 13.00, 8.50, 10.00, 11.50, 13.50 };
    double quantity[itemNumber] = 0;
    char pitz; 
    double amountDue = 0, amountDue1, amountDue2,amountDue3,amountDue4, amountReceived , totalDue, change; 
    int customerID, option;
    float sum = 0, n, choice;
    cout << "\n\n\n\t\t\t!!MENU";
    cout << "\n\n\nInsert Customer Order Number : ";
    cin >> customerID;
    cout << "\n\nCustomer No:" << customerID << endl;





    cout << "\n\n            \t\t\t MENU\n" << endl;
    
    cout << "   ==| COFFEE FLAVOURS |==| REGULAR |==| LARGE |==|EXTRA CHARGE|==========";
    cout << "\n   | 1-Cappucino           RM 10.50    RM 13.00  RM 2.00 for Extra Shot |";
    cout << "\n   | 2-Latte               RM 10.50    RM 13.00                         |";
    cout << "\n   | 3-Long Black          RM 8.50     RM 10.00                         |";
    cout << "\n   | 4-Ice Coffee          RM 11.50    RM 13.50  RM 1.00 for Ice Cream  |\n";
    cout << "\n  =======================================================================";
    cout << endl;


    do {
        cout << "\nWhat do you want to order?";
        cin >> option;

        switch (option) {

        case 1:
        A :
          
          cout << "Cappucino" << endl ;
          cout << "How many?";
          cin >> quantity[itemNumber]; 
          cout << "\n\nDo you prefer :";
          cout << "\n\n1-Regular or 2-Large :";
          cin >> choice;
          if (choice == 1)
          {
              cout << "\n\tPrice: RM" << price[0]<< endl;
              amountDue1 = price[0] * quantity[itemNumber];
              cout << "\n\tAmount Due: RM" << amountDue << endl;
          }
          else if (choice == 2)
          {
              cout << "\n\tPrice: RM" << price[1] << endl;
              amountDue1 = price[1] * quantity[itemNumber];
              cout << "\n\tAmount Due: RM" << amountDue << endl;
          }
          else
          {
              cout << "Please enter 1 or 2 only";
              goto A;
          }
        break;
        case 2:
        B :
          
          cout << "Latte" << endl ;
          cout << "How many?";
          cin >> quantity[itemNumber]; 
          cout << "\n\nDo you prefer :";
          cout << "\n\n1-Regular or 2-Large :";
          cin >> choice;
          if (choice == 1)
          {
              cout << "\n\tPrice: RM" << price[2]<< endl;
              amountDue2 = price[2] * quantity[itemNumber];
              cout << "\n\tAmount Due: RM" << amountDue << endl;
          }
          else if (choice == 2)
          {
              cout << "\n\tPrice: RM" << price[3] << endl;
              amountDue2 = price[3] * quantity[itemNumber];
              cout << "\n\tAmount Due: RM" << amountDue << endl;
          }
          else
          {
              cout << "Please enter 1 or 2 only";
              goto B;
          }
        break;
        case 3:
        C :
          
          cout << "Long Black" << endl ;
          cout << "How many?";
          cin >> quantity[itemNumber]; 
          cout << "\n\nDo you prefer :";
          cout << "\n\n1-Regular or 2-Large :";
          cin >> choice;
          if (choice == 1)
          {
              cout << "\n\tPrice: RM" << price[4]<< endl;
              amountDue3 = price[4] * quantity[itemNumber];
              cout << "\n\tAmount Due: RM" << amountDue << endl;
          }
          else if (choice == 2)
          {
              cout << "\n\tPrice: RM" << price[5] << endl;
              amountDue3 = price[5] * quantity[itemNumber];
              cout << "\n\tAmount Due: RM" << amountDue << endl;
          }
          else
          {
              cout << "Please enter 1 or 2 only";
              goto C;
          }
        break;
        case 4:
        D :
          
          cout << "Iced Coffee" << endl ;
          cout << "How many?";
          cin >> quantity[itemNumber]; 
          cout << "\n\nDo you prefer :";
          cout << "\n\n1-Regular or 2-Large :";
          cin >> choice;
          if (choice == 1)
          {
              cout << "\n\tPrice: RM" << price[6]<< endl;
              amountDue4 = price[6] * quantity[itemNumber];
              cout << "\n\tAmount Due: RM" << amountDue << endl;
          }
          else if (choice == 2)
          {
              cout << "\n\tPrice: RM" << price[7] << endl;
              amountDue4 = price[7] * quantity[itemNumber];
              cout << "\n\tAmount Due: RM" << amountDue << endl;
          }
          else
          {
              cout << "Please enter 1 or 2 only";
              goto D;
          }
break;
        default:
            break;

        }
        cout << "\nDo you want to order another (y/n)? ";
        cin >> pitz;
    } while ((pitz == 'y') || (pitz == 'Y'));

    amountDue = amountDue1 + amountDue2 + amountDue3 + amountDue4;
    cout << "\nAmount to be paid: RM" << amountDue;

    do {
        cout << "\n\nYour cash is   : RM";
        cin >> amountReceived;
        if (amountReceived <amountDue){
            cout << "\a\n\t     Your money is not enough! " << endl;
        }
    }

    while (amountReceived <amountDue);

    change = amountReceived- amountDue;
    cout << "\nYour Balance  : RM" << change << "\n\n" << endl << endl;

    cout << "\t\t  For your receipt: ";
    system("pause");
    system("cls");

Code:

const int itemNumber = 8;

cin >> quantity[itemNumber

What you're telling the computer to do is access the 9 th element of the array, but the array only has 8 members. Array start their count at 0. This means the 8 element is your array is 7. There is a mysterious 9 element but you would go out of bounds. I'm not a good coder but this would make it work and keep you in bounds.

Code:

const int itemNumber = 8;

cin >> quantity[itemNumber - 1]

You should replace all those backward jumping goto statements with an appropriate loop structure.

You should also move some code into other functions, say

Code:

    cout << "\n   | 1-Cappucino           RM 10.50    RM 13.00  RM 2.00 for Extra Shot |";
    cout << "\n   | 2-Latte               RM 10.50    RM 13.00                         |";
    cout << "\n   | 3-Long Black          RM 8.50     RM 10.00                         |";
    cout << "\n   | 4-Ice Coffee          RM 11.50    RM 13.50  RM 1.00 for Ice Cream  |\n";
    cout << "\n  =======================================================================";
    cout << endl;
 
    do {
        cout << "\nWhat do you want to order?";
        cin >> option;
 
        switch (option) {
        case 1:
            doCappuchino();
            break;
        case 2:
            doLatte();
            break;
        // etc
        }

        cout << "\nDo you want to order another (y/n)? ";
        cin >> pitz;
    } while ((pitz == 'y') || (pitz == 'Y'));

Originally Posted by Aslaville

If you want to initialize all the array elements to zero the syntax is:

Code:

 doublequantity[itemNumber] = { 0 };

Actually, it's more appropriate to say

Code:

double quantity[ItemNumber] = {};

You don't have to initialize any elements of the array in C++. All elements which are not explicitly initialized as initialized to 0.

Originally Posted by Aslaville

If you want to initialize all the array elements to zero the syntax is:

Code:

 doublequantity[itemNumber] = { 0 };

This solution is misleading, because one could deduce from it that changing 0 to 1

Code:

doublequantity[itemNumber] = { 1 };

would initialize all elements to 1, which is wrong, because it would only initialize the first one (the rest would still be zero).