Rule proposal: Maintainability rule - Remove unnecessary parenthesis for object initializers.

Product p = new Product() { Id= 10, Name = "Cup" }; 

should be

Product p = new Product { Id= 10, Name = "Cup" };

Originally posted by ericslattery http://stylecop.codeplex.com/workitem/6793

follow up discussion:

andyr wrote Feb 4, 2011 at 9:47 PM Jason, please review and comment. Then I’ll fix/add.

andyr wrote Mar 15, 2011 at 5:32 PM From Jason, “Love it”

Tratcher wrote Jul 19, 2012 at 7:04 PM Dislike. If we must choose one or the other I’d rather require the parens than prohibit them.

Keeping the parens is more consistent with normal constructor requirements. E.g. If I already had new Products(); adding {…} should not force me to remove the parens.

Also, the parens will always be required for any constructors with parameters, so it is inconsistent to prohibit them on the parameter-less constructor.

[ZizhengTai]

I know this was from five years ago. Has anything been implemented for this? I searched through the documentation but didn’t find anything.

[Przemyslaw-W]

Syntactically, it is argument list, yes. Semantically, applied attribute is kinda declarative - it is not “live” code, which is invoked in runtime.