help with Implementing class Item

[seeplus]

the only thing is that the only cin line should be after user is asked how many objects they want.

???

[ocKiNOsIi]

sorry. the only user input in the program should come after the user is asked "How many objects do you want?" the user is not setting the fruit, quantity, weight, etc.

[ocKiNOsIi]

if anyone is available for help....

I am still looking for help on this program, it is about 95% done. I just need help with the counter. the code is posted above, and It shows the error my code has.

When the user is asked how many items they want even if they input a number less than 5, all 5 items are printed. How do I fix this? and how do I print and error message if they go outside the limit of 1-5?

[jonnin]

do
{
cin >> counter;
if(counter > limit || counter < 1) complain
}while(counter > limit || counter < 1)


for(int i = 0; i < counter; i++)
{
print something[i];
}

[ocKiNOsIi]

trying to understand vectors here but I'm not sure what I'm doing wrong. its still just printing all 5 objects even when prompted to print less

int main()
{
   
    cout
    << "This program creates at least 5 separate, co-existing instances of a\n"
    << "custom designed object. Information about each object is displayed.\n"
    << "Each constructor, setter and getter is tested at least once.\n\n";
    /*
     Modify the program to ask: "How many objects do you want?". Loop as many times specified by the user
     (5 is enough for testing) to create that many objects. You can use 5 individual variables,
     or you can use an array or vector.*/
     
    constexpr size_t maxObj {5};
    size_t no_fruits {};
      do {
        cout << "How many objects do you want? (1 - " << maxObj << "): ";
            cin >> no_fruits;
            cin.clear();
        } while ((!cin || (no_fruits < 1 || no_fruits > maxObj)) && (cout << "Invalid input! Please enter a number (1-5)..\n") && cin.ignore(1000, '\n'));
    
    
   // cout<<"You created "<< counter<<" objects:\n# 1 of "
  //  << limit << " objects: "<< endl;
    
    cout << "\nYou created " << no_fruits << " objects:" << endl<<endl;
    cout <<"# 1 of "
    << no_fruits << " objects:\n";

//vector <type> name (size)
    vector<Fruit> fruits(no_fruits);
    
for (size_t f = 0; f < no_fruits; ++f) {    // will only loop once
    // fruit 1 - set's method
    Fruit fruit1;
    fruit1.set_Name("Apple ");
    fruit1.set_Quantity(4);
    fruit1.set_Weight(4.56);
    fruit1.set_Price(4);
    fruit1.set_Is_purchased(true);
    fruit1.print(); // <----
    
    
    // fruit 2 - constructor method
    cout<<"\n# 2 of " << no_fruits<< " objects: "<< endl;
    Fruit fruit2("Orange", 7, 2.65, 5, true);
    fruit2.print(); // <--
    
    
    // fruit 3 - constructor method
    cout<<"\n# 3 of "<< no_fruits<<" objects: "<< endl;
    Fruit fruit3("Banana", 4, 2.34, 2, false);
    fruit3.print(); // <--
    
    
    // fruit 4 - constructor method
   cout<<"\n# 4 of "<< no_fruits<<" objects: "<< endl;
    Fruit fruit4("Avocado", 16, 2.59, -6, false);
    fruit4.print(); // <--
    
    
    // fruit 5 - constructor method
    cout<<"\n# 5 of "<< no_fruits<<" objects: "<< endl;
    Fruit fruit5("Grapes", -7, -2.65, 5, false);
    fruit5.print(); // <--
    
    cout
    << "\nThis ends the class design, implementation, test program. Goodbye!"<< endl;

    return EXIT_SUCCESS;
}   // end of main 

[coder777]

This requirements doesn't make sense. When you have 5 individual variables they are created. A vector or array doesn't make sense because the data is completely different. What you can actually do would be something like this:

for (size_t f = 0; f < no_fruits; ++f) {    // will only loop once
if(f == 0)
{
    // fruit 1 - set's method
    Fruit fruit1;
    fruit1.set_Name("Apple ");
    fruit1.set_Quantity(4);
    fruit1.set_Weight(4.56);
    fruit1.set_Price(4);
    fruit1.set_Is_purchased(true);
    fruit1.print(); // <----
}
else if(f == 1)
{
    // fruit 2 - constructor method
    cout<<"\n# 2 of " << no_fruits<< " objects: "<< endl;
    Fruit fruit2("Orange", 7, 2.65, 5, true);
    fruit2.print(); // <--
}    
else if(f == 2)
{
    // fruit 3 - constructor method
    cout<<"\n# 3 of "<< no_fruits<<" objects: "<< endl;
    Fruit fruit3("Banana", 4, 2.34, 2, false);
    fruit3.print(); // <--

...

}

Not that it make too much sense...

[ocKiNOsIi]

ty! that finally worked.