redhat-developer / redhat-developer/vscode-java
Type parameter related false positive error messages.
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 2.3k
- Forks
- 546
- Merge moyen
- 20 h 1 min
- PR mergées (30 j)
- 11
Description
I encountered some error messages on my code that can be correctly compiled and run, meaning that these errors could be false positives. After simplifying the code, I found that these false positives could be caused by incorrect name shadowing for type parameters.
Environment
- Operating System: Linux
- JDK version: 21
- Visual Studio Code version: 1.94.2
- Java extension version: 1.35.1
Steps To Reproduce
- Create an empty java project and open it with vscode.
- Create a package
demoand aApp.javafile in this package. - Paste the code below into
App.java:
package demo;
interface BinaryExpr {
interface Op { }
Op getOperator();
}
abstract class AbstractBinaryExpr<Op extends BinaryExpr.Op> implements BinaryExpr {
private final Op op;
public AbstractBinaryExpr(Op op) { this.op = op; }
@Override
public Op getOperator() {
return op;
}
}
final class ArithExpr extends AbstractBinaryExpr<ArithExpr.Op> {
enum Op implements BinaryExpr.Op {
ADD, SUB
}
public ArithExpr(Op op) { super(op); }
}
public class App {
public static void main(String[] args) {
var e = new ArithExpr(ArithExpr.Op.ADD);
System.out.println(toStr(e));
}
public static String toStr(ArithExpr e) {
return switch(e.getOperator()) {
case ADD -> "+";
case SUB -> "-";
};
}
}
Current Result
You will see the following three error messages:
// ...
public static String toStr(ArithExpr e) {
return switch(e.getOperator()) { // Error 1: A switch expression should have a default case
case ADD -> "+"; // Error 2: ADD cannot be resolved to a variable
case SUB -> "-"; // Error 3: SUB cannot be resolved to a variable
};
}
Expected Result
No error messages are expected because java compiler can handle it correctly.
Additional Informations
If you rename the type parameter Op and its following occurrances in class AbstractBinaryExpr into O, the error messages will disappear. This hints that the return type Op of AbstractBinaryExpr.getOperator() may be resolved to the Op in the superclass BinaryExpr rather than the type parameter.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Reproduisez le rapport dans le projet Java vide en utilisant le code fourni de demo/App.java, puis comparez les diagnostics de VS Code avec le comportement du compilateur Java. Suivez la résolution des noms des paramètres de type impliquée dans AbstractBinaryExpr et vérifiez que l’expression switch ainsi que ses cas ADD et SUB ne produisent plus d’erreurs faussement positives.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java, vscode
- Domaine
- compilers, devtools
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100