VB -> C#: Avoid creating code blocks for each case when converting Select Case
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 65/100
Línea de trabajo
Comienza localizando la lógica de conversión de Select Case y el código que emite los bloques de las ramas case. Usa los fragmentos de VB proporcionados y los fragmentos de C# esperados como caso de regresión; el trabajo estará terminado cuando las ramas que no crean variables ya no reciban llaves ni indentación adicional, mientras que las ramas que necesitan variables conserven sus bloques.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When converting a Select Case statement, each outcome is wrapped in braces, creating a code block. This adds unnecessary indentation.
For example:
Private Shared Function d1mach(i As Integer) As Double
Select Case i
Case 1
Return 2.2250738585072014E-308 ' the smallest positive magnitude.
Case 2
Return Double.MaxValu ' the largest magnitude.
End Select
Return 0
End Function
Is converted as:
private static double d1mach(int i)
{
switch (i)
{
case 1:
{
return 2.2250738585072014E-308d; // the smallest positive magnitude.
}
case 2:
{
return double.MaxValue; // the largest magnitude.
}
}
return 0d;
}
Proposed solution
When no variable is created in the case branch, avoid creating a code block. The output code would be:
private static double d1mach(int i)
{
switch (i)
{
case 1:
return 2.2250738585072014E-308d; // the smallest positive magnitude.
case 2:
return double.MaxValue; // the largest magnitude.
}
return 0d;
}
- Lenguaje dominante
- C#
- Estrellas
- 914
- Forks
- 242
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de icsharpcode/CodeConverter
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
icsharpcode/CodeConverter#1271 · 1 comentario ·
-
C# -> VB: most pattern matching fails to convertPosiblemente ocupada @gherards99 la tomó hace 33 días. AbiertoC# -> VB
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
icsharpcode/CodeConverter#1281 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
icsharpcode/CodeConverter#1275 ·
-
VB -> C#: Handleres of sub properties being untrackedPosiblemente ocupada @Igor-Tomasi la tomó hace 69 días. AbiertoVB -> C#
Dificultad 3/5 1-2 días Aptitud para principiantes 62/100
icsharpcode/CodeConverter#1273 ·
-
VB -> C#: "Group By Into" Linq queries supressing aggregations ("Into" part) in the converted codeAbiertoDifficult area VB -> C#
Dificultad 3/5 1-2 días Aptitud para principiantes 74/100
icsharpcode/CodeConverter#1272 ·
Todos los issues de icsharpcode/CodeConverter
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
microsoft/fluentui-blazor#5410 ·
Los mantenedores suelen responder en 1 día
-
Bug pulumi/pulumi
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
activescott/lessmsi#306 ·
-
Docs MSBuild
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
getsentry/sentry-dotnet#5691 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día