Errors, with vectors, windows programming... etc

Code:

#include <windows.h>
#include <vector>
#define ourKey VK_CONTROL
typedef std:: vector< char > file;

file.push_back('myfile.txt');
file.push_back('myfile2.txt');
file.push_back('pop_c.txt');

int i=0;

int main ( )
{
	

   if ( GetAsyncKeyState( ourKey ) <= -32767 )
   {

	   bool KeyPress = true;

   } else bool KeyPress = false;

   if ( KeyPress == false ) keybd_event( ourKey, );

	for ( i; i<=file.size(); i++ )
	{

		CreateFile(  file[i] , 
					GENERIC_WRITE, 
					0x00000002,  
					NULL,
					2,
					0x80,
					NULL );
	}
   

   
}

Errors:

Code:

(5) : error C2143: syntax error : missing ';' before '.'
(5) : error C2059: syntax error : '.'
(5) : error C2015: too many characters in constant
(6) : error C2143: syntax error : missing ';' before '.'
(6) : error C2059: syntax error : '.'
(6) : error C2015: too many characters in constant
(7) : error C2143: syntax error : missing ';' before '.'
(7) : error C2059: syntax error : '.'
(7) : error C2015: too many characters in constant
(16) : error C2275: 'file' : illegal use of this type as an expression
1>        (3) : see declaration of 'file'
(19) : error C2275: 'file' : illegal use of this type as an expression
1>        (3) : see declaration of 'file'

Dunno what happened, :S can someone help?

> typedef std:: vector< char > file;
Use a std::string, not a char

> file.push_back('myfile.txt');
Use "double" quotes, not 'single'

> int i=0;
... a lot of lines
> for ( i;
This is just an invitation for a lot of later grief. Where is it declared, and is it initialised?
Use
for ( int i = 0

> CreateFile( file[i]
Having changed to a std::string, you'll need file[i].c_str()

Originally Posted by Salem

> typedef std:: vector< char > file;
Use a std::string, not a char

> file.push_back('myfile.txt');
Use "double" quotes, not 'single'

> int i=0;
... a lot of lines
> for ( i;
This is just an invitation for a lot of later grief. Where is it declared, and is it initialised?
Use
for ( int i = 0

> CreateFile( file[i]
Having changed to a std::string, you'll need file[i].c_str()

Thanks, I'll try it.

new errors:

Code:

(6) : error C2143: syntax error : missing ';' before '.'
(6) : error C2059: syntax error : '.'
(22) : error C2275: 'file' : illegal use of this type as an expression
1>        (4) : see declaration of 'file'
(25) : error C2275: 'file' : illegal use of this type as an expression
1>        (4) : see declaration of 'file'
(25) : error C2228: left of '.c_str' must have class/struct/union

I removed the keybd_event till I could come back to it.

Post your latest code

Originally Posted by Salem

Post your latest code

Code:

#include <windows.h>
#include <vector>
#include <string>
#define ourKey VK_CONTROL
typedef std:: vector< std::string > file;

file.push_back("myfile.txt");


int main ( )
{
	

   if ( GetAsyncKeyState( ourKey ) <= -32767 )
   {

	   bool KeyPress = true;

   } else bool KeyPress = false;



	for ( int i = 0; i<=file.size(); i++ )
	{

		CreateFile(  file[i].c_str() , 
					GENERIC_WRITE, 
					0x00000002,  
					NULL,
					2,
					0x80,
					NULL );
	}
   

   
}

> file.push_back("myfile.txt");
Put this inside main() perhaps?

The error is that you don't instantiate an object of your 'vector<char>', but that you just give this specialisation ('char') of the template 'vector' an alias: typedef.
You should instantiate an object before you can call member functions like "push_back" :

Code:

typedef std:: vector<char> file; // file is not the object, but a rename
file myfile; // myfile is now the object
myfile.push_back(..)

Second, your loop loops one time too much: try

Code:

	
for ( int i = 0; i<file.size(); i++ ) // no = !!

If you count from 0, a vector with N elements has 0..N-1 entries.

Originally Posted by MarkZWEERS

The error is that you don't instantiate an object of your 'vector<char>', but that you just give this specialisation ('char') of the template 'vector' an alias: typedef.
You should instantiate an object before you can call member functions like "push_back" :

Code:

typedef std:: vector<char> file; // file is not the object, but a rename
file myfile; // myfile is now the object
myfile.push_back(..)

Second, your loop loops one time too much: try

Code:

	
for ( int i = 0; i<file.size(); i++ ) // no = !!

If you count from 0, a vector with N elements has 0..N-1 entries.

But I don't understand, I'm not using frstream/ofstream/etc for my files, I'm using windows/com CreateFile function, the vector is just used so I can store an infinite amount of elements ( file names ), so I can create unlimited files from how many push_backs()'s I have, using for loop.

As Mark tries to explain to you:
A typedef is an alias. You are creating an alias of the name of the class. A class is a blueprint; not an object. You cannot work on things that are not objects.
So you must create an object of the type first.

Try
std::vector<std::string> file;

This will work because it actually creates a vector of strings.

This
typedef std::vector<std::string> file;

Merely says that you can now type "file" instead of "std::vector<std::string>". An alias.

It works none the less, thanks!