Loading objects into an array of pointer to objects

Question

I am trying to load competitor objects (starter) created in main to a ranker object that stores the competitor objects that are read in as an array of pointers to objects that are read in. For some reason it is only reading the first object that is passed in. How can I read all of the objects in from the main function to the storage in the class?

ranker.cpp:

#include "ranker.h"
#include "competitor.h"
#include <stdlib.h>

Ranker::Ranker(int lanes) {
    athlete = new Competitor*[lanes];   // fix
    numAthletes = 0;
    maxAthletes = lanes;
}

int Ranker::addList(Competitor* starter) {
    if (numAthletes < maxAthletes && &starter != NULL) {
        athlete[numAthletes] = starter;
        numAthletes++;

        return numAthletes;
    }
    else
        return 0;
}

Competitor* Ranker::getLane(int lane) {
    for (int i = 0; i < numAthletes; i++) {
        if (athlete[i]->getLane() == lane - 1) {
            return athlete[i];
        }
    }
}

Competitor* Ranker::getFinish(int position) {
    switch(position) {
        case 1:
            return athlete[3];
            break;
        case 2:
            return athlete[2];
            break;
        case 3:
            return athlete[1];
            break;
        case 4:
            return athlete[0];
            break;
    }
}

int Ranker::getFilled() {
    return numAthletes;
}

Ranker::~Ranker() {
    delete athlete;
}

ranker.h:

#ifndef _RANKER_H
#define _RANKER_H

#include "competitor.h"

class Ranker {
private:
Competitor** athlete;
    int numAthletes;
    int maxAthletes;
public:
    Ranker(int lanes);
    int addList(Competitor* starter);
    Competitor* getLane(int lane);
    Competitor* getFinish(int position);
    int getFilled();
    ~Ranker();
};

#endif

part of main function:

for (int i = 0; i < lanes; i++)
    rank.addList(starters[i]);

Your first problem is you using pointers, your second problem is also using pointers. Learn more about the C++ standard containers. — Some programmer dude
– Some programmer dude, Commented Jul 19, 2013 at 20:30

Casey · Accepted Answer · 2013-07-19 20:21:51Z

1

Use a std::vector<Competitor> to store your competitors, and add them with push_back().

answered Jul 19, 2013 at 20:21

Casey

43k7 gold badges102 silver badges128 bronze badges

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

Collectives™ on Stack Overflow

Loading objects into an array of pointer to objects

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related