21 Nov
2011
21 Nov
'11
8:18 p.m.
a++; // add one to a
Totally pointless.
[...] Or, of course, if you've been intentionally obtuse
about your
variable names
...then, unless you're writing for the IOCCC or some such, you've got
bad code there, in at least that respect.
Of course, one person's "intentionally obtuse" is another's "why
bother, it's a temporary for all of ten lines of code" or even "what do
you mean obtuse, it's perfectly clear".
(chances are you should never REALLY have a variable
named "a" unless
there's a very good reason to do so),
NODE *sort_nodes(NODE *list)
{
NODE *a;
NODE *b;
NODE *t;
NODE **tp;
if (!list || !list->link) return(list);
a = 0;
b = 0;
while (list)
{ t = list;
list = list->link;
t->link = a;
a = b;
b = t;
}
sort_nodes(a);
sort_nodes(b):
...(merge step omitted)...
}
I am unconvinced this is either a bad use of the names "a" and "b" or
a
case where there's "a very good reason" to use them (x and y, or s1 and
s2, or any of many other choices, would be equally good).
/~\ The ASCII Mouse
\ / Ribbon Campaign
X Against HTML mouse at rodents-montreal.org
/ \ Email! 7D C8 61 52 5D E7 2D 39 4E F1 31 3E E8 B3 27 4B