Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

VB -> C#: If interface and class member differ only by capitalization, the class member's case is changed.

Aperta
  • Qualcosa non va?
#963 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
csharp
Ambito
devtools

Direzione di ricerca

Riproduci l’input VB segnalato e confronta il C# generato con l’implementazione esplicita dell’interfaccia prevista. Traccia la gestione da parte del convertitore dei membri Implements i cui nomi differiscono solo per maiuscole e minuscole, quindi aggiungi una copertura di regressione che dimostri che vengono preservati la combinazione originale di maiuscole e minuscole del membro e il mapping dell’interfaccia.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

VB -> C#
VB.Net input code
Public Interface I
    Public Sub A()
End Interface

Public Class C
    Implements I

    Public Sub a() Implements I.A
    End Sub
End Class
Erroneous output
public partial interface I
{
    public void A();
}

public partial class C : I
{

    public void A()
    {
    }
}
Expected output
public partial interface I
{
    public void A();
}

public partial class C : I
{

    public void a()
    {
    }

    void I.A() => a();
}

OR

public partial interface I
{
    public void A();
}

public partial class C : I
{
    /* TODO WARNING: Case changed from a to A (to match supertype) */
    public void A()
    {
    }
}
Details
  • Product in use: icsharpcode.github.io/CodeConverter
  • Version in use: 9.0.4.0
  • Why is this a problem?
    • VB.NET is case-insensitive, but CIL isn't. Thus, if you convert a library from VB to C# and the capitalization of a public member changes, you break binary-compatibility, i.e., an application compiled to the VB version of your library won't work with the C# version.
    • You also break source-compatibility, if your VB library is used by C# applications.
  • The converter already produces the expected output (method + second method with explicit interface implementation) if you replace Public Sub a() Implements I.A by Public Sub b() Implements I.A. So the fix might be as easy as changing a case-insensitive comparison to a case-sensitive one.
  • I admit that a/A is a contrived example, but a real-life use case would be an abbreviation that is capitalized differently. In our code base, we stumbled upon this issue with Sub AddSubSQL (interface) vs Sub AddSubSql (class).
  • Note that this change will trigger a warning: CS3005 Identifier 'C.a' differing only in case is not CLS-compliant, so it might make sense to additionally add the a [CLSCompliant(false)] attribute to the method. Unfortunately, I don't know enough about CLS compliance to be able to evaluate the impact of this change.
Lingua principale
C#
Stelle
913
Fork
243
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di icsharpcode/CodeConverter

Tutte le issue di icsharpcode/CodeConverter

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.