ViewBag analyzer

I would like to suggest a new analyzer. An analyzer to point out the usage of ViewBag and replace it with an model

Before:

MyController.cs

ViewBag.Message = "Hello World";

After:

MyModel.cs

public string Message { get; set; }

MyController.cs

model.Message = "Hello World;"

Why I think this is useful? ViewBag is a dynamic property and you will only catch errors at runtime making your code error prone.

[JohanLarsson]

If this is for AspNetCore there is the official analyzer

There is also AspNetCoreAnalyzers.

[sharwell]

@kidchenko It probably wouldn’t make sense for StyleCop Analyzers, but would make sense for one of the other projects linked above. If you are interested in contributing, I would suggest opening an issue on those to check if the project would use it. 👍