Thread: Array of structures in a class, init

Hi,

Is there any simpler way of initiating an array used in a class than looping or single instance init.
I want to do EXACTLY as the code states but I don't get the damn thing working.

Code:

typedef void InstructionProc(uint32_t icode);

typedef struct Instruction {
        uint32_t mask;
        uint32_t value;
        char *name;
        InstructionProc *proc;
} Instruction;


class test {
Instruction instr[];
}



test::test() {

instr[] = {
	{0x0de00000,0x00a00000,"adc"    ,i_adc}
};

}

/S

In C++, structs are almost exactly the same as classes (default members and access is public, classes default is private). This means you can have a clever little struct constructor. This means you could have something like:

Code:

typedef void InstructionProc(uint32_t icode);

void i_adc(uint32_t meh) 
{ 
	// We're here
}
struct Instruction {
	uint32_t mask;
	uint32_t value;
	char *name;
	InstructionProc *proc;


	Instruction(uint32_t mask, uint32_t value, char* name, InstructionProc proc)
	{
		this->mask = mask;
		this->value = value;
		this->name = name;
		this->proc = proc;
	}
	Instruction()
	{
		// default values here?
	}
};

class Test
{
public:
	Test()
	{
		instructionsA = new Instruction[50];
		for(int i = 0; i < 50; i++)
		{
			instructionsA[i] = Instruction(0x0de00000, 0x00a00000, "adc", i_adc);
			instructionsB[i] = Instruction(0x0de00000, 0x00a00000, "adc", i_adc);
			instructionsA[i].proc(i);
			instructionsB[i].proc(i);
		}
	}
	~Test() 
	{ 
		delete instructionsA; 
	}

private:
	Instruction* instructionsA;
	Instruction instructionsB[50];
};

Not that I know what this will be doing, doesn't have much direction on it yet really

In addition to the two potential solutions Tonto gave, you could also use a vector, which can be intialized with the appropriate values instead of requiring a for loop:

Code:

#include <vector>
 
// Assume same Instruction definition as Tonto's code
 
class Test
{
public:
	Test() : instructionsC(50, Instruction(0x0de00000, 0x00a00000, "adc", i_adc))
	{ }
	// Not Needed: ~Test() { }
 
private:
	std::vector<Instruction> instructionsC;
};

I want something like this, but instructionA.proc(i) does not evaluate to a function, why?

Code:

typedef unsigned int uint32_t;

class Test;

typedef void (Test::*InstructionProc)(uint32_t icode);


struct Instruction {
	uint32_t mask;
	uint32_t value;
	char *name;
	InstructionProc proc;


	Instruction(uint32_t mask, uint32_t value, char* name, InstructionProc proc)
	{
		this->mask = mask;
		this->value = value;
		this->name = name;
		this->proc = proc;
	}
	Instruction()
	{
		// default values here?
	}
};

class Test
{

public:
	Test()
	{
		instructionsA = new Instruction[50];
		for(int i = 0; i < 50; i++)
		{
			instructionsA[i] = Instruction(0x0de00000, 0x00a00000, "adc", &Test::i_adc);

		//	instructionsB[i] = Instruction(0x0de00000, 0x00a00000, "adc", i_adc);
		//	instructionsA[i].proc(i);
		//	instructionsB[i].proc(i);
		}
	}
	~Test() 
	{ 
		delete instructionsA; 
	}

	void i_adc(uint32_t opcode) {
	}

	void execute() {

	instructionsA[0].proc(0x02000000);
		
	}

private:
	Instruction* instructionsA;
	Instruction instructionsB[50];

};

void main() {


Test *cpu;
cpu = new Test();

cpu->execute();

}

why should that fix anything...hmmm?

hmmm....maybe you could correct it then...cause i want to point to a Test memberfunction!!!
How can I do that?

Please!!

but i doesn't solve the problem!!!

Greatly appreciate your help.....but I have one more dilemma

I have a function which returns a InstructionProc

Code:

inline InstructionProc Test::
		InstructionProcFind(uint32_t icode) {
        int index=INSTR_INDEX(icode); //just picks out some bits to be used as an index
	InstructionProc proc=iProcTab[index];
//iProcTab is an array of InstructionProc's

	return proc;
}

then I want to be able to run this like

Code:

InstructionProc iproc;
iproc=InstructionProcFind(1234);
iproc(1234);

i know it sounds fishy but i need to get this running in this way, and then I can figure out how to make it smoother!!!

Same thing you need an instance.

Code:

InstructionProc iproc;
iproc=InstructionProcFind(1234);
 (*this.*iproc)(1234);

have a look at this

Kurt

greatly appreciate your help....thanks alot