wrapper of methods

[jacek]

Isn't ................._vec.at(0). an instance of a class? I don't understand.....

Yes, it it, but "objA._vec.at(0).fptr[0]" is an expression which returns a pointer to member. It doesn't mean "invoke the method that fptr[0] points to". As caduel wrote you have to use special syntax to invoke that method.

Can you give me an example of what you mean with overload an operator for this case?

The user of your class wants to perform some action depending on integer value. If you make him use pointers to members everywhere the code will be error-prone and unreadable. It is better to hide this inside your class and give your user only a convenient method and an enum:

double db = obj.get( B::GeometricMean );
// or using an operator:
double db = obj[ B::GeometricMean ];

double db = obj.get( B::GeometricMean );
// or using an operator:
double db = obj[ B::GeometricMean ];

To copy to clipboard, switch view to plain text mode 

[mickey]

hello,
I understand the alternative but not why I shouldn't use pointertomembers.
Anyway: What do you suggest for this case? The same?

class A {
public:
     trasformData1() { ......}
     trasformData2() {..........}
     trasformData3() { .....}
     //maybe here is better use another enum and one "TrasformData(enum);" ? I mean in the same way below....
//trasFormData1,2,3 here are very different
};
class B {
   vector<A> _va;
   enum trasf { trasf1, trasf2, trasf3; }
public:
   	void trasform( trasf  n) { 
		switch (n) {
			case trasf1 :
                        //loop on _va and call "_va.trasformData1();"
                        break;
			case trasf2 : "call _va.trasformData2();" break;
			case trasf3 :"call _va.trasformData3();" break;
			default: exit(-1) //error
		};	
	}
};
//main.cpp
B b;
//fill the vector
//the type of trasformation is chose at command line
b.trasform(B::Trasf1);

class A {
public:
     trasformData1() { ......}
     trasformData2() {..........}
     trasformData3() { .....}
     //maybe here is better use another enum and one "TrasformData(enum);" ? I mean in the same way below....
//trasFormData1,2,3 here are very different
};
class B {
   vector<A> _va;
   enum trasf { trasf1, trasf2, trasf3; }
public:
   	void trasform( trasf  n) { 
		switch (n) {
			case trasf1 :
                        //loop on _va and call "_va.trasformData1();"
                        break;
			case trasf2 : "call _va.trasformData2();" break;
			case trasf3 :"call _va.trasformData3();" break;
			default: exit(-1) //error
		};	
	}
};
//main.cpp
B b;
//fill the vector
//the type of trasformation is chose at command line
b.trasform(B::Trasf1);

To copy to clipboard, switch view to plain text mode 

[jacek]

why I shouldn't use pointertomembers.

They're ugly, so if you want ot use them, hide them inside the class.

Anyway: What do you suggest for this case? The same?

How about this?

TransformationFactory::TransformationFactory() 
{
   transformations[ SomeTransformation ] = new SomeTransformation();
   ...
}
 
const Transformation * TransformationFactory::get( TransformationType type ) 
{
   return transformations[ type ];
}
 
...
 
transformedVector = factory.get( SomeTransformation )->transform( originalVector );

TransformationFactory::TransformationFactory() 
{
   transformations[ SomeTransformation ] = new SomeTransformation();
   ...
}

const Transformation * TransformationFactory::get( TransformationType type ) 
{
   return transformations[ type ];
}

...

transformedVector = factory.get( SomeTransformation )->transform( originalVector );

To copy to clipboard, switch view to plain text mode 

[mickey]

How about this?

TransformationFactory::TransformationFactory() 
{
   transformations[ SomeTransformation ] = new SomeTransformation();
   ...
}
const Transformation * TransformationFactory::get( TransformationType type ) 
{
   return transformations[ type ];
}
transformedVector = factory.get( SomeTransformation )->transform( originalVector );

TransformationFactory::TransformationFactory() 
{
   transformations[ SomeTransformation ] = new SomeTransformation();
   ...
}
const Transformation * TransformationFactory::get( TransformationType type ) 
{
   return transformations[ type ];
}
transformedVector = factory.get( SomeTransformation )->transform( originalVector );

To copy to clipboard, switch view to plain text mode 

Sorry, but I don't understand your example. My trasformation1,2,3 inside A are methods and not object (the do a normalization of the data).....maybe you understood they're objects? Isn't this a "pointer to member" way? I thought that you didn't suggest to use pointer to member..

[jacek]

My trasformation1,2,3 inside A are methods and not object (the do a normalization of the data).....

Yes and my proposition is to change them into objects.

Isn't this a "pointer to member" way? I thought that you didn't suggest to use pointer to member..

It's very similar, but you can avoid that ugly syntax. Pointers to members will be OK too if you hide them behind some nice interface.