Avoid name collisions with enum in C (C99)

Question

enum elements' names are susceptible to overlap/collide with both other enum elements names, variable names, etc...

enum Fruit
{
    apple,
    orange
};
typedef enum Fruit Fruit;


enum Color
{
    red,
    orange // <-- ERROR
};
typedef enum Color Color;


char apple='a'; // <-- ERROR

Is there a C99 compliant solution to avoid collision other than prefixing every enum element name?

Side note: this question has already an answer for C++

How to avoid name conflicts for two enum values with the same name in C++?

I'm looking for a C99 solution.

Yes, use an appropriate name like `color_orange` and `fruit_orange`. It should also avoid the confusion when reading. — Iharob Al Asimi, Feb 13 '16 at 13:09
Is there a [...] solution **other than prefixing every enum element name** ? — Paolo, Feb 13 '16 at 13:11
Maybe but it would likely be a bad solution, this is a good solution that helps readability too. — Iharob Al Asimi, Feb 13 '16 at 13:11
is there anything better readable than `Fruit f = orange;` and `Color c = orange;` ? It's tidy and no doubt than `f` is a fruit and `c` a color — Paolo, Feb 13 '16 at 13:16
Yes `Fruit fruit = FruitOrange;`. Oh and in this case `switch (fruit) {case FruitOrange: break ...}`. If there are many fruits you will be glad to prefix them. Also `f` can be for `file`, `f**ck`, and many other wrods that start with `f` `fruit` instead is simply `fruit`. — Iharob Al Asimi, Feb 13 '16 at 13:16
that's ugly! :-) but of course if it's the only solution I'll go with that. "**There is no solution for that**" (if true) may be an appropriate answer. — Paolo, Feb 13 '16 at 13:18
It's not ugly `Fuirt f = orange;` is ugly, why not make it `Fruit f = o;` too and shorten it more to make it prettier? — Iharob Al Asimi, Feb 13 '16 at 13:19
I don't understand the question entirely. Do you want to refer to different entities with the same name? — jxh, Feb 13 '16 at 17:56
@IharobAlAsimi, `f` is a letter. Why do you assume `f` must always stand for a word? Sometimes verbosity is pointless within context, more difficult to read, and leads to typos as you already mixed up "Fuirt" with "Fruit." When you start down the path of "do this thing because I was told it is good," you often end up in a mire, as is evidenced by the appalling state of software "development" we see these days; where all the code runs slower, is harder to read, harder to maintain, and with more bugs every year. A million 4-line, verbose functions in a half-million 2-function files. Yuck! — SO_fix_the_vote_sorting_bug, Dec 28 '22 at 00:40

rici · Accepted Answer · 2016-02-13T14:25:57.990

In C, there is no solution other than prefixing the names of the enum values.

As pointed out in the OP, C++ has a number of mechanisms, of which enum class is probably indicated for modern code. However, in practice the result is the same: you end up prefixing the name of the enum element with the name of the enum. Arguably, Fruit::orange is tidier than FruitOrange, but really it makes little difference to my eyes.

In some parallel universe, it would be great to have a language in which you could write:

Fruit selected = orange;

and have the compiler deduce the namespace of the constant on the right-hand side. But I don't see how that language could be C. C doesn't have namespaces in that sense, and even if it did, the type system only allows conversions; you cannot condition the syntax of the RHS of an operator based on the LHS (and I use the word syntax deliberately, because name lookup is a syntactic property in C).

Even if you did have some language hack which sometimes implictly inserted an enum namespace, you would still need the explicit prefix on any comparison, because

if (apple > orange)

does not have a context in which deduction could take place, even though the fact that enum values in C are all of type int does make FruitApple and FruitOrange comparable.

Would just like to add a link to an answer that complements this one: https://stackoverflow.com/a/48231009/6197439 — sdbbs, Dec 21 '22 at 01:44
Some claim that `Namespace::variableName` or `Namespace.variable_name` is "prettier" or "better," without specifying a metric with which to make these subjective judgements. A simple `Namespace_variable` does the same thing, with less typing. `NamespaceVariable` also. It's mainly personal preference unless extra type info can be utilized with an `enum class` or equivalent, if the language supports it. — SO_fix_the_vote_sorting_bug, Dec 28 '22 at 00:44
In C++20 at least you are not forced to always use the enum-name for scoped enums - `using`allows you to bring the enum directly into scope. Example: `using enum Fruit; Fruit f = orange;` — ABaumstumpf, Jan 05 '23 at 10:21

Avoid name collisions with enum in C (C99)

1 Answers1