Adding an object pointer to a vector of pointer-to-objects located inside a class

Question

I'm trying to dynamically allocate objects, then add those pointers to the object to a vector. However I get the error: "conversion from 'Library (*)()' to non-scalar type 'Library' requested". How do I correct this?

I've included the files below

//Item class

class Item
    private:
        std::string idCode;
        std::string title;
    public:
        Item(std::string idc, std::string t)
        {
            this->idCode=idc;
            this->title=t;
        };

//Book Class inheriting from Item class

class Book: public Item   //class inherits from parent Item
{
private:
    std::string author;
public:
    //constructor
    Book(std::string idcIn,std::string tIn,std::string authorIn)
    :Item(idcIn, tIn)
    {    author=authorIn;}
};

//Library class which holds the vector of pointers to

class Library
    private:
        std::vector<Item*>holdings;
    public:
        void addLibraryItem(Item* item)
        {
            holdings.push_back(item);
        }

And here is the main file

void addItem(Library);  //prototype for adding Item function

int main()
{
    Library lib();  //create Library object

    addItem(lib);   //ERROR POINTS TO HERE

    return 0;
}

void addItem(Library lib)
{
    Item *ptr=new Book("bookID", "Title", "Author")
    lib.Library::addLibraryItem(ptr);
}

Thank you for your time

Karthik Kalyanasundaram · Accepted Answer · 2015-03-13 05:28:52Z

2

Library lib(); is wrong. This statement declares a function lib which returns Library and takes no parameter

This should have been Library lib;

answered Mar 13, 2015 at 5:28

Karthik Kalyanasundaram

1,52511 silver badges24 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

EDkan · Accepted Answer · 2015-03-13 05:35:05Z

2

you should change definition of function addItem to

void addItem(Library& lib)
{
    Item *ptr=new Book("bookID", "Title", "Author")
    lib.addLibraryItem(ptr);
}

to be able to call method addLibraryItem of your referenced object, not copy of it

answered Mar 13, 2015 at 5:35

EDkan

5721 gold badge4 silver badges10 bronze badges

Comments

Hayden · Accepted Answer · 2015-03-13 05:59:46Z

1

Change Library lib() to Library lib.

edited Mar 13, 2015 at 5:59

answered Mar 13, 2015 at 5:30

Hayden

3,0162 gold badges18 silver badges33 bronze badges

1 Comment

xai Over a year ago

What does it mean when it will fall out of scope?

Collectives™ on Stack Overflow

Adding an object pointer to a vector of pointer-to-objects located inside a class

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related