U++ framework
Do not panic. Ask here before giving up.

Home » U++ Library support » U++ Core » NEW: generic Toupel grouper
NEW: generic Toupel grouper [message #27972] Thu, 12 August 2010 16:03 Go to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
sometimes its cool to group things together in an easy manner, especially when wroking with the template containers. to build a grouping container over and over again each time for different purposes, just for classes that would have the parts public anyway,is odd.

say you want to have a vector that simply contains a 2-tupel of values. maybe some measurements consisting of two values at once, or anything that logically belongs together but does not need any abstraction or scope hiding. a normal container cant do it.

Vector<float, float> vi; //would resemble VectorMap.


you just need to quickly define and access 2 things at once in a container and want to have sth like
Vector<Duo<int, int> > vi;
...
vi[i].t1 = 123;
vi[i].t2 = 234;

this looks good and is handy.

so here comes some helpers to do that

template<class T>
class Solo
{
public:
	typedef Solo<T> CLASSNAME;
	Solo(const T & _t) : t(_t) {}
	Solo() {}

	operator T & () { return t; }
	operator const T & () const { return t; }
	
	T t;
};

template<class T1, class T2>
class Duo
{
public:
	typedef Duo<T1, T2> CLASSNAME;
	Duo(const T1 & _t1, const T2 & _t2) : t1(_t1), t2(_t2) {}
	Duo() {}

	operator T1 & () { return t1; }
	operator const T1 & () const { return t1; }
	operator T2 & () { return t2; }
	operator const T2 & () const { return t2; }
	
	T1 t1;
	T2 t2;
};

template<class T1, class T2, class T3>
class Trio
{
public:
	typedef Trio<T1, T2, T3> CLASSNAME;
	Trio(const T1 & _t1, const T2 & _t2, const T3 & _t3) : t1(_t1), t2(_t2), t3(_t3) {}
	Trio() {}

	operator T1 & () { return t1; }
	operator const T1 & () const { return t1; }
	operator T2 & () { return t2; }
	operator const T2 & () const { return t2; }
	operator T3 & () { return t3; }
	operator const T3 & () const { return t3; }
	
	T1 t1;
	T2 t2;
	T3 t3;
};

template<class T1, class T2, class T3, class T4>
class Quartett
{
public:
	typedef Quartett<T1, T2, T3, T4> CLASSNAME;
	Quartett(const T1 & _t1, const T2 & _t2, const T3 & _t3, const T4 & _t4) : t1(_t1), t2(_t2), t3(_t3), t4(_t4) {}
	Quartett() {}

	operator T1 & () { return t1; }
	operator const T1 & () const { return t1; }
	operator T2 & () { return t2; }
	operator const T2 & () const { return t2; }
	operator T3 & () { return t3; }
	operator const T3 & () const { return t3; }
	operator T4 & () { return t4; }
	operator const T4 & () const { return t4; }
	
	T1 t1;
	T2 t2;
	T3 t3;
	T4 t4;
};

maybe they can go to Others.h..
Re: NEW: generic Toupel grouper [message #27974 is a reply to message #27972] Thu, 12 August 2010 16:07 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
or, which would be better maybe, to have Two<T1, T2>, Three<T1,T2,T3>, Four<T1,T2,T3,T4> as analoge to One<T>..

but sometimes that much of complexity is not needed
Re: NEW: generic Toupel grouper [message #27976 is a reply to message #27974] Thu, 12 August 2010 16:14 Go to previous messageGo to next message
koldo is currently offline  koldo
Messages: 3461
Registered: August 2008
Senior Veteran
Hello Kohait

This does not seem bad. However is it worthwhile the added code to support it?, as we always can insert in a Vector any class or struct.


Best regards
Iñaki
Re: NEW: generic Toupel grouper [message #27978 is a reply to message #27976] Thu, 12 August 2010 16:58 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
ofcorse..but imagine, you need to setup a new class each time you simply just want to group/pack some things together without further class implications / namespaces / accessscopes. i imagine this to be quite often the case. (Point_ is not quite the same but is a small example of grouping things)

but here comes another option, which is maybe better...donnow.

template<class T>
class O
{
public:
	typedef O<T> CLASSNAME;
	O(const T & _t) : t(_t) {}
	O() {}

	operator T & () { return t; }
	operator const T & () const { return t; }

	T t;
};

template<class T>
class O1 : public O<T> {};

template<class T1, class T2>
class O2 : public O<T1>, public O<T2> {};

template<class T1, class T2, class T3>
class O3 : public O<T1>, public O<T2>, public O<T3> {};

template<class T1, class T2, class T3, class T4>
class O4 : public O<T1>, public O<T2> , public O<T3> , public O<T4> {};


beeing able to access stuff like this, which is more clear
	O2<int, float> o2;
	
	o2.O<int>::t = 123;
	o2.O<float>::t = 23.10f;

[Updated on: Thu, 12 August 2010 17:03]

Report message to a moderator

Re: NEW: generic Toupel grouper [message #27979 is a reply to message #27978] Thu, 12 August 2010 17:03 Go to previous messageGo to next message
koldo is currently offline  koldo
Messages: 3461
Registered: August 2008
Senior Veteran
Shocked -> Smile

Best regards
Iñaki
Re: NEW: generic Toupel grouper [message #27980 is a reply to message #27979] Thu, 12 August 2010 17:04 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
even got another idea..

template<class T1, class T2>
class Two : public One<T1>, public One<T2> {};

template<class T1, class T2, class T3>
class Three : public One<T1>, public One<T2>, public One<T3> {};

template<class T1, class T2, class T3, class T4>
class Four : public One<T1>, public One<T2>, public One<T3>, public One<T4> {};

EDIT: too quick Smile forgot a type

	Four<int, char, float, unsigned> four;
	four.One<int>::Create() = 123;

[Updated on: Thu, 12 August 2010 17:20]

Report message to a moderator

Re: NEW: generic Toupel grouper [message #27982 is a reply to message #27980] Thu, 12 August 2010 17:27 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
a bit more usable Smile
template<class T>
class O
{
public:
	typedef O<T> CLASSNAME;
	O(const T & _t) : t(_t) {}
	O() {}

	operator T & () { return t; }
	operator const T & () const { return t; }
	
	T& operator=(const T & _t) { t = _t; return t; }
	bool operator ==(const T& _t) const { return (t==_t); }
	
	T t;
};
Re: NEW: generic Toupel grouper [message #27985 is a reply to message #27982] Thu, 12 August 2010 20:14 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1796
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

Hi kohait00,

I've got two questions Wink

1) Does it work when you need to group several values of same type? E.g. Three<String,int,int>. I have a suspicion that the overloaded operators for int wouldn't make much sense than...

2) What is the advantage of your touples to using simple Vector<Value>? The later might have bit more verbose interface, but on the other hand it offers variable element count.

Best regards,
Honza
Re: NEW: generic Toupel grouper [message #27986 is a reply to message #27985] Thu, 12 August 2010 20:58 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
hey doli, you are absolutely right with your point 1)
but in this case Duo<int,int> work, becasue there the items are named explicitely.

de advantage is, that zou dont always want to store all data inside Value, but as an explicit tzype.

but using same tzpe one might think of using Duo<String, Vector<int> >, grouping it Smile

it is still a development, so expect some changes here. but the idea is to loosely group stuff together without the need to create a whole new class for it.. its maybe considred a class template, instead of a template class. a public members class.

cheers
Re: NEW: generic Toupel grouper [message #28015 is a reply to message #27986] Fri, 13 August 2010 10:53 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Actually, I think this feature is somewhat missing in U++ (and is quite common in other frameworks).

But I am quite sceptical about those cast overloads too.

Imo, something as simple as

template<class T1, class T2>
struct Pair {
Pair(const T1 & _t1, const T2 & _t2) : t1(_t1), t2(_t2) {}
Pair() {}

T1 first;
T2 second;
};

would solve most of usage scenarios where I was eventually missing tuples.

Maybe first/second are bad names though...

Hm, maybe, what about

template<class T1, class T2>
struct AB {
AB(const T1 & _t1, const T2 & _t2) : t1(_t1), t2(_t2) {}
AB() {}

T1 a;
T2 b;
};

? Smile

Here is boost's take on the issue:

http://www.boost.org/doc/libs/1_34_0/libs/tuple/doc/tuple_us ers_guide.html

IMO a little bit overengineered...
Re: NEW: generic Toupel grouper [message #28017 is a reply to message #28015] Fri, 13 August 2010 10:58 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
simple is most times better Smile

good option. maybe AB, ABC is not 'verbose' enough (as of readability of code, but tfcorse a short option)

Re: NEW: generic Toupel grouper [message #28026 is a reply to message #28017] Fri, 13 August 2010 11:16 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
Quote:


But I am quite sceptical about those cast overloads too.


its just for easy handling, while opening doors to implicit casted assigns to wrong parameter (dont know if compiler would warn in such case, which operator() to take)

can be left out i think. if user uses it, he can type some 2 more letters and be on safe side. if he really needs this behaviour, polimophism is a good thing..
Re: NEW: generic Toupel grouper [message #28037 is a reply to message #28026] Fri, 13 August 2010 14:59 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
i found AB, ABC etc. beeing difficult to use, because wingdi.h already defines ABC. Confused

ive looked into the boost stuff, it's really a bit over the line.
beeing on the secure side and type one letter is cool enough..

so this is a proposal, short enough not to blow Others.h

template<class T1, class T2>
class Duo
{
public:
	Duo(const T1 & _a, const T2 & _b) : a(_a), b(_b) {}
	Duo() {}
	T1 a; T2 b;
};

template<class T1, class T2, class T3>
class Trio
{
public:
	Trio(const T1 & _a, const T2 & _b, const T3 & _c) : a(_a), b(_b), c(_c) {}
	Trio() {}
	T1 a; T2 b; T3 c;
};

template<class T1, class T2, class T3, class T4>
class Quartett
{
public:
	Quartett(const T1 & _a, const T2 & _b, const T3 & _c, const T4 & _d) : a(_a), b(_b), c(_c), d(_d) {}
	Quartett() {}
	T1 a; T2 b; T3 c; T4 d;
};

[Updated on: Fri, 13 August 2010 14:59]

Report message to a moderator

Re: NEW: generic Toupel grouper [message #28053 is a reply to message #28037] Fri, 13 August 2010 20:21 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1796
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

The boost implementation is really overkill. But one thing I like about it is the "indexed" access using the get<N>() function.

Also, something like
//for Two (similar for bigger touples):
	Value operator[](int i)const{
		ASSERT(i>=0&&i<2);
		if(i==0) return a;
		else return b;
	}
would be nice thing to have.

Honza
Re: NEW: generic Toupel grouper [message #28058 is a reply to message #27972] Sat, 14 August 2010 01:40 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1796
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

So, after criticizing kohaits proposals I felt obligated to show my own idea about how Touples should be implemented... Here is the result:
#include <Core/Core.h>
using namespace Upp;

String AsString(Nuller n){return "";}

template <int N,class A,class B,class C> struct TTypes          {typedef Nuller Type;};
template <class A,class B,class C>       struct TTypes<0,A,B,C> {typedef A Type;};
template <class A,class B,class C>       struct TTypes<1,A,B,C> {typedef B Type;};
template <class A,class B,class C>       struct TTypes<2,A,B,C> {typedef C Type;};

template<class A, class B,class C> struct Touple;

template<int N,class A,class B,class C> struct Retriever{
	static Nuller Get(Touple<A,B,C>& t){return Null;};
};
template<class A,class B,class C> struct Retriever<0,A,B,C>{
	static typename TTypes<0,A,B,C>::Type Get(Touple<A,B,C>& t){return t.a;};
};
template<class A,class B,class C> struct Retriever<1,A,B,C>{
	static typename TTypes<1,A,B,C>::Type Get(Touple<A,B,C>& t){return t.b;};
};
template<class A,class B,class C> struct Retriever<2,A,B,C>{
	static typename TTypes<2,A,B,C>::Type Get(Touple<A,B,C>& t){return t.c;};
};

template<class A, class B=Nuller,class C=Nuller>
struct Touple{
	A a;
	B b;
	C c;
	Touple(){};
	Touple(A a):a(a){};
	Touple(A a,B b):a(a),b(b){};
	Touple(A a,B b,C c):a(a),b(b),c(c){};
	
	template<int N>
	typename TTypes<N,A,B,C>::Type Get(){
		return Retriever<N,A,B,C>::Get(*this);
	};
	template<class T>
	Touple& operator=(const T& t){
		a=t.a; b=t.b; c=t.c;
	}
	int GetCount()const{
		for(int i=2; i>0; i--){
			if((*this)[i]!=Value(Null)) return i+1;
		}
		return 1;
	}
	Value operator[](int i)const{
		if     (i==0) return Value(a);
		else if(i==1) return Value(b);
		else if(i==2) return Value(c);
		else ASSERT_(false,"index out of bounds");
	}
	String ToString()const{
		String s=AsString(c);
		s=AsString(b)+(s.GetLength()>0?",":"")+s;
		s=AsString(a)+(s.GetLength()>0?",":"")+s;
		return "{"+s+"}";
	}
};

template<class A>
Touple<A> Solo(const A& a){
	return Touple<A>(a);
};

template<class A,class B>
Touple<A,B> Duo(const A& a,const B& b){
	return Touple<A,B>(a,b);
};

template<class A,class B,class C>
Touple<A,B,C> Trio(const A& a,const B& b,const C& c){
	return Touple<A,B,C>(a,b,c);
};

CONSOLE_APP_MAIN{
	Touple<double,const char*> s;
	Touple<int,String> t;
	Touple<int,String,double> u;
	
	t.a=1; t.b="test";
	DUMP(t.Get<0>()); DUMP(t.Get<1>());
	for(int i=0; i<t.GetCount(); i++){
		LOG(i<<": "<<t[i]);
	}
	s=t;                             DUMP(s);
	t=Duo(1,String("hello world"));  DUMP(t);
	u=t;                             DUMP(u);
	u.c=3.2;                         DUMP(u);
	// <double>=<triple> fails to compile:
	// t=Triple(1,String("dsd"),3);  DUMP(t);
}


The main difference is that there is no specific type for two,three, etc. values, but rather a single Touple class that can be used universally. The implementation above allows 1,2 or 3 elements, but can be easily extended. The main idea is that smaller touple can be assigned into bigger (extra elements are Null), while bigger into smaller triggers compilation errors. The functions Solo,Duo and Trio are supposed to save you some typing by generating the touples based on the arguments types. The only thing I found missing in Core was AsString(Nuller) which should be no problem to add.

The ideas from my previous post are included. I am aware that touple.Get<0>() doesn't have any significant syntactic value, since it is the same as touple.a, but it might help the readability a bit. On the other hand, the operator[] is quite important, since it allows iterating through the Touple. TEven though it returns Values, together with GetCount() (returning number of elements from 0 to last non-Null) should be quite powerfull tool.

What do you think?

Honza

PS: The Solo,Duo and Trio names were chosen just because there already is One and Single. Otherwise I would prefer Single, Double, etc.
Re: NEW: generic Toupel grouper [message #28064 is a reply to message #28053] Sat, 14 August 2010 11:53 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
dolik.rce wrote on Fri, 13 August 2010 14:21

The boost implementation is really overkill. But one thing I like about it is the "indexed" access using the get<N>() function.

Also, something like
//for Two (similar for bigger touples):
	Value operator[](int i)const{
		ASSERT(i>=0&&i<2);
		if(i==0) return a;
		else return b;
	}
would be nice thing to have.

Honza


You expect too much about types involved here IMO.
Re: NEW: generic Toupel grouper [message #28066 is a reply to message #28064] Sat, 14 August 2010 12:25 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1796
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

luzr wrote on Sat, 14 August 2010 11:53

dolik.rce wrote on Fri, 13 August 2010 14:21

The boost implementation is really overkill. But one thing I like about it is the "indexed" access using the get<N>() function.

Also, something like
//for Two (similar for bigger touples):
	Value operator[](int i)const{
		ASSERT(i>=0&&i<2);
		if(i==0) return a;
		else return b;
	}
would be nice thing to have.

Honza


You expect too much about types involved here IMO.

I am aware of that. But if I am not mistaken, it doesn't affect functionality of the class too much. The compilation should fail only in case when the operator is actually used, since it is templated. If user knows the type is not compatible, he will just have to resort back to using the members directly. There should be no or minimal performance penalty for that... Am I right?

Moreover, there are some further assumption made anyway, about compatibility with Null. That might be even more restricting...

Honza
Re: NEW: generic Toupel grouper [message #28091 is a reply to message #28058] Sun, 15 August 2010 09:16 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
Quote:


Otherwise I would prefer Single, Double, etc.


there, similarities to data types wouled be to close (Double) and misleading. Solo is not usefull anyway (wraping one T is .. nonsense), it should start with Duo, Trio, etc.. to set Apart from the One (and maybe later Two, Three)

Quote:


after criticizing kohaits proposals


dont worry, it's part of development, and didnt feel criticised Smile. i am always looking forward to meeting better ideas..

to the above implementation:

the Get<1>() option (idea from boost?) is a cool trick, but IMHO of little use because it's not a runtime check, but a compile time definition, thus u.a, u.b, u.c is much simpler and clearer in that sense, and less to type anyway. i mean, in terms of compile time specialisation u.Get<1> is same as u.a, you have to provide the index at compiletime, so you know which type.

the Value operator[](int i) is a good idea though. to wrap / unwrap in value (boxing / unboxing is used in C# and others, though there in different context, as base class object).

having Duo, Trio, etc is, as you pointed out, more or less useless, even if it's better to read Smile so i added a 5th T and deaulted past second T. (i'd rater use EmptyClass, but there is no Value(const EmptyClass &) for it, so i used Nuller. might be usefull to have an EmptyClass Value as well?)

so here comes another option.

template<class T1, class T2, class T3=Nuller, class T4=Nuller, class T5=Nuller>
class Tupel
{
public:
	Tupel(const T1 & _a, const T2 & _b, const T3 & _c, const T4 & _d, const T5 & _e) 
		: a(_a), b(_b), c(_c), d(_d), e(_e) {}
	Tupel() {}

	Value operator[](int i) {
		switch(i) {
			case 1: return Value(a);	
			case 2: return Value(b);	
			case 3: return Value(c);	
			case 4: return Value(d);	
			case 5: return Value(e);	
			default:
			case 0: ASSERT(0); return Value();	
		}
		return Value(); //dummy
	}

	T1 a; T2 b; T3 c; T4 d; T5 e;
};


what about this one? it provides the simplicity desired and has the wrapper.

Re: NEW: generic Toupel grouper [message #28092 is a reply to message #28091] Sun, 15 August 2010 13:09 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1796
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

Hi Kohait

kohait00 wrote on Sun, 15 August 2010 09:16

dont worry, it's part of development, and didnt feel criticised Smile. i am always looking forward to meeting better ideas..

I don't worry, it was a joke Wink

kohait00 wrote on Sun, 15 August 2010 09:16


the Get<1>() option (idea from boost?) is a cool trick, but IMHO of little use because it's not a runtime check, but a compile time definition, thus u.a, u.b, u.c is much simpler and clearer in that sense, and less to type anyway. i mean, in terms of compile time specialisation u.Get<1> is same as u.a, you have to provide the index at compiletime, so you know which type.

Yes, it is from boost. As I said, it is mostly useless and the only way it might be helpful is making the code look better and hopefully better readable. But I don't insist on having it at all. At least it learned me some interesting new things about templates Smile

kohait00 wrote on Sun, 15 August 2010 09:16


the Value operator[](int i) is a good idea though. to wrap / unwrap in value (boxing / unboxing is used in C# and others, though there in different context, as base class object).

Don't forget about the GetCount() too Wink I just don't like my implementation of it very much, but I can't come up with anything better. And if possible I would also like to see Begin() and End() implemented, so I could do DUMPC(touple)...

kohait00 wrote on Sun, 15 August 2010 09:16


having Duo, Trio, etc is, as you pointed out, more or less useless, even if it's better to read Smile so i added a 5th T and deaulted past second T. (i'd rater use EmptyClass, but there is no Value(const EmptyClass &) for it, so i used Nuller. might be usefull to have an EmptyClass Value as well?)

I would strongly prefer EmptyClass too. But there might be idealogical problem: Once you make it value compatible, it won't be empty any more Smile Maybe we should do a special class for this purpose, let's say DummyElement, which would be Value and Null compatible.

Apart from what I said above, especially the missing GetCount(), your last code seems reasonable. Definitely not that difficult to read as mine Smile (which is good)

Honza

[Updated on: Sun, 15 August 2010 13:10]

Report message to a moderator

Re: NEW: generic Toupel grouper [message #28128 is a reply to message #28092] Tue, 17 August 2010 22:22 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
Quote:


learned me some interesting new things about templates


i'm always looking to learn as well Smile

Quote:


Don't forget about the GetCount()


i dont think that it is that much of use either. becasue everything is know at compiletime, and Tupel is not meant to be a base class. so i cant imagine any practicle usecase. i'd prefer to, as mirek statet, have it as simple as possible, and serve its due as beeing a compile time known simple strong type container / grouper of data types. not more nor less, except there is more things to have on it, that are of real use.

EmptyClass would be great, but..your point is really good. it wouldnt be empty. but thinking of Nuller as base class in that field is ok too. actually it is a logical requirement, when enabling the Value wrapping of the types. so it's not that tragic.

what do you guys think? is there more to add to this one?
should i add it to bazaar or will it join Others.h?

(I'm trying to 'empty' my development nest, which in time has grown with some little things maybe usefull)
Re: NEW: generic Toupel grouper [message #28129 is a reply to message #28128] Tue, 17 August 2010 22:52 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1796
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

kohait00 wrote on Tue, 17 August 2010 22:22

Quote:

Don't forget about the GetCount()
i dont think that it is that much of use either. becasue everything is know at compiletime, and Tupel is not meant to be a base class. so i cant imagine any practicle usecase. i'd prefer to, as mirek statet, have it as simple as possible, and serve its due as beeing a compile time known simple strong type container / grouper of data types. not more nor less, except there is more things to have on it, that are of real use.

The main reason I want GetCount() is that it is in every container, from Vector to String. You are right that everything is known at compile time, but it doesn't really mean that you know everything when you write some code Smile E.g.: Sometimes there are situations where you create templated function which iterates through a container. It doesn't even have to be you who uses the code in the end.

One of the reasons why I like U++ so much is its consistency of interfaces. I feel safe to write template or macro that is doing something general with only basic assumptions about interface, often without knowing what some crazy user of my code will send to it Smile And it is not only about iterating in containers - also ToString (DUMP and LOG are great example of what I tried to describe in previous paragraph), Serialize, Null constructors, operator<<= etc.

Honza

Re: NEW: generic Toupel grouper [message #28131 is a reply to message #28129] Tue, 17 August 2010 23:21 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
well it's hard to think of Tupel beeing a 'iterable' container.
but i can 'feel' the need somehow as well, especially looking at Value operator[]. so here the type is actually hidden. here is another possib for GetCount()


	int GetCount() const
	{
		int c = 2;
		if(typeid(T3) != typeid(Nuller)) ++c;
		if(typeid(T4) != typeid(Nuller)) ++c;
		if(typeid(T5) != typeid(Nuller)) ++c;
		return c;
	}

but doesnt save us from things like
Typel<int, int, Nuller, float> t; //float type unreachable
for(int i = 0; i<t.GetCount(); i++)
   Value v = t[i];

but here, the user is dumb Smile
Re: NEW: generic Toupel grouper [message #28133 is a reply to message #28131] Tue, 17 August 2010 23:53 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1796
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

I would use
int GetCount() const
	{
		if(typeid(T5) != typeid(Nuller)) return 5;
		if(typeid(T4) != typeid(Nuller)) return 4;
		if(typeid(T3) != typeid(Nuller)) return 3;
		return 2;
	}

That works even for the dumb user Wink
Honza
Re: NEW: generic Toupel grouper [message #28152 is a reply to message #28133] Thu, 19 August 2010 08:30 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
thats way better. one would need to check for intermediate Nuller though anyway, beeing that the case, why not simply do this Smile
inline int GetCount() const { return 5; }


so it's up to mirek to decide what happens with this stuff

[Updated on: Thu, 19 August 2010 08:30]

Report message to a moderator

Re: NEW: generic Toupel grouper [message #28365 is a reply to message #28152] Mon, 30 August 2010 20:57 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
kohait00 wrote on Thu, 19 August 2010 02:30

thats way better. one would need to check for intermediate Nuller though anyway, beeing that the case, why not simply do this Smile
inline int GetCount() const { return 5; }


so it's up to mirek to decide what happens with this stuff



I believe it is not very helpful to mix Tuples with Value or try to pretend they are maps. In fact, ValueArray covers such usage pretty well IMO.

I believe that the common usage scenario is in cases where you need "quick" struct which is not worth defining, like

static Tuple<int, const char *> mapping[] = {
   0, "foo",
   1, "bar"
}


In fact, the only think to resolve is how to name its members.

Right now, "a, b, c, d, e..." sound like the best option. STL templated indexing seems overengineered to me, "first, second, third..." are too long and "v0, v1, v2, v3..." have "zero index issue".

Well, maybe something like "key, value, value1, value2" would reflect the most typical use.

More ideas? Or letters are it?
Re: NEW: generic Toupel grouper [message #28366 is a reply to message #28365] Mon, 30 August 2010 22:23 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
i'd prefer a,b,c,d,e...
its just sweeter Smile and one can give it the meaning it needs, key, value...is too specific already
Re: NEW: generic Toupel grouper [message #28367 is a reply to message #27972] Mon, 30 August 2010 23:13 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1796
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

I agree with the a,b,c,d,e... too. It is nice and simple and fast to write. One more question is how many values should be supported. I personally think that a-f should be about enough...

Honza
Re: NEW: generic Toupel grouper [message #28369 is a reply to message #28367] Tue, 31 August 2010 08:31 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
if more is needed, this should be possible.. Smile
Tupel<int,float,int,float,Tupel<String,char,String,char,double> >


in terms of Tupel as container: Tupel is actually no container..here mirek is right.its a grouper class. anyway, in case such a behaviour is needed, it can be subsequently added deriving.
Re: NEW: generic Toupel grouper [message #28370 is a reply to message #28369] Tue, 31 August 2010 08:44 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
so here is a cleaned up proposal..
template<class T1, class T2, class T3=EmptyClass, class T4=EmptyClass, class T5=EmptyClass>
class Tupel {
public:
	Tupel(const T1 & a, const T2 & b, const T3 & c, const T4 & d, const T5 & e) 
		: a(a), b(b), c(c), d(d), e(e) {}
	Tupel() {}
	T1 a; T2 b; T3 c; T4 d; T5 e;
};
Re: NEW: generic Toupel grouper [message #28371 is a reply to message #28370] Tue, 31 August 2010 09:20 Go to previous messageGo to next message
cbpporter is currently offline  cbpporter
Messages: 1428
Registered: September 2007
Ultimate Contributor
Just a little nitpicking: it is called tuple, unless I am not aware of some British/American alternate spelling.
Re: NEW: generic Toupel grouper [message #28372 is a reply to message #28371] Tue, 31 August 2010 09:44 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
typedef Tupel Tuple; Very Happy
no, you're absolutely right.
Re: NEW: generic Toupel grouper [message #28378 is a reply to message #28367] Tue, 31 August 2010 13:29 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
dolik.rce wrote on Mon, 30 August 2010 17:13

I agree with the a,b,c,d,e... too. It is nice and simple and fast to write. One more question is how many values should be supported. I personally think that a-f should be about enough...

Honza


IMO, 4 is more than enough....

Here is what I got after bit of experimenting:

template <typename A, typename B>
struct Tuple2 {
	union {
		A a;
		A key;
	};
	union {
		B b;
		B value;
	};
	
	int  Compare(const Tuple2& x) const       { return CombineCompare(a, x.a)(b, x.b); }
	bool operator==(const Tuple2& x) const    { return Compare(x) == 0; }
	bool operator!=(const Tuple2& x) const    { return Compare(x) != 0; }
	bool operator<=(const Tuple2& x) const    { return Compare(x) <= 0; }
	bool operator>=(const Tuple2& x) const    { return Compare(x) >= 0; }
	bool operator<(const Tuple2& x) const     { return Compare(x) != 0; }
	bool operator>(const Tuple2& x) const     { return Compare(x) != 0; }
	
	unsigned GetHashValue() const             { return CombineHash(a, b); }
};

template <typename A, typename B>
inline Tuple2<A, B> MakeTuple(const A& a, const B& b)
{
	Tuple2<A, B> r;
	r.a = a;
	r.b = b;
	return r;
}

template <typename A, typename B, typename C>
struct Tuple3 {
	union {
		A a;
		A key;
	};
	union {
		B b;
		B value;
	};
	union {
		C c;
		C value1;
	};
	
	int  Compare(const Tuple3& x) const       { return CombineCompare(a, x.a)(b, x.b)(c, x.c); }
	bool operator==(const Tuple3& x) const    { return Compare(x) == 0; }
	bool operator!=(const Tuple3& x) const    { return Compare(x) != 0; }
	bool operator<=(const Tuple3& x) const    { return Compare(x) <= 0; }
	bool operator>=(const Tuple3& x) const    { return Compare(x) >= 0; }
	bool operator<(const Tuple3& x) const     { return Compare(x) != 0; }
	bool operator>(const Tuple3& x) const     { return Compare(x) != 0; }
	
	unsigned GetHashValue() const             { return CombineHash(a, b, c); }
};

template <typename A, typename B, typename C>
inline Tuple3<A, B, C> MakeTuple(const A& a, const B& b, const C& c)
{
	Tuple3<A, B, C> r;
	r.a = a;
	r.b = b;
	r.c = c;
	return r;
}

template <typename A, typename B, typename C, typename D>
struct Tuple4 {
	union {
		A a;
		A key;
	};
	union {
		B b;
		B value;
	};
	union {
		C c;
		C value1;
	};
	union {
		D d;
		D value2;
	};
	
	int  Compare(const Tuple4& x) const       { return CombineCompare(a, x.a)(b, x.b)(c, x.c)(d, x.d); }
	bool operator==(const Tuple4& x) const    { return Compare(x) == 0; }
	bool operator!=(const Tuple4& x) const    { return Compare(x) != 0; }
	bool operator<=(const Tuple4& x) const    { return Compare(x) <= 0; }
	bool operator>=(const Tuple4& x) const    { return Compare(x) >= 0; }
	bool operator<(const Tuple4& x) const     { return Compare(x) != 0; }
	bool operator>(const Tuple4& x) const     { return Compare(x) != 0; }
	
	unsigned GetHashValue() const             { return CombineHash(a, b, c, d); }
};

template <typename A, typename B, typename C, typename D>
inline Tuple4<A, B, C, D> MakeTuple(const A& a, const B& b, const C& c, const D& d)
{
	Tuple4<A, B, C, D> r;
	r.a = a;
	r.b = b;
	r.c = c;
	r.d = d;
	return r;
}

Re: NEW: generic Toupel grouper [message #28379 is a reply to message #28378] Tue, 31 August 2010 13:34 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
Upgrade:

template <typename A, typename B>
struct Tuple2 {
	union {
		A a;
		A key;
	};
	union {
		B b;
		B value;
	};
	
	bool operator==(const Tuple2& x) const    { return a == x.a && b == x.b; }
	bool operator!=(const Tuple2& x) const    { return !operator==(x); }

	int  Compare(const Tuple2& x) const       { return CombineCompare(a, x.a)(b, x.b); }
	bool operator<=(const Tuple2& x) const    { return Compare(x) <= 0; }
	bool operator>=(const Tuple2& x) const    { return Compare(x) >= 0; }
	bool operator<(const Tuple2& x) const     { return Compare(x) != 0; }
	bool operator>(const Tuple2& x) const     { return Compare(x) != 0; }
	
	unsigned GetHashValue() const             { return CombineHash(a, b); }
};

template <typename A, typename B>
inline Tuple2<A, B> MakeTuple(const A& a, const B& b)
{
	Tuple2<A, B> r;
	r.a = a;
	r.b = b;
	return r;
}

template <typename A, typename B, typename C>
struct Tuple3 {
	union {
		A a;
		A key;
	};
	union {
		B b;
		B value;
	};
	union {
		C c;
		C value1;
	};
	
	bool operator==(const Tuple3& x) const    { return a == x.a && b == x.b && c == x.c; }
	bool operator!=(const Tuple3& x) const    { return !operator==(x); }

	int  Compare(const Tuple3& x) const       { return CombineCompare(a, x.a)(b, x.b)(c, x.c); }
	bool operator<=(const Tuple3& x) const    { return Compare(x) <= 0; }
	bool operator>=(const Tuple3& x) const    { return Compare(x) >= 0; }
	bool operator<(const Tuple3& x) const     { return Compare(x) != 0; }
	bool operator>(const Tuple3& x) const     { return Compare(x) != 0; }
	
	unsigned GetHashValue() const             { return CombineHash(a, b, c); }
};

template <typename A, typename B, typename C>
inline Tuple3<A, B, C> MakeTuple(const A& a, const B& b, const C& c)
{
	Tuple3<A, B, C> r;
	r.a = a;
	r.b = b;
	r.c = c;
	return r;
}

template <typename A, typename B, typename C, typename D>
struct Tuple4 {
	union {
		A a;
		A key;
	};
	union {
		B b;
		B value;
	};
	union {
		C c;
		C value1;
	};
	union {
		D d;
		D value2;
	};
	
	bool operator==(const Tuple4& x) const    { return a == x.a && b == x.b && c == x.c && d == x.d; }
	bool operator!=(const Tuple4& x) const    { return !operator==(x); }

	int  Compare(const Tuple4& x) const       { return CombineCompare(a, x.a)(b, x.b)(c, x.c)(d, x.d); }
	bool operator<=(const Tuple4& x) const    { return Compare(x) <= 0; }
	bool operator>=(const Tuple4& x) const    { return Compare(x) >= 0; }
	bool operator<(const Tuple4& x) const     { return Compare(x) != 0; }
	bool operator>(const Tuple4& x) const     { return Compare(x) != 0; }
	
	unsigned GetHashValue() const             { return CombineHash(a, b, c, d); }
};

template <typename A, typename B, typename C, typename D>
inline Tuple4<A, B, C, D> MakeTuple(const A& a, const B& b, const C& c, const D& d)
{
	Tuple4<A, B, C, D> r;
	r.a = a;
	r.b = b;
	r.c = c;
	r.d = d;
	return r;
}

Re: NEW: generic Toupel grouper [message #28380 is a reply to message #28370] Tue, 31 August 2010 13:36 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
kohait00 wrote on Tue, 31 August 2010 02:44

so here is a cleaned up proposal..
template<class T1, class T2, class T3=EmptyClass, class T4=EmptyClass, class T5=EmptyClass>
class Tupel {
public:
	Tupel(const T1 & a, const T2 & b, const T3 & c, const T4 & d, const T5 & e) 
		: a(a), b(b), c(c), d(d), e(e) {}
	Tupel() {}
	T1 a; T2 b; T3 c; T4 d; T5 e;
};



There are two troubles:

- sizeof(EmptyClass) == 1, and all these members make it impossible to be initialized with braced list.

- with constructor, again it cannot be initialized with braced list (which, for me, is very important)
Re: NEW: generic Toupel grouper [message #28382 is a reply to message #28380] Tue, 31 August 2010 13:51 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
nice work,

btw: doensnt ==, !=, etc imply a quite a lot on class capabilities

EmptyClass: what about a DummyClass
class DummyClass
{
public:
DummyClass() {}
DummyClass(const DummyClass&) {}
};

it even could derive from EmptyClass to compare maybe

[Updated on: Tue, 31 August 2010 13:55]

Report message to a moderator

Re: NEW: generic Toupel grouper [message #28386 is a reply to message #28382] Tue, 31 August 2010 14:04 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
void* is simple but makes things possible
template<class A, class B, class C=void*, class D=void*>
class Tupel {
public:
	Tupel(const A& a, const B& b, const C& c = NULL, const D& d = NULL) 
		: a(a), b(b), c(c), d(d) {}
	Tupel() {}
	A a; B b; C c; D d;
};


EDIT: union is problematic, if T=One<int> i.e.

template<class A, class B, class C=void*, class D=void*>
class Tupel {
public:
	Tupel(const A& a, const B& b, const C& c = NULL, const D& d = NULL) 
		: a(a), b(b), c(c), d(d) {}
	Tupel() {}
	union { A a, key, v1; };
	union { B b, value, v2; }; 
	union { C c, v3; };
	union { D d, v4; };
};


Quote:



error: member 'Upp::One<int> Tupel<int, Upp::One<int>, Upp::String, int>::<anonymous union>::b' with constructor not allowe
d in union


TDMGCC

[Updated on: Tue, 31 August 2010 14:09]

Report message to a moderator

Re: NEW: generic Toupel grouper [message #28389 is a reply to message #28386] Tue, 31 August 2010 14:18 Go to previous messageGo to next message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
i'd suggest to leave it simple..just as u said.
Re: NEW: generic Toupel grouper [message #28393 is a reply to message #28382] Tue, 31 August 2010 15:21 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
kohait00 wrote on Tue, 31 August 2010 07:51

nice work,

btw: doensnt ==, !=, etc imply a quite a lot on class capabilities



It does, but it is not a problem, because compiler will not instatiate methods unless requested by using them. (Unless I am terribly mistaken Smile.
Re: NEW: generic Toupel grouper [message #28394 is a reply to message #28386] Tue, 31 August 2010 15:23 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 14291
Registered: November 2005
Ultimate Member
kohait00 wrote on Tue, 31 August 2010 08:04


EDIT: union is problematic, if T=One<int> i.e.



Ops, good point. What a pity, I guess I will have to live without 'key' Smile

Mirek
Re: NEW: generic Toupel grouper [message #28395 is a reply to message #28393] Tue, 31 August 2010 15:31 Go to previous messageGo to previous message
kohait00 is currently offline  kohait00
Messages: 939
Registered: July 2009
Location: Germany
Experienced Contributor
MSC i.e. is not even complaining about mistyped template implementations, as long as they are not instantiated. GCC is better here, it checks implementation. but of corse code instantiation is another topic..
Previous Topic: Date limited to 2020 and 2015 does not work ?!?
Next Topic: Value question (memory consumption)
Goto Forum:
  


Current Time: Tue Sep 15 00:48:23 GMT+2 2026

Total time taken to generate the page: 0.02038 seconds