sourceanalysis Forum Rss Feed

NEW POST: VB Version?

danieldsmith — Mon, 23 Nov 2009 19:06:00 GMT

Seems to me like the VB version would only need one rule:

if (language == "VB.NET")
{
DisplayWarning("Use a real language like C#");
}

I am (partially!) joking of course. I'm sure there will be support in a future version. That said however, VB usage in the industry is on a steady decline, so how much of a priority it should be getting is another question. For example in a recent Telerik poll asking which langauge devs are primarily developing in, VB usage trailed in at just 30% compared to C# on nearly 70%. Here's some facts to back this up:

http://telerikwatch.com/2009/06/survey-says-c-still-more-popular-than.html
http://www.google.com/trends?q=VB.NET%2C+C%23&ctab=0&geo=all&date=ytd&sort=0
http://radar.oreilly.com/2009/02/state-of-the-computer-book-mar-22.html

NEW POST: Hide StyleCop Warnings when running msbuild

jasonall — Thu, 19 Nov 2009 18:19:18 GMT

Try setting StyleCopEnabled=false at the top of your .bat file.

NEW POST: VB Version?

LeeB — Thu, 19 Nov 2009 16:56:20 GMT

(Sigh.) Much as we'd all like to believe the mighty Microsoft has unlimited resources, it just isn't true. Hey, I'd like to see a StyleCop for every language available on .NET, but I don't think we'll be seeing that anytime soon. Really folks, we have four options, not necessarily mutually exclusive:

Be patient while the folks at MS break down the technical barrier
Write the tool ourselves, perhaps adapting something that already exists
Improve our coding habits--is it really that hard to follow the conventions used by MS's code examples?
Learn a different language--no single language is best at everything; .NET gives you plenty to pick from

Regarding the last point, the days of being a monolingual, single-platform developer are over. Expand your horizons, and bring back what you've learned to your preferred language and platform.

NEW POST: VS2010 Beta2?

TATWORTH — Thu, 19 Nov 2009 15:40:01 GMT

Any update on when VS2010 support will be added?

NEW POST: Hide StyleCop Warnings when running msbuild

eusebiu — Thu, 19 Nov 2009 08:57:34 GMT

Hello.

I have a .bat file that is compiling some targets projects using msbuild command and I use also style cop to verify my code rules.
I want to hide the warnings that stylecop produces. I tried /consoleloggerparameters:ErrorsOnly;Verbosity=q with no luck. The only thing that did it is specifying /noconsolelogger but this one hides everything.

Is this possible?

Thanks,
Eusebiu

NEW POST: Cannot Suppress SA1002 and SA1009

Tallek — Wed, 18 Nov 2009 17:21:28 GMT

I was having the same problem with SA1118 but managed to fix it. I opened up the rules assembly in reflector and looked at the rules enumeration. Turns out the rule name in the help file and the enumeration representing it don't always match 100%. In this case, the enumeration was ParameterMustNotSpanMultipleLines versus the rule name of ParametersMustNotSpanMultipleLines. Notice the extra "s" on "Parameter" in the rule name. When i left out the "s" and just used the enumeration name it worked. Looking a bit more, i can see the xml files for all the rules in the Resources folder in reflector (looking at Microsoft.StyleCop.CSharp.Rules.dll). In the ReadabilityRules xml file, i see the same issue with it using the name without the "s" on "Parameter".

NEW POST: SA1009 with type-casting parenthesis

shuruev — Tue, 17 Nov 2009 10:02:03 GMT

jasonall wrote:
The recommendation is to have no space after the cast. I'm not sure why you're getting the violatiohn on that line. Maybe if you include more of the code we can figure it out.

Jason,
Thanks for the answer.

Actually, I have two questions:
1. I am using reflection and getting SA1009 when using "typeof" and casting in one line. It seems to be a little bug. I don't get SA1009 when I don't use "typeof" or when I put space after cast and before "typeof".
2. What do you think about creating a rule that would prohibit putting a space after cast? I consider this as a "bad style", but it's up to you to decide, of course.

Here is the code example that illustrates them.

using System;
using System.Reflection;

namespace Shuruev.StyleCop.Test
{
/// <summary>
/// Some code that illustrates wrong SA1009 firing.
/// </summary>
public static class Class1
{
public static readonly Uri TestUri = new Uri("http://www.microsoft.com");

/// <summary>
/// I assume there is some problem with "typeof" checking
/// because the first part of code produces SA1009 (seems it should not).
/// The second part (with space after cast) doesn't produce SA1009,
/// but I I really don't like this way of typing.
/// </summary>
public static void Test1()
{
string innerA = (string)typeof(Uri).InvokeMember(
"m_String",
BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.GetField,
null,
TestUri,
null);

string innerB = (string) typeof(Uri).InvokeMember(
"m_String",
BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.GetField,
null,
TestUri,
null);
}

/// <summary>
/// Here both parts work fine and don't produce SA1009.
/// But I think it would be nice to create a new rule that would prohibit putting a space after cast.
/// </summary>
public static void Test2()
{
string innerA = (string)TestUri.GetType().InvokeMember(
"m_String",
BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.GetField,
null,
TestUri,
null);

string innerB = (string) TestUri.GetType().InvokeMember(
"m_String",
BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.GetField,
null,
TestUri,
null);
}

/// <summary>
/// Here both parts work fine and don't produce SA1009.
/// But I think it would be nice to create a new rule that would prohibit putting a space after cast.
/// </summary>
public static void Test3()
{
Type type = typeof(Uri);

string innerA = (string)type.InvokeMember(
"m_String",
BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.GetField,
null,
TestUri,
null);

string innerB = (string) type.InvokeMember(
"m_String",
BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.GetField,
null,
TestUri,
null);
}
}
}

Best regards,
Oleg Shuruev

NEW POST: Visual Web Developer

krappies — Tue, 17 Nov 2009 09:42:11 GMT

Thanks

NEW POST: SA1009 with type-casting parenthesis

jasonall — Tue, 17 Nov 2009 08:40:31 GMT

The recommendation is to have no space after the cast. I'm not sure why you're getting the violatiohn on that line. Maybe if you include more of the code we can figure it out.

Jason

NEW POST: Visual Web Developer

jasonall — Mon, 16 Nov 2009 18:21:16 GMT

VWD does not allow loading external VS packages, which prevents StyleCop from integrating into the shell. You can still use StyleCop by setting up MSBuild integration, as described in this blog post: http://blogs.msdn.com/sourceanalysis/pages/source-analysis-msbuild-integration.aspx

NEW POST: Visual Web Developer

krappies — Mon, 16 Nov 2009 16:24:43 GMT

Hi

Is there a way to use StyleCop with VWD 2008? I cannot find a way to do this.

Thanks,
Pieter

NEW POST: File Header Format

olivier34 — Thu, 12 Nov 2009 12:48:27 GMT

olivier34 wrote:
It seems also that special chars like the (c) from copyright do not work when set in the match expression...

A new issue has been posted: http://code.msdn.microsoft.com/sourceanalysis/WorkItem/View.aspx?WorkItemId=260

NEW POST: how to disable some rules

lory — Wed, 11 Nov 2009 12:58:33 GMT

I like most of rules, but in real practice, I would like to disable some of them. But I cannot find how to disable some rules. Can anyone give some help?

NEW POST: Visual Studio 2010 & .NET 4.0

MarcoErni — Tue, 10 Nov 2009 10:06:24 GMT

I have solved problem. The issue is that the default C# MSBuild targets file generates and adds this file intermediatly to the project structure, so StyleCop will check this file. To avoid that StyleCop is checking this file simply import the StyleCop targets before the import of the C# default targets.

Importing projects like this:
<Import Project="$(ProgramFiles)\MSBuild\Microsoft\StyleCop\v4.3\Microsoft.StyleCop.targets"/>
<Import Project="$(MSBuildBinPath)\Microsoft.CSharp.targets"/>

rather then explained on http://code.msdn.microsoft.com/sourceanalysis/Thread/View.aspx?ThreadId=2283
<Import Project="$(MSBuildBinPath)\Microsoft.CSharp.targets"/>
<Import Project="$(ProgramFiles)\MSBuild\Microsoft\StyleCop\v4.3\Microsoft.StyleCop.targets"/>

Hope this helps also others ...
Marco Erni

NEW POST: Visual Studio 2010 & .NET 4.0

MarcoErni — Fri, 06 Nov 2009 12:17:45 GMT

Hi

I have installed and integrated StyleCop into Visual Studio 2010, respectively into the C# projects as described in the following thread: http://code.msdn.microsoft.com/sourceanalysis/Thread/View.aspx?ThreadId=2283
It works just fine for Projects targeting .NET Framework 3.5 (or 3.0 & 2.0). But if the Project targets the .NET Framework 4.0, StyleCop fires allways the following two warnings:

- SA1003: The spacing around the symbol '=' is invalid.
- SA1633: The file has no header, the header Xml is invalid, or the header is not located at the top of the file.

Both warnings are issued by the the file "..\Local Settings\Temp\.NETFramework,Version=v4.0.AssemblyAttributes.cs"
It seems that Visual Studio 2010 produces that file an intermediate link this file into the C# project while building the project inside Visual Stduio.

Has anbody the same problem or better a solution for that issue?

Best Regards,
Marco Erni

NEW POST: VB Version?

sedge55 — Thu, 05 Nov 2009 23:04:06 GMT

I agree. Where's the VB version? Apart from the similarity with Java I completely fail to see the fascination with C#. VB code is so much easier to read. I'd also put in an ad for a documentation tool like NDoc for VB comments.

NEW POST: Installing StyleCop in w Team enviroment

Task — Thu, 05 Nov 2009 14:10:11 GMT

Uh, what?

I'm afraid I don't understand your question.

If you're hand-editing a .csproj file, something has gone terribly wrong.

NEW POST: SuppressMessage not working for certain rules??

olivier34 — Thu, 05 Nov 2009 13:00:38 GMT

Hi!

I do think that as StyleCop SuppressMessage must be set on an element, some rules, that applies to the whole file, can't be excluded this way.

Regards,

Olivier.

NEW POST: Installing StyleCop in w Team enviroment

Tokyoant — Wed, 04 Nov 2009 22:55:03 GMT

Thanks very much Task.

Do you need to add any import statements or the like into project files?

Thank you!

NEW POST: SuppressMessage not working for certain rules??

Task — Wed, 04 Nov 2009 20:14:28 GMT

jestersoftware wrote:
... any ideas? Am I placing the attribute wrong?

Could be.

He-he, show us yer code!