Hash Table Display All with chaining

[CJPowell27]

hey guys I'm trying to write a display all function for my hash table and I'm running into

issues with trying to access private data members of the videoGame class in the

displayAll function for my table class.  I was thinking about writing a function of sorts 

to call from the tables displayAll function but I have no idea. Any help is much appreciated. 

class videoGame
{
	public:
		videoGame(void);//Done
		~videoGame(void);//Done
		int createGame(char * ctitle, char * cdescription, char * ccategory, char * cplatform, char * crecommendations, int crating);//Done
		int copyGame(const videoGame & newGame);//Done
		int editRec(/*Game*/);
		int retrieve(char * keyValue, videoGame & found)const;//Done
	private:
		char * title;
		char * description;
		char * category;
		char * platform;
		char * recommendations;
		int rating;

};


struct node
{
	videoGame game;
	node * next;
};


class table
{
	public:
		table(int size=5);//Done
		~table();
		int insert(char * keyValue, const videoGame & toAdd);//Done
		int remove(char * keyValue, videoGame & found);
		int retrieve(char * searchTitle, videoGame & found)const;//Done
		int displayAll(videoGame)const;
		int displayPlatform(char * searchPlatform);
	private:
		node ** hashTable;
		int tableSize;
		int hashFunction(char * keyValue)const;
};

 

[mathijs727]

Look into iterators such as used in std::unordered_map (C++ own hashmap).

You might also want to use templates so that your hashtable can also store other classes than "videoGame".

[Guest]

You need to add public accessor methods to the VideoGame class. That is the only way to expose private members while maintaining encapsulation.

[CJPowell27]

20 hours ago, SSL said:

You need to add public accessor methods to the VideoGame class. That is the only way to expose private members while maintaining encapsulation.

i finally figured to just add a display function to the videoGame class and have the display all function call the basic display

[Guest]

1 minute ago, CJPowell27 said:

i finally figured to just add a display function to the videoGame class and have the display all function call the basic display

 

Except that is a violation of the single-responsibility principle.

[CJPowell27]

On 11/11/2016 at 1:14 PM, SSL said:

 

Except that is a violation of the single-responsibility principle.

I'm currently in a low level programming class so it isn't as big of an issue right now.  For future reference, how would I do that without violating that principle?

[Guest]

1 minute ago, CJPowell27 said:

I'm currently in a low level programming class so it isn't as big of an issue right now.  For future reference, how would I do that without violating that principle?

 

Just put the display code in a separate class. You're using classes to begin with, so "low level" hardly need be a barrier to following basic principles of good design.