Back to General and Gameplay Programming

More Advanced, maybe not Class problem binary file mode

justinwalsh · 2003-11-17T02:04:43

I am writing a Class that needs to store its own data to a config file. I will give you the code below that works and reads the data just fine, in one fasion... Hear is my class first... the functions readInConfigFile, and writeOutConfigFile give me the troubles... but ignore those functions for now... ConfigFileWriter.h #ifndef CONFIGFILEWRITER_H #define CONFIGFILEWRITER_H class GameSettings { public: GameSettings(int CwindowWidth, int CwindowHeight, int CwindowXpos, int CwindowYpos, int CwindowBitDepth, bool CwindowFullscreen) :windowWidth(CwindowWidth),windowHeight(CwindowHeight), windowXPos(CwindowXpos),windowYPos(CwindowYpos), windowBitDepth(CwindowBitDepth),windowFullscreen(CwindowFullscreen){} GameSettings(void); ~GameSettings(){} //Data Reading Functions int getWindowWidth()const {return windowWidth;} int getWindowHeight()const {return windowHeight;} int getWindowX()const {return windowXPos;} int getWindowY()const {return windowYPos;} int getWindowBitDepth()const {return windowBitDepth;} bool getWindowFullscreen()const {return windowFullscreen;} bool readInConfigFile(char filename[20]); //Data Storing Functions void setWindow(int width, int height, int x, int y, int depth, bool fullscreen); void setWindowSize(int width, int height) {windowWidth = width; windowHeight = height;} void setWindowPos(int x, int y) {windowXPos = x; windowYPos = y;} void setWindowBitDepth(int BD) {windowBitDepth = BD;} void setWindowFullscreen(bool set) {windowFullscreen = set;} bool writeOutConfigFile(char filename[20]); private: int windowWidth; int windowHeight; int windowXPos; int windowYPos; int windowBitDepth; bool windowFullscreen; }; #endif ConfigFileWriter.cpp #include "ConfigFileWriter.h" #include <fstream.h> GameSettings :: GameSettings(void) { windowWidth = 640; windowHeight = 480; windowXPos = 100; windowYPos = 100; windowBitDepth = 16; windowFullscreen = false; } bool GameSettings::readInConfigFile(char filename[20]) { ifstream fin(filename, ios::binary); if(!fin) return false; fin.read((char *) this, sizeof this); fin.close(); return true; } void GameSettings::setWindow(int width, int height, int x, int y, int depth, bool fullscreen) { windowWidth = width; windowHeight = height; windowXPos = x; windowYPos = y; windowBitDepth = depth; windowFullscreen = fullscreen; } bool GameSettings::writeOutConfigFile(char filename[20]) { ofstream fout(filename, ios::binary); if (!fout) return false; fout.write((char*) this, sizeof this); fout.close(); return true; } Ok now you have the class so hear is a working implementation of writing the class to a file... WORKING VERSION: #include "ConfigFileWriter.h" #include <fstream.h> int main(int argc, char* argv[]) { ofstream fout("config.cfg", ios::binary); if (!fout) { cout << "Unable to open file\n"; return(1); } GameSettings Temp(640,480,10,10,16,false); //Temp.writeOutConfigFile("config.cfg"); fout.write((char*) &Temp,sizeof Temp); fout.close(); cout << "Wrote Class Temp to file\n"; ifstream fin("config.cfg", ios::binary); if(!fin) { cout << "Unable to open file\n"; return(1); } GameSettings Temp2(100,200,300,400,500,false); cout << "Temp2 windowHeight = " << Temp2.getWindowHeight() << endl; cout << "Temp2 windowWidth = " << Temp2.getWindowWidth() << endl; //Temp2.readInConfigFile("config.cfg"); fin.read((char *) &Temp2, sizeof Temp2); fin.close(); cout << "Read Class Temp from file\n"; cout << "Temp2 windowHeight = " << Temp2.getWindowHeight() << endl; cout << "Temp2 windowWidth = " << Temp2.getWindowWidth() << endl; return 0; } now what i want to do, is have the class know how to write and read itself hence the functions readInCOnfigFile but it dsnt work, not sure how to make it work, do i need copy constructors and define a new class? or is there an easier way? hear is what i am trying to do.... #include "ConfigFileWriter.h" #include <fstream.h> int main(int argc, char* argv[]) { //ofstream fout("config.cfg", ios::binary); //if (!fout) { // cout << "Unable to open file\n"; // return(1); //} GameSettings Temp(640,480,10,10,16,false); Temp.writeOutConfigFile("config.cfg"); //fout.write((char*) &Temp,sizeof Temp); //fout.close(); cout << "Wrote Class Temp to file\n"; //ifstream fin("config.cfg", ios::binary); //if(!fin) { // cout << "Unable to open file\n"; // return(1); //} GameSettings Temp2(100,200,300,400,500,false); cout << "Temp2 windowHeight = " << Temp2.getWindowHeight() << endl; cout << "Temp2 windowWidth = " << Temp2.getWindowWidth() << endl; Temp2.readInConfigFile("config.cfg"); //fin.read((char *) &Temp2, sizeof Temp2); //fin.close(); cout << "Read Class Temp from file\n"; cout << "Temp2 windowHeight = " << Temp2.getWindowHeight() << endl; cout << "Temp2 windowWidth = " << Temp2.getWindowWidth() << endl; return 0; } please help me out, i dont understand why it dsnt work? i think the this is the problem.

General and Gameplay Programming Programming

Started by justinwalsh November 12, 2003 11:50 AM

10 comments, last by justinwalsh 20 years, 5 months ago

justinwalsh

122

Author

November 17, 2003 01:27 AM

Well, I took all of your advice and made a config file class, thats a tad more generic, not too advanced but gets the job done forwhat i want it to do. Hear is the code...

ConfigWriter.h

#ifndef CONFIGWRITER_H#define CONFIGWRITER_H//////////////////////////////////////////////////////////		ConfigWriter.h//		Justin Walsh 11/17/2003//		ConfigWriter  - a class to handle writing and//			reading a basic config file, stores values//			as labels, followed by an int.///////////////////////////////////////////////////////						#include <iostream>#include <fstream>#include <vector>#include <string>using namespace std;#define cfg_nomatch 9999struct field {	int value;	string label;};class ConfigWriter  {	private:		vector fileEntries;		char* filename;		ofstream fout;		ifstream fin;			public:		ConfigWriter(char* filename);		~ConfigWriter();		void AddEntry(string eLabel, int eValue);		int GetEntry(string searchVal);		bool write(void);		bool read(void);};#endif

ConfigWriter.cpp

   #include "ConfigWriter.h"ConfigWriter::ConfigWriter(char *filename)  {	ConfigWriter::filename = filename;}ConfigWriter::~ConfigWriter()  {	fileEntries.clear();}bool ConfigWriter::write()  {	fout.open(filename);	if(!fout.good())	//if not good rtrn false		return false;	for(int i=0; i < fileEntries.size(); i++)  {		fout << fileEntries.label << " " << fileEntries.value << endl;<br>	}<br><br>	fout << flush;<br>	fout.close();<br><br>	return true;<br>}<br><br>bool ConfigWriter::read()  {<br>	fin.open(filename);<br>	field temp;<br><br>	if(!fin.good())<br>		return false;<br><br>	while(!fin.eof()) {<br>		fin >> temp.label;<br>		fin >> temp.value;<br>		fileEntries.push_back(temp);<br>	}<br>	fin.close();<br>	return true;<br>}<br><br>void ConfigWriter::AddEntry(string eLabel, int eValue)  {<br>	field temp;<br>	temp.label = eLabel;<br>	temp.value = eValue;<br>	fileEntries.push_back(temp);<br>}<br><br>int ConfigWriter::GetEntry(string searchVal)  {<br>	for(int i=0; i < fileEntries.size(); i++)  {<br>		if(fileEntries.label == searchVal)<br>			return fileEntries.value;<br>	}<br>	return cfg_nomatch;<br>}<br> </pre> <br><br>I know it isnt very advanced,  but again gets th job done, has readable text file.  Will probably write another class over this &#111;ne that will actually do the settings of the window etc, but give some replies as to this way of doing things.  </i>  <br><br><SPAN CLASS=editedby>[edited by - justinwalsh on November 17, 2003 2:29:19 AM]</SPAN>

Fruny

1,658

November 17, 2003 02:04 AM

"Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it." — Brian W. Kernighan

More Advanced, maybe not Class problem binary file mode

This topic is closed to new replies.

Popular Topics

Recommended Tutorials

More Advanced, maybe not Class problem binary file mode

This topic is closed to new replies.

Popular Topics

Recommended Tutorials

Reticulating splines