How to create a vector of class objects in C++?

Question

I am trying to create a simple stack using vector in C++.

Here is the code:

#include <vector>

class Site
{
    public:
        int i; // site position i (x-axis)
        int s; // site state
        vector<Site> neighbors;
        Site(void);
        Site(int ii, int ss);
        void AddNeighbor(Site &site);
};
Site::Site()
{
    i = -1;
    s = -1;
    vector<Site> neighbors;
}
Site::Site(int ii, int ss) 
{
    i = ii;
    s = ss;
}
void Site::AddNeighbor(Site &site)
{
    neighbors.push_back(site);
}

void testStack()
{
    int tot = 600;
    vector<Site> myStack();
    int i = 0;
    while (i < tot)
    {
        Site site(i, 1);
        myStack.push_back(site);
        i++;
    }

    i = 0;
    while (i < tot)
    {
        Site *site = myStack.back();
        myStack.pop_back();
        cout << site->i << site->s << endl;
        i++;
    }
}

Compiler errors:

ising_wolff.cpp: In function ‘void testStack()’: ising_wolff.cpp:373:17: error: request for member ‘push_back’ in ‘myStack’, which is of non-class type ‘std::vector()’ myStack.push_back(site); ^ ising_wolff.cpp:380:30: error: request for member ‘back’ in ‘myStack’, which is of non-class type ‘std::vector()’ Site *site = myStack.back(); ^ ising_wolff.cpp:381:17: error: request for member ‘pop_back’ in ‘myStack’, which is of non-class type ‘std::vector()’ myStack.pop_back();

What do these errors mean?

Here are some sites I have looked at:

1) Creating objects while adding them into vectors

2) push_back causing errors in C

3) how to create vectors of class object

Eric Leschinski · Accepted Answer · 2014-02-20 22:05:11Z

How to create a vector of class objects in C++?

Start with something simpler so you can get the hang of it.

First, create a vector of primitive ints:

#include <vector>
#include <iostream>
using namespace std;
int main(){
  vector<int> sites(5);
  sites.push_back(5);
  for(int x = 0; x < sites.size(); x++){
    cout << sites[x];
  }
  cout << endl;
  return 0;
}

Compiling it:

g++ -o test test.cpp

Running it:

./test
000005

Create a vector of class objects in a similar way as above:

#include <iostream>
#include <vector>
using namespace std;

class Site {
public:
    int i;
};

int main() {
    vector<Site> listofsites;
    Site *s1 = new Site;
    s1->i = 7;
    Site *s2 = new Site;
    s2->i = 9;
    listofsites.push_back(*s1);
    listofsites.push_back(*s2);
    vector<Site>::iterator it;
    for (it = listofsites.begin(); it != listofsites.end(); ++it) {
        cout << it->i;
    }
    return 0;
}

Which should print:

Joseph Mansfield · Accepted Answer · 2014-02-20 21:10:03Z

13

vector<Site> myStack();

This is actually a function declaration. The function is called myStack and it returns a vector<Site>. What you actually want is:

vector<Site> myStack;

The type of neighbours at the moment will store copies of the objects, not references. If you really want to store references, I recommend using a std::reference_wrapper (rather than using pointers):

vector<reference_wrapper<Site>> neighbors;

edited Feb 20, 2014 at 21:10

answered Feb 20, 2014 at 20:53

Joseph Mansfield

111k22 gold badges249 silver badges329 bronze badges

5 Comments

Girardi Over a year ago

If I declare neighbors as you said, the method AddNeighbors in class Site will remain correct? Thanks for the answer :)

Girardi Over a year ago

when I use std::reference_wrapper, like you proposed, compiler gives me the error:

In member function ‘void Site::AddNeighbor(Site&)’: ising_wolff.cpp:38:29: error: no matching function for call to ‘std::vector<std::reference_wrapper<Site> >::push_back(Site&)’

in neighbors.push_back(site);. And then prints a lot of "candidates" to replace it... The list is long that it is impossible to read

Joseph Mansfield Over a year ago

@Girardi Oh sorry, I misunderstood your previous question. You need to do neighbors.push_back(std::ref(site)).

Girardi Over a year ago

it says that ref is not a member of std

Girardi Over a year ago

I am sorry, I haven't notice the header in the link you referenced =)

Lightness Races in Orbit · Accepted Answer · 2014-02-20 20:51:55Z

7

vector<Site> myStack();

This is wrong. Lose the ().

You're declaring a function, not a vector.

Just write:

vector<Site> myStack;

answered Feb 20, 2014 at 20:51

Lightness Races in Orbit

387k77 gold badges670 silver badges1.1k bronze badges

Comments

Rodrigo Saucedo · Accepted Answer · 2016-03-13 20:38:55Z

2

You could use:

vector<Site> myStack;
myStack.resize(100); //will create 100 <Site> objects

answered Mar 13, 2016 at 20:38

Rodrigo Saucedo

211 bronze badge

Collectives™ on Stack Overflow

How to create a vector of class objects in C++?

4 Answers 4

How to create a vector of class objects in C++?

Comments

5 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

How to create a vector of class objects in C++?

Comments

5 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related