Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

Options for migration of string comparisons (helper method?)

Aberta
  • Algo errado?
#1,034 6 comentários 1 reação 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
5/5
Tempo estimado
Mais de uma semana
Facilidade para iniciantes
25/100
Tipo de issue
Funcionalidade
Clareza
Precisa de esclarecimento
Status de atividade
Estagnada
Stack de tecnologia
csharp, visualbasic
Domínio
tooling

Direção de pesquisa

A issue não nomeia nenhum arquivo, teste ou ponto de entrada específico. Comece rastreando como Option Compare Text é convertido em CultureInfo.CurrentCulture.CompareInfo.Compare no código-fonte de CodeConverter e, em seguida, determine se o resultado da comparação pode ser configurado ou representado por meio de um método auxiliar; o trabalho estará concluído quando houver uma abordagem acordada e um escopo de implementação correspondente.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

enhancement

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 ?

Linguagem predominante
C#
Estrelas
913
Forks
244
Nenhum PR com merge em 30d

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de icsharpcode/CodeConverter

Todas as issues de icsharpcode/CodeConverter

Issues semelhantes

Mais issues de C#

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.