# Problem with fstream.

This topic is 4153 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

## Recommended Posts

Currently got a weird looking problem with fstream.
1>C:\Program Files\Microsoft Visual Studio 8\VC\include\fstream(675) : error C2248: 'std::basic_ios<_Elem,_Traits>::basic_ios' : cannot access private member declared in class 'std::basic_ios<_Elem,_Traits>'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        C:\Program Files\Microsoft Visual Studio 8\VC\include\ios(151) : see declaration of 'std::basic_ios<_Elem,_Traits>::basic_ios'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        This diagnostic occurred in the compiler generated function 'std::basic_ifstream<_Elem,_Traits>::basic_ifstream(const std::basic_ifstream<_Elem,_Traits> &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]


#pragma once

#include <iostream>
#include <fstream>
#include <cmath>
#include <vector>
#include "..\Vectors.h"
#include "BulletManager.h"

using namespace std;

struct sUnitInfo
{
int					Health;
unsigned int		Speed;
unsigned int		Power;
unsigned short		UnitType;

double				VisibleRange;
double				FiringRange;
double				RateOfFire;
};

class CUnit
{
public:
CUnit(unsigned int UnitID, CBulletManager *inBulletManager);
~CUnit(void);

void			Initialise(sUnitInfo inInfo);

unsigned int	GetID() { return m_ID; }
unsigned short	GetType() { return m_Type; }

CVector2d		GetPosition();// return CVector2d(m_RealX, m_RealY); } //m_Position; }
#ifdef _DEBUG
double			GetRealX() { return m_RealX; }
double			GetRealY() { return m_RealY; }
#endif
double			GetShootingRange() { return m_FiringRange; }
double			GetViewingRange() { return m_VisibleRange; }
double			GetFirepower() { return m_Power; }

bool			InflictDamage(double Amount);
void			Shoot(CVector2d TargetPosition);

void			SetRoute(vector<CVector2i>* Route);
void			SetPosition(CVector2d inPosition);
CVector2i		NextDestination() { return m_Route[0]; }

void			Update(double inFrameTime);
private:
void			Move();
void			Animate(){ return; }
void			Draw(){ return; }

bool				Initialised;
unsigned int		m_ID;
unsigned int		m_Power;
unsigned int		m_Speed;
double				m_Health;
unsigned int		m_TargetID;

unsigned short		m_Type;
unsigned short		m_Formation;

double				m_VisibleRange;
double				m_FiringRange;
double				m_RateOfFire;
double				m_FireTimer;

double				m_FrameTime;
double				m_RealX;
double				m_RealY;
CVector2d			m_Position;
vector<CVector2i>	m_Route;

bool				m_isEnemy;
bool				m_isFiring;
bool				m_HasRoute;

CBulletManager*		m_BulletManager;
ifstream			UnitLog;
};


I would have edited this down a little but I'm unsure how it would affect the declaration of fstream. This isn't the only class where I'm using the fstream object. I have it declared exactly the same in 3 other classes. Specifically in "FileReader.h"
#pragma once
#include <iostream>
#include <fstream>
#include <vector>

#include "MapData.h"
#include "Unit.h"

using namespace std;

typedef vector<unsigned short> MapY;
typedef vector<MapY> Map;

{
public:

bool			SaveMap(char* Mapname, CMapData* MapData);
//	bool			SaveUnit();
private:
fstream Logger;
};


Which works perfectly. fstream& UnitLog; doesn't work fstream* UnitLog; doesn't work ifstream UnitLog; (or any variation */&) doesn't work ofstream UnitLog; (or any variation */&) doesn't work I have tried creating a completely empty class and including it there instead with "Logger.h"
#pragma once

#include <iostream>
#include <fstream>

#using namespace std;

class CLogger
{
public:
CLogger();
~CLogger();
private:
fstream Log;
};


Which also fails. So I'm at a loss now. I don't know what is causing the error. Any help is greatly appreciated and urgently required (as this is my Final Year Project for University) >.< [Edited by - Ehrys on March 7, 2007 8:29:55 AM]

##### Share on other sites
I can't see any problem with your code. Try removing the iostream inclusion - you only need that for certain streams and manipulators. If you do need it in there, try re-ordering the header inclusions. Or move the fstream member from the private block to being public.

##### Share on other sites
Tried what you suggested, still same error. Is there a limit as to how many instances you can have per application?

Just a note, removing iostream has changed the line number of the error in fstream.h from 675 to 933. But I don't know enough about the fstream internals on Win32 to debug.

##### Share on other sites
"#using namespace std;" <--- The hell?

Quote:
 1> This diagnostic occurred in the compiler generated function 'std::basic_ifstream<_Elem,_Traits>::basic_ifstream(const std::basic_ifstream<_Elem,_Traits> &)'1> with1> [1> _Elem=char,1> _Traits=std::char_traits1> ]

This means you're trying to copy an std::ifstream somewhere, which is illegal. If there was more to your error message after this, please post it, and the source code that it references.

##### Share on other sites
Everything was in the OP

Also, I use the std namespace because I get fed up of writing std:: everywhere. I don't have another namespace, nor am I using statics. so not using it would have been a waste of time. Unless you know a better way.

Also, the code only generates the error when I declare the object
"fstream UnitLog;"

There has been no initialisation anywhere in the code. As I previously stated trying to use references and pointers also failed to work, giving me the same error as before.

EDIT:

I just thought of something. Could it be the way that I'm declaring/instantiating the CUnit class in my Database? I can't open VS atm however it goes along the lines of:

1. Create a vector of CUnit.2. Resize vector to number of CUnit that will be inited.3. Load unit info from disk.4. DO WHILE : Number of units created is less than the specified amount.5. Create a variable of type CUnit6. Initialise variable with UnitInfo "CUnit::Initialiase(sUnitInfo UnitInfo);"7. Push unit onto vector. (vector::push_back())8. LOOP

Header File:vector<CUnit> m_UnitList;CPP File:CDBase::GenerateUnits(unsigned int numGrunts){	for(unsigned int Counter = 0; Counter < numGrunts; Counter++)	{		CUnit Grunt(Counter + 1, m_BulletManager);		Grunt.Initialise(m_UnitInfo[0]);		m_UnitList.push_back(Grunt);	}	for(unsigned int Counter = numGrunts; Counter < numGrunts + (numGrunts / 5); Counter++)	{		CUnit General(Counter + 1, m_BulletManager);		General.Initialise(m_UnitInfo[1]);		m_UnitList.push_back(General);	}}

That's all there is.

Please forgive my use of "Counter" but I was in a rush ;)

[Edited by - Ehrys on March 7, 2007 1:48:31 PM]

##### Share on other sites
Quote:
 Original post by MaulingMonkeyThis means you're trying to copy an std::ifstream somewhere, which is illegal. If there was more to your error message after this, please post it, and the source code that it references.

Right. Perhaps an implicit copy constructor? Are you doing something like this somewhere?

CLogger log1;CLogger log2 = log1;

If so, you might need to write your own copy constructor.

EDIT: Seeing your edit above, that's probably it. Using std::vector<CUnit> means that you're putting units into the vector by value, which means the ones in the vector are copied from the ones you have. Try using a std::vector<CUnit*> instead, but watch out - you're going to have to deal with allocating and deleting those objects if you do.

##### Share on other sites
No, I'm simply trying to declare it.

In that particular class, I have not got any more code relating to fstream or the variable I am trying to create.
In reply to justinian's edit -
Yeah I guess that makes sense. Completely forgot that it would be making copies that way >.<

I'll begin recoding and let you know how it goes :)

Thanks for the help so far everyone.

##### Share on other sites
Problem solved. Now I just have to re-write my application to use pointers, heh.

Thanks for the help.

While I'm here I'm going to hijack my own thread. Anyone know where I can get some good info about distributed networking patterns/models. Preferably code based ones. Win32 or Linux, doesn't bother me :)

##### Share on other sites
Don't place "using namespace std" in header files. It will mess stuff up real good when you least expect it.

##### Share on other sites
Howcome?

Sorry I'm one of those people that like to understand the principle rather than just make do because "it does". ^.^

1. 1
Rutin
23
2. 2
3. 3
4. 4
JoeJ
18
5. 5

• 14
• 16
• 11
• 11
• 9
• ### Forum Statistics

• Total Topics
631757
• Total Posts
3002151
×