On Wed 5/7/2025 12:37 AM, David Brown wrote:
That would get an immediate downcheck during review for exactly
that reason.
Of course. In fact, if someone presented such code for review (and
assuming I noticed the commas!) I'd have to consider whether it was
done maliciously, intentionally deceptively, due to incompetence, or
smart- > arse coding. In all my C coding experience, I can't recall
ever coming across a single situation when I thought the use of the
comma operator was appropriate in the kind of code I work with.
Wow! That's catastrophically bad.
As it has been stated many times before, both C and C++ are
programming languages that embrace both statement-level and
expression-level programming. Expression-level programming
(e.g. where ?:` is used for branching and `,` for sequencing) is a
very valuable and massively important programming paradigm in these languages. The fact that elaborate expression-level programming is
not in nay way abandoned or shunned today is pretty obvious in C++,
since C++ took major steps lately to develop its expression-level capabilities. But it has always been and will always remain
important in C as well.
The proclivity to stick exclusively to statement-level programming
in C and, God forbid, impose it in others through so called "code
reviews"... that would be a trait specific to "sweatshop"
development outfits, which strive to replace quality with quantity.
I'd agree that in a revolving door employment environment relying on
a large number of low-competence developers such code might be seen
as "too confusing". But I don't see why we should set our standards
that low here, in `comp.lang.c`.
| Sysop: | DaiTengu |
|---|---|
| Location: | Appleton, WI |
| Users: | 1,090 |
| Nodes: | 10 (0 / 10) |
| Uptime: | 45:21:29 |
| Calls: | 13,946 |
| Calls today: | 3 |
| Files: | 187,034 |
| D/L today: |
8,062 files (2,942M bytes) |
| Messages: | 2,460,945 |