[Lemon-devel] Renaming basic classes

Alpár Jüttner alpar at cs.elte.hu
Sun Sep 16 19:39:05 CEST 2007


Hi,

> a() and b() are not words, only letters. I think such names are usually used
> for temporary varibles. The others are whole words, so using them makes easier
> to avoid conflicts. 

They are in different namespaces, so it won't cause conflicts.

> On the other hand if 'arc' is used instead of 'edge', the typical iterator
> varible for arcs will be 'a', not 'e'. So IMHO red/blue, first/second, etc.
> would be better than a/b.

I may be, but red/blue cannot be used for denoting the ends of an
undirected edge,
while first/second a bit strange for denoting the sides of a bipartite
graph. Or not? I don't know. Think of e.g. Graph::FirstIt,
Graph::FirstMap<>, Graph::SecondMap<> as well.

Now, I see two contradicting opinion here. In my opinion is not really
strong, and is seems that so do Peter's one, as well.

Let's see the other people's opinion.

Alpar






More information about the Lemon-devel mailing list