Switch with string constant as a label: Exception: [source error] bad case label

[SergiusSidorov]

Hi,

I'm using javassist 3.12.1.GA with java 8.
When I try to generate method with the switch statement, which uses string constant as a label, javassist presents the following exception: "Exception: [source error] bad case label"
However, generated source is correct and works (I checked it in a real java class).

For example, generated method body:

{
switch($1.getClass().getName()) {
case "example.EventA":
handler0.onEvent((example.EventA)$1);
break;
case "example.EventB":
handler1.onEvent((example.EventB)$1);
break;
}
}

Does javassist support switch statement with string label?

[chibash]

The specification is that Javassist's compiler only supports Java 1.4 (yes, old!) syntax.
But, I'll check the code and, if it's easy to implement, I'll support string constants as a label.

[SergiusSidorov]

Thanks!

[QIvan]

@chibash
Hi! I need this feature too. Did you check code? Is it hard to implement?

[smilanko]

any progress on this? While I do not mind using the if cases, a switch would really be nice ;)

[lygoing]

@chibash
Hi!
I also encountered the same situation! May I ask how to solve this problem now? thank you very much

[smilanko]

@lygoing I doubt that a solution has been made, in the version we are referencing. I ended up just using if statements to solve the issue. More code, but oh well, it works. Does doing the same not solve the problem for you?

[lygoing]

@smilanko
Hi!
The case is more efficient than if, and I am now solving this with the string hashcode.
like this:
switch (fieldName.hashCode()) {
case -861253476:
return context.getField13();
case -861253471:
return context.getField18();
}

[andreysubbotin]

Hi,
Do you have any plans to fix the problem?

[chibash]

I'm working... Check the test/switch-case branch.

[chibash]

I've done. See Test/switch case #231.