Programming Forums > Application Development > C++

Kilo · May 22nd, 2006, 10:38 AM

My stupid variables are declared.. this is retarded!

// located in Engine.h
class Engine
{
public:

             bool b_mailFile;
	bool b_xFile    = false;
	bool b_samFile  = false;
	bool b_upload   = false;
};

// located in cpp file (that includes "engine.h")
bool Engine::ParseFunctionHeaders(std::string script[])
{
	b_mailFile = false; 
	b_xFile    = false;
	b_samFile  = false;
	b_upload   = false;

             return true;
}

could someone please shed some light on my stupidity issue?

EDIT: YUP it was stupidity! i guess throwing a prototype of the function into the class declaration would have helped lol

Kilo · May 22nd, 2006, 10:55 AM

New Problems

(Toggle Plain Text)

std::string Engine::LTrim(std::string trimString)
{
	std::string temp = trimString.c_str();

	for(int i = 0; i<= sizeof(temp); i++)  //THIS LINE IS PROBLEM #1
	{
		if(temp[i] = ' ')
		{
			trimString = "";

			for(int k = i+1; k <= sizeof(temp) - (i+1); k++)  // THIS LINE
			{
				trimString += temp[k];
			}
		}
	}

	return trimString;
}

for the code above im getting this warning. I understand the warning just cant follow appropriate steps to overcoming it.

(Toggle Plain Text)

1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(56) : warning C4018: '<=' : signed/unsigned mismatch
1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(77) : warning C4018: '<=' : signed/unsigned mismatch

And below i have a stupidity problem.. it will not let me use a string for comparison!

(Toggle Plain Text)

// PARSE HEADERS FUNCTION
bool Engine::ParseFunctionHeaders(std::string script[])
{
	b_mailFile = false; 
	b_xFile    = false;
	b_samFile  = false;
	b_upload   = false;

    int scriptSize = sizeof(script)/sizeof(script[0]);
	
	for(int i = 0; i <= scriptSize; i++)
	{
		if(script[i] == "")
			break;
		
		if(Trim(script[i]) == "MAILFILE" && b_mailFile == false)
		{
			i_mailFilePos = i;
			b_mailFile = true;
		}
		else if(Trim(script[i]) == "MAILFILE" && b_mailFile == true)
		{
			//AddError("[LINE#" + (i+1) + "] ER1003 Multiple Instance (MAILFILE)");
		}

        // THE REST OF THE FUNCTION IS UN-NEEDED

and below it the error

(Toggle Plain Text)

1>        ]
1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(102) : error C2678: binary '==' : no operator found which takes a left-hand operand of type 'std::basic_string<_Elem,_Traits,_Ax>' (or there is no acceptable conversion)
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>,
1>            _Ax=std::allocator<char>
1>        ]
1>        could be 'built-in C++ operator==(const char [9], const char [9])'
1>        c:\program files\microsoft visual studio 8\vc\platformsdk\include\guiddef.h(192): or 'int operator ==(const GUID &,const GUID &)'
1>        while trying to match the argument list '(std::basic_string<_Elem,_Traits,_Ax>, const char [9])'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>,
1>            _Ax=std::allocator<char>
1>        ]

Kilo · May 22nd, 2006, 11:07 AM

Ok i fixed my problem again by appending ".c_str()" to all "script[i]"

Sorry to waste space! I should have known this.

EDIT: OH i still have the signed/unsigned warning issue!

Cache · May 22nd, 2006, 1:11 PM

Quote:

Originally Posted by Kilo

(Toggle Plain Text)

	//...
	for(int i = 0; i<= sizeof(temp); i++)  //THIS LINE IS PROBLEM #1
	//...

			for(int k = i+1; k <= sizeof(temp) - (i+1); k++)  // THIS LINE
	//...

sizeof() returns type size_t, which a typedef of 'unsigned int'. Also, I think you want temp.size() for your comparison. The above is essensially the same as: sizeof( std::string );

Ooble · May 22nd, 2006, 5:52 PM

(Toggle Plain Text)

int scriptSize = sizeof(script)/sizeof(script[0]);

This will not work. A std::string is not just a collection of characters. It's a linked list of characters with other variables and functions thrown into the mix.

Try this:

(Toggle Plain Text)

#include <iostream>
#include <string>

int main ()
{
        std::string moo = "This is a very big string.";
        std::cout << "The string 'moo' contains \"" << moo << "\" and is " << sizeof(moo) << " bytes big." << std::endl;
}

Cache · May 22nd, 2006, 6:38 PM

Quote:

Originally Posted by Ooble

(Toggle Plain Text)

int scriptSize = sizeof(script)/sizeof(script[0]);

This will not work. A std::string is not just a collection of characters. It's a linked list of characters with other variables and functions thrown into the mix.

That depends on what you think it's going to do. Note that 'script' is an array of std::string. Each element of the array will == sizeof( std::string ). So, if you use 'sizeof(script)/sizeof(script[0])' to calculate the number of elements in the array, it will work.

Ooble · May 22nd, 2006, 6:43 PM

Ah, crap... ignore me. Must be late.

OK, if you want some useful advice, how 'bout this: use a freakin' std::vector instead?

nnxion · May 22nd, 2006, 6:51 PM

Quote:

Originally Posted by Ooble

A std::string is not just a collection of characters. It's a linked list of characters with other variables and functions thrown into the mix.

Ooble you should know better than to provide misinformation. std::string is not a linked list of characters.

Ooble · May 22nd, 2006, 6:54 PM

I believe that's how the information is stored... you sure I'm wrong? At the lowest level, I could swear it's a wrapper for a linked list.

Jimbo · May 22nd, 2006, 6:55 PM

nnxion:

Quote:

Originally Posted by Ooble

Ah, crap... ignore me. Must be late.

:p

[edit:] since std::string supports arbitrary indexing, I doubt a linked list would be used...

May 22nd, 2006, 10:55 AM	#2
Kilo Expert Programmer Join Date: Nov 2005 Location: In Pink Clam? Posts: 542 Rep Power: 0	New Problems (Toggle Plain Text) std::string Engine::LTrim(std::string trimString) { std::string temp = trimString.c_str(); for(int i = 0; i<= sizeof(temp); i++) //THIS LINE IS PROBLEM #1 { if(temp[i] = ' ') { trimString = ""; for(int k = i+1; k <= sizeof(temp) - (i+1); k++) // THIS LINE { trimString += temp[k]; } } } return trimString; } std::string Engine::LTrim(std::string trimString) { std::string temp = trimString.c_str(); for(int i = 0; i<= sizeof(temp); i++) //THIS LINE IS PROBLEM #1 { if(temp[i] = ' ') { trimString = ""; for(int k = i+1; k <= sizeof(temp) - (i+1); k++) // THIS LINE { trimString += temp[k]; } } } return trimString; } for the code above im getting this warning. I understand the warning just cant follow appropriate steps to overcoming it. (Toggle Plain Text) 1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(56) : warning C4018: '<=' : signed/unsigned mismatch 1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(77) : warning C4018: '<=' : signed/unsigned mismatch 1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(56) : warning C4018: '<=' : signed/unsigned mismatch 1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(77) : warning C4018: '<=' : signed/unsigned mismatch And below i have a stupidity problem.. it will not let me use a string for comparison! (Toggle Plain Text) // PARSE HEADERS FUNCTION bool Engine::ParseFunctionHeaders(std::string script[]) { b_mailFile = false; b_xFile = false; b_samFile = false; b_upload = false; int scriptSize = sizeof(script)/sizeof(script[0]); for(int i = 0; i <= scriptSize; i++) { if(script[i] == "") break; if(Trim(script[i]) == "MAILFILE" && b_mailFile == false) { i_mailFilePos = i; b_mailFile = true; } else if(Trim(script[i]) == "MAILFILE" && b_mailFile == true) { //AddError("[LINE#" + (i+1) + "] ER1003 Multiple Instance (MAILFILE)"); } // THE REST OF THE FUNCTION IS UN-NEEDED // PARSE HEADERS FUNCTION bool Engine::ParseFunctionHeaders(std::string script[]) { b_mailFile = false; b_xFile = false; b_samFile = false; b_upload = false; int scriptSize = sizeof(script)/sizeof(script[0]); for(int i = 0; i <= scriptSize; i++) { if(script[i] == "") break; if(Trim(script[i]) == "MAILFILE" && b_mailFile == false) { i_mailFilePos = i; b_mailFile = true; } else if(Trim(script[i]) == "MAILFILE" && b_mailFile == true) { //AddError("[LINE#" + (i+1) + "] ER1003 Multiple Instance (MAILFILE)"); } // THE REST OF THE FUNCTION IS UN-NEEDED and below it the error (Toggle Plain Text) 1> ] 1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(102) : error C2678: binary '==' : no operator found which takes a left-hand operand of type 'std::basic_string<_Elem,_Traits,_Ax>' (or there is no acceptable conversion) 1> with 1> [ 1> _Elem=char, 1> _Traits=std::char_traits<char>, 1> _Ax=std::allocator<char> 1> ] 1> could be 'built-in C++ operator==(const char [9], const char [9])' 1> c:\program files\microsoft visual studio 8\vc\platformsdk\include\guiddef.h(192): or 'int operator ==(const GUID &,const GUID &)' 1> while trying to match the argument list '(std::basic_string<_Elem,_Traits,_Ax>, const char [9])' 1> with 1> [ 1> _Elem=char, 1> _Traits=std::char_traits<char>, 1> _Ax=std::allocator<char> 1> ] 1> ] 1>c:\documents and settings\knowell\my documents\visual studio 2005\projects\sed\sed\sed.cpp(102) : error C2678: binary '==' : no operator found which takes a left-hand operand of type 'std::basic_string<_Elem,_Traits,_Ax>' (or there is no acceptable conversion) 1> with 1> [ 1> _Elem=char, 1> _Traits=std::char_traits<char>, 1> _Ax=std::allocator<char> 1> ] 1> could be 'built-in C++ operator==(const char [9], const char [9])' 1> c:\program files\microsoft visual studio 8\vc\platformsdk\include\guiddef.h(192): or 'int operator ==(const GUID &,const GUID &)' 1> while trying to match the argument list '(std::basic_string<_Elem,_Traits,_Ax>, const char [9])' 1> with 1> [ 1> _Elem=char, 1> _Traits=std::char_traits<char>, 1> _Ax=std::allocator<char> 1> ] __________________ "When in Rome, Do as the Romans Do" "Beauty is in the eye of the BEER holder" "Save your breath your going to need it for your blow up doll later" SearchLores.org

May 22nd, 2006, 11:07 AM	#3
Kilo Expert Programmer Join Date: Nov 2005 Location: In Pink Clam? Posts: 542 Rep Power: 0	Ok i fixed my problem again by appending ".c_str()" to all "script[i]" Sorry to waste space! I should have known this. EDIT: OH i still have the signed/unsigned warning issue! __________________ "When in Rome, Do as the Romans Do" "Beauty is in the eye of the BEER holder" "Save your breath your going to need it for your blow up doll later" SearchLores.org

May 22nd, 2006, 5:52 PM	#5
Ooble I eat cake for breakfast. Join Date: Jul 2004 Location: In my box. Posts: 4,434 Rep Power: 9	(Toggle Plain Text) int scriptSize = sizeof(script)/sizeof(script[0]); int scriptSize = sizeof(script)/sizeof(script[0]); This will not work. A std::string is not just a collection of characters. It's a linked list of characters with other variables and functions thrown into the mix. Try this: (Toggle Plain Text) #include <iostream> #include <string> int main () { std::string moo = "This is a very big string."; std::cout << "The string 'moo' contains \"" << moo << "\" and is " << sizeof(moo) << " bytes big." << std::endl; } #include <iostream> #include <string> int main () { std::string moo = "This is a very big string."; std::cout << "The string 'moo' contains \"" << moo << "\" and is " << sizeof(moo) << " bytes big." << std::endl; } __________________ Me :: You :: Them

May 22nd, 2006, 6:43 PM	#7
Ooble I eat cake for breakfast. Join Date: Jul 2004 Location: In my box. Posts: 4,434 Rep Power: 9	Ah, crap... ignore me. Must be late. OK, if you want some useful advice, how 'bout this: use a freakin' std::vector instead? __________________ Me :: You :: Them

May 22nd, 2006, 6:54 PM	#9
Ooble I eat cake for breakfast. Join Date: Jul 2004 Location: In my box. Posts: 4,434 Rep Power: 9	I believe that's how the information is stored... you sure I'm wrong? At the lowest level, I could swear it's a wrapper for a linked list. __________________ Me :: You :: Them

May 22nd, 2006, 10:38 AM	#1
Kilo Expert Programmer Join Date: Nov 2005 Location: In Pink Clam? Posts: 542 Rep Power: 0	undeclared identifier... My stupid variables are declared.. this is retarded! (Toggle Plain Text) // located in Engine.h class Engine { public: bool b_mailFile; bool b_xFile = false; bool b_samFile = false; bool b_upload = false; }; // located in cpp file (that includes "engine.h") bool Engine::ParseFunctionHeaders(std::string script[]) { b_mailFile = false; b_xFile = false; b_samFile = false; b_upload = false; return true; } // located in Engine.h class Engine { public: bool b_mailFile; bool b_xFile = false; bool b_samFile = false; bool b_upload = false; }; // located in cpp file (that includes "engine.h") bool Engine::ParseFunctionHeaders(std::string script[]) { b_mailFile = false; b_xFile = false; b_samFile = false; b_upload = false; return true; } could someone please shed some light on my stupidity issue? EDIT: YUP it was stupidity! i guess throwing a prototype of the function into the class declaration would have helped lol __________________ "When in Rome, Do as the Romans Do" "Beauty is in the eye of the BEER holder" "Save your breath your going to need it for your blow up doll later" SearchLores.org

Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

Programming Forums > Application Development > C++

undeclared identifier...