Class Objects

[Raspb3rry14]

When I enter 3 different presidents and their numbers/quotes into the for loop, it only displays the last entered president in all iterations of the display member function loop.

How can I fix this?

Thank you!

// MAIN
#include "Presidents.h"

int main()
{
	Presidents president;

	string name;
	string quote;
	int number;

	for (int i = 0; i < 3; i++)
	{
	cout << "Enter the first president's number: ";
	cin >> number;
	president.setNumber(number);
	cout << "Enter his name: ";
	cin >> name;
	president.setName(name);
	cout << "Enter his quote: ";
	cin >> quote;
	president.setQuote(quote);
	cout << "\n";
	}

	president.displayPresidents();

	return 0;
}

// PRESIDENTS.H
#include <iostream>
#include <string>
using namespace std;

#ifndef PRESIDENTS_H_
#define PRESIDENTS_H_

class Presidents
{
private:
	int _number;
	string _name;
	string _quote;

public:
	Presidents();
	~Presidents();
	void setNumber(int number);
	void setName(string name);
	void setQuote(string quote);
	void displayPresidents()const;
};

#endif

//PRESIDENTS.CPP
#include "Presidents.h"

	Presidents::Presidents()
	{
		_number = 0;
		_name = "";
		_quote = "";
	}
	//--------------------------------------------------------------------------
	Presidents::~Presidents(){}
	//--------------------------------------------------------------------------
	void Presidents::setNumber(int number)
	{
		_number = number;
	}
	//--------------------------------------------------------------------------
	void Presidents::setName(string name)
	{
		_name = name;
	}
	//--------------------------------------------------------------------------
	void Presidents::setQuote(string quote)
	{
		_quote = quote;
	}
	//--------------------------------------------------------------------------
	void Presidents::displayPresidents()const
	{
		for (int i = 0; i < 3; i++)
		{
		cout << "The Presidents are:\n"
			 << _name << ", " << _number << " President, said: "
			 << "\"" << _quote << ".\"\n\n";
		}
	}
	//--------------------------------------------------------------------------

[wildblue]

You only create one president object in main, so the code is just updating the same president object with different values. So when you print out, you're printing the details for the same president three times.

[j3Rz8vqX]

As wildblue stated.

A possible example with vectors:

#include <iostream>
#include <vector>
using std::cout;
using std::cin;
using std::endl;

class Presidents{
private:
    std::string name, quote;
    int number;
public:
    Presidents(){
        number=0;
    }
    void setNumber(int n){
        number = n;
    }
    void setName(std::string str){
        name = str;
    }
    void setQuote(std::string str){
        quote = str;
    }
    void displayPresidents(){
        cout<<number<<' '<<name<<' '<<quote<<'\n';
    }
};
int main()
{
    Presidents president;
    std::vector<Presidents> presidents;

    std::string name;
    std::string quote;
    int number;

    for (int i = 0; i < 3; i++)
    {
        cout << "Enter the first president's number: ";
        cin >> number;
        cin.ignore(255,'\n');
        president.setNumber(number);
        cout << "Enter his name: ";
        getline(cin,name);
        president.setName(name);
        cout << "Enter his quote: ";
        getline(cin,quote);
        president.setQuote(quote);
        cout << "\n";
        presidents.push_back(president);
    }
    for(int i = 0; i < 3; ++i)
    {
        presidents[i].displayPresidents();
    }

    return 0;
}

Be careful when reading string data from the user, you must clear the buffer before you operate with cin again; if using cin.