IntelliTect / IntelliTect/EssentialCSharp
Improve GetAllValidNETPreprocessorSymbols
- Lenguaje dominante
- C#
- Estrellas
- 418
- Forks
- 173
- Merge medio
- 3 min
- PR fusionados (30 d)
- 3
Descripción
This works, but I think it is cleaner, to just write out the individual `yield return` lines.
_Originally posted by @Keboo in https://github.com/IntelliTect/EssentialCSharp/pull/584#discussion_r1383683019_
Maybe something like:
```C#
switch
(
(majorVersion, minorVersion)
)
{
case (5, 0):
yield return "NET5_0_OR_GREATER";
yield return "NET5_0";
goto case (3, 1);
case (3, 1):
yield return "NETCOREAPP3_1_OR_GREATER";
yield return "NETCOREAPP3_1";
goto case (3, 0);
case (3, 0):
yield return "NETCOREAPP3_0_OR_GREATER";
goto case (2, 1);
case (2, 1):
yield return "NETCOREAPP2_1_OR_GREATER";
goto case (2, 0);
case (2, 0):
yield return "NETCOREAPP2_0_OR_GREATER";
goto case (1, 1);
case (1, 1):
yield return "NETCOREAPP1_1_OR_GREATER";
goto case (1, 0);
case (1, 0):
yield return "NETCOREAPP1_0_OR_GREATER";
break;
default:
break;
```
This needs more cleanup and work, but general structure is there
Guía de contribución
Línea de trabajo
Localiza el punto de entrada GetAllValidNETPreprocessorSymbols y compara su implementación actual con la estructura switch sugerida en el issue. Limpia la implementación conservando los símbolos esperados para cada versión compatible y, después, verifica que el proyecto siga compilándose y que sus comprobaciones existentes pasen.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp
- Área
- build-system
- Tipo de issue
- Refactorización
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100