Options for migration of string comparisons (helper method?)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
El issue no nombra ningún archivo, prueba ni punto de entrada específicos. Empieza rastreando cómo Option Compare Text se convierte en CultureInfo.CurrentCulture.CompareInfo.Compare en el código fuente de CodeConverter y, a continuación, determina si el resultado de la comparación puede configurarse o representarse mediante un método auxiliar; el trabajo estará terminado cuando haya un enfoque acordado y un alcance de implementación asociado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Perhaps this should not be an issue or a feature request but I think it's worthy of discussion, and we can always delete later if needed.
I'm about to Migrate perhaps 50k to 100k lines of code from VB to C#. Almost all of it has Option Compare Text.
Now when we migrate, code like this
If kv.Key = sVariableName Then
It gets converted to this:
if (CultureInfo.CurrentCulture.CompareInfo.Compare(kv.Key ?? "", sVariableName ?? "", CompareOptions.IgnoreCase | CompareOptions.IgnoreKanaType | CompareOptions.IgnoreWidth) == 0)
I'm sure I'm not the only one who doesn't want to live with that kind of code going forward, especially in a large project.
Ideally I would like improve the syntactic elegance using something like an Extension method
[Extension]
public static bool EqualsIgnoreCase(this string str1, string str2)
{
return string.Equals(str1 ?? "", str2 ?? "", StringComparison.OrdinalIgnoreCase);
}
Then the migrated c# would be
if (kv.Key.EqualsIgnoreCase(sVariableName)) {
I would have liked to overload the string == operator but that doesn't sound possible.
So, before I start digging in the CodeConverter source code for how to implement this, are there any other options or ways to influence the conversion as above, and does anyone else feel that there would be value in changing the code if appropriate for something like the above ?
- 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 36 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 71 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
-
VideoViewer: rotated (portrait phone) videos shown sideways when system decimal separator is a commaAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Volodymyr-Petrunin/Bankomaten#45 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
AvaloniaUI/Avalonia#22420 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
dotnet/SqlClient#4823 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
type/automation type/tech-debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día