dotnet/roslyn

Renaming an identifier should undo implicit property naming

Aperta

#54.571 aperta il 2 lug 2021

 (1 commento) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-IDEConcept-Continuous ImprovementFeature - Renamehelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

Version Used: Visual Studio Enterprise 2019 Version 16.9.4 C# Tools 3.9.0-6.21160.10+59eedc33d35754759994155ea2f4e1012a9951e3 Steps to Reproduce:

  1. Instantiate an anonymous type with an implicit property name (new { K })
  2. Rename the identifier referenced in the instantiation

Expected Behavior: The instantiation would use the explicit property name instead of implicit property naming, in order to preserve the definition of the anonymous type

Actual Behavior: Implicit property naming is still used, changing the definition of the anonymous type

Example:

int K = 0;
var x = new { K };
int y = x.K;

After renaming K -> Q

int Q = 0;
var x = new { Q };
int y = x.K;               // This is now a compile error

Expected behavior after renaming K -> Q

int Q = 0;
var x = new { K = Q };
int y = x.K;               // This works fine

In this example, the compiler catches the error. But this could result in silent bugs in use cases such as such as MVC where an anonymously typed value can be translated to a JSON response object.

Guida contributor