17

StyleCop 提供检查空格使用的一致性,但遗憾的是缺少相反的想法:强制源代码使用制表符。有没有办法添加这个功能?它不一定是 StyleCop,也欢迎使用其他工具。

4

7 回答 7

12

我也是一个制表符而不是空格的人,尽管有很多理由使用其中一个,并且还有其他地方可以解释为什么你认为一个比另一个更好。:)

我实际上想要同样的东西——检查制表符缩进的规则——所以我根据 StyleCop 的 SpacingRules 源编写了它。它似乎工作得相当好,尽管到目前为止我只在几个项目中使用过它。它可能会被优化或其他什么......但它可以工作。

using System;
using System.Text.RegularExpressions;
using Microsoft.StyleCop;
using Microsoft.StyleCop.CSharp;

namespace CustomRules.StyleCop.CSharp
{
  [SourceAnalyzer(typeof(CsParser))]
  public class SpacingRules : SourceAnalyzer
  {
    public SpacingRules()
    {
    }

    public override void AnalyzeDocument(CodeDocument document)
    {
      Param.RequireNotNull(document, "document");

      CsDocument csdocument = (CsDocument)document;
      if (csdocument.RootElement != null && !csdocument.RootElement.Generated)
      {
        this.CheckSpacing(csdocument.Tokens);
      }
    }

    private void CheckSpacing(MasterList<CsToken> tokens)
    {
      Param.AssertNotNull(tokens, "tokens");

      foreach (var token in tokens)
      {
        if (this.Cancel)
        {
          break;
        }

        if (token.Generated)
        {
          continue;
        }

        switch (token.CsTokenType)
        {
          case CsTokenType.WhiteSpace:
            this.CheckWhitespace(token as Whitespace);
            break;

          case CsTokenType.XmlHeader:
            XmlHeader header = (XmlHeader)token;
            foreach (var xmlChild in header.ChildTokens)
            {
              this.CheckTabsInComment(xmlChild);
            }
            break;

          case CsTokenType.SingleLineComment:
          case CsTokenType.MultiLineComment:
            this.CheckTabsInComment(token);
            break;
        }

        switch (token.CsTokenClass)
        {
          case CsTokenClass.ConstructorConstraint:
            this.CheckSpacing(((ConstructorConstraint)token).ChildTokens);
            break;

          case CsTokenClass.GenericType:
            this.CheckGenericSpacing((GenericType)token);
            this.CheckSpacing(((TypeToken)token).ChildTokens);
            break;

          case CsTokenClass.Type:
            this.CheckSpacing(((TypeToken)token).ChildTokens);
            break;
        }
      }
    }

    private void CheckGenericSpacing(GenericType generic)
    {
      Param.AssertNotNull(generic, "generic");
      if (generic.ChildTokens.Count == 0)
      {
        return;
      }

      foreach (var token in generic.ChildTokens)
      {
        if (this.Cancel)
        {
          break;
        }

        if (token.CsTokenClass == CsTokenClass.GenericType)
        {
          this.CheckGenericSpacing(token as GenericType);
        }

        if (!token.Generated && token.CsTokenType == CsTokenType.WhiteSpace)
        {
          this.CheckWhitespace(token as Whitespace);
        }
      }
    }

    private void CheckWhitespace(Whitespace whitespace)
    {
      Param.AssertNotNull(whitespace, "whitespace");

      if (whitespace.Location.StartPoint.IndexOnLine == 0 && Regex.IsMatch(whitespace.Text, "^ +"))
      {
        this.AddViolation(whitespace.FindParentElement(), whitespace.LineNumber, "TabsMustBeUsed");
      }
    }

    private void CheckTabsInComment(CsToken comment)
    {
      Param.AssertNotNull(comment, "comment");

      var lines = comment.Text.Split('\n');
      for (int i = 0; i < lines.Length; i++)
      {
        if (Regex.IsMatch(lines[i], "^ +"))
        {
          this.AddViolation(comment.FindParentElement(), comment.LineNumber + i, "TabsMustBeUsed");
        }
      }
    }
  }
}

请注意,您还必须在程序集中与这个东西一起嵌入 XML 文件“SpacingRules.xml”。(阅读 StyleCop SDK 文档了解更多信息。)

<?xml version="1.0" encoding="utf-8" ?>
<SourceAnalyzer Name="Custom Spacing Rules">
  <Description>
    Rules which verify the spacing placed between keywords and symbols in the code.
  </Description>
  <Rules>
    <Rule Name="TabsMustBeUsed" CheckId="MY1027">
      <Context>Spaces are not allowed. Use tabs instead.</Context>
      <Description>Verifies that the code does not contain spaces.</Description>
    </Rule>
  </Rules>
</SourceAnalyzer>
于 2011-04-07T23:23:36.770 回答
8

您可以使用StyleCop+插件来强制使用选项卡。

下载后将StyleCopPlus.dll其放在Custom RulesStyleCop 主文件夹内的文件夹C:\Program Files (x86)\StyleCop 4.7\Custom Rules中或直接放在主文件夹中。

现在,当你打开一个Settings.StyleCopStyleCopSettingsEditor你将能够设置规则SP2001: CheckAllowedIndentationCharacters

此规则可以在StyleCop+选项卡下、More Custom Rules子选项卡下、Formatting标题下找到:

规则选项

于 2012-05-16T10:57:50.760 回答
3

假设您使用 Visual Studio 作为您的 IDE,并且您的团队成员认同这个想法,您可以做的一件事是将 VS 设置为使用制表符而不是空格,导出并共享设置文件。

该设置可以在“工具”>“选项”>“文本编辑器”>“所有语言”(或您希望使用的语言)>“选项卡”下找到,然后在右侧您可以选择“插入空格”或“保留选项卡”。

要从 Visual Studio 导出设置:工具 > 导入和导出设置 > 导出选定的环境设置 > 选择“选项”

只是一个想法——但老实说,真正的问题似乎是你队友的支持。否则,他们总是可以恢复到他们的设置。或者,在登记入住时,正如 Sam 建议的那样,您可以进行一些自动重新格式化。

高温高压

于 2010-02-17T19:52:13.533 回答
3

打包方式:

似乎当前的趋势是通过 nuget 包来做到这一点(并且经典的 StyleCop 可能会在某个时候被淘汰)。因此,要对包执行此操作,请执行以下操作:

通过nuget:

Install-Package Microsoft.CodeAnalysis.FxCopAnalyzers
Install-Package StyleCop.Analyzers -Version 1.1.0-beta006

请注意对预发布的引用(此时),选项卡的设置仅在测试版中可用。

将以下代码作为 ca.ruleset 添加到您的项目中:

<?xml version="1.0" encoding="utf-8"?>
<RuleSet Name="Custom Rulset" Description="Custom Rulset" ToolsVersion="14.0">
    <Rules AnalyzerId="AsyncUsageAnalyzers" RuleNamespace="AsyncUsageAnalyzers">
        <Rule Id="UseConfigureAwait" Action="Warning" />
    </Rules>
    <Rules AnalyzerId="Microsoft.Analyzers.ManagedCodeAnalysis" RuleNamespace="Microsoft.Rules.Managed">
        <Rule Id="CA1001" Action="Warning" />
        <Rule Id="CA1009" Action="Warning" />
        <Rule Id="CA1016" Action="Warning" />
        <Rule Id="CA1033" Action="Warning" />
        <Rule Id="CA1049" Action="Warning" />
        <Rule Id="CA1060" Action="Warning" />
        <Rule Id="CA1061" Action="Warning" />
        <Rule Id="CA1063" Action="Warning" />
        <Rule Id="CA1065" Action="Warning" />
        <Rule Id="CA1301" Action="Warning" />
        <Rule Id="CA1400" Action="Warning" />
        <Rule Id="CA1401" Action="Warning" />
        <Rule Id="CA1403" Action="Warning" />
        <Rule Id="CA1404" Action="Warning" />
        <Rule Id="CA1405" Action="Warning" />
        <Rule Id="CA1410" Action="Warning" />
        <Rule Id="CA1415" Action="Warning" />
        <Rule Id="CA1821" Action="Warning" />
        <Rule Id="CA1900" Action="Warning" />
        <Rule Id="CA1901" Action="Warning" />
        <Rule Id="CA2002" Action="Warning" />
        <Rule Id="CA2100" Action="Warning" />
        <Rule Id="CA2101" Action="Warning" />
        <Rule Id="CA2108" Action="Warning" />
        <Rule Id="CA2111" Action="Warning" />
        <Rule Id="CA2112" Action="Warning" />
        <Rule Id="CA2114" Action="Warning" />
        <Rule Id="CA2116" Action="Warning" />
        <Rule Id="CA2117" Action="Warning" />
        <Rule Id="CA2122" Action="Warning" />
        <Rule Id="CA2123" Action="Warning" />
        <Rule Id="CA2124" Action="Warning" />
        <Rule Id="CA2126" Action="Warning" />
        <Rule Id="CA2131" Action="Warning" />
        <Rule Id="CA2132" Action="Warning" />
        <Rule Id="CA2133" Action="Warning" />
        <Rule Id="CA2134" Action="Warning" />
        <Rule Id="CA2137" Action="Warning" />
        <Rule Id="CA2138" Action="Warning" />
        <Rule Id="CA2140" Action="Warning" />
        <Rule Id="CA2141" Action="Warning" />
        <Rule Id="CA2146" Action="Warning" />
        <Rule Id="CA2147" Action="Warning" />
        <Rule Id="CA2149" Action="Warning" />
        <Rule Id="CA2200" Action="Warning" />
        <Rule Id="CA2202" Action="Warning" />
        <Rule Id="CA2207" Action="Warning" />
        <Rule Id="CA2212" Action="Warning" />
        <Rule Id="CA2213" Action="Warning" />
        <Rule Id="CA2214" Action="Warning" />
        <Rule Id="CA2216" Action="Warning" />
        <Rule Id="CA2220" Action="Warning" />
        <Rule Id="CA2229" Action="Warning" />
        <Rule Id="CA2231" Action="Warning" />
        <Rule Id="CA2232" Action="Warning" />
        <Rule Id="CA2235" Action="Warning" />
        <Rule Id="CA2236" Action="Warning" />
        <Rule Id="CA2237" Action="Warning" />
        <Rule Id="CA2238" Action="Warning" />
        <Rule Id="CA2240" Action="Warning" />
        <Rule Id="CA2241" Action="Warning" />
        <Rule Id="CA2242" Action="Warning" />
        <Rule Id="CA1012" Action="Warning" />
    </Rules>
    <Rules AnalyzerId="StyleCop.Analyzers" RuleNamespace="StyleCop.Analyzers">
        <Rule Id="SA1305" Action="Warning" />
        <Rule Id="SA1412" Action="Warning" />
        <Rule Id="SA1600" Action="None" />
        <Rule Id="SA1609" Action="Warning" />
    </Rules>
</RuleSet>

通过编辑 .csproj 文件并添加以下内容将其添加到您的项目文件中:

<PropertyGroup>
    <CodeAnalysisRuleSet>ca.ruleset</CodeAnalysisRuleSet>
</PropertyGroup>

要覆盖选项卡(和其他)设置,您需要将 stylecop.json 文件添加到您的项目中。在文件属性中,将构建操作设置为“(分析器)附加文件”。根据项目类型,实际单词“分析器”可能不存在。

将 stylecop.json 文件编辑为如下内容:

{
  "$schema": "https://raw.githubusercontent.com/DotNetAnalyzers/StyleCopAnalyzers/master/StyleCop.Analyzers/StyleCop.Analyzers/Settings/stylecop.schema.json",
  "settings": {
    "documentationRules": {
      "companyName": "YourCompanyName",
      "copyrightText": "Copyright (c) {companyName}. All Rights Reserved.\r\nLicensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.",
      "xmlHeader": false,
      "fileNamingConvention": "metadata"
    },
    "indentation": {
      "useTabs": true
    }
  }
}

至少对于 .NET 标准项目,您需要确保 csproj 文件中包含以下内容(并且没有其他对文件的引用):

<ItemGroup>
  <AdditionalFiles Include="stylecop.json" />
</ItemGroup>

您可能必须重新加载项目和包才能让它们识别 stylecop.json 文件。

参考:

DotNetAnalyzers/StyleCopAnalyzers

.NET Core、代码分析和 StyleCop

于 2018-03-05T14:40:06.633 回答
2

StyleCop 支持创建自定义规则,因此您可以添加自己的“使用前导制表符代替空格”规则。如果您不想尝试开发自己的规则,可以在http://stylecopcontrib.codeplex.com/http://github.com/AArnott/nerdbank.stylecop.rules获取现有规则。

于 2010-02-17T19:59:50.853 回答
2

好的,我设法找出了问题,因为出现此警告的原因,似乎是因为开发人员有时会复制并粘贴代码

如果您使用的是 VS2010 转到解决方案资源管理器检查设置样式警察然后修改样式警察内的设置,例如禁用或取消选中设置[间距.....]

于 2011-06-01T11:29:36.553 回答
1

在您的源代码控制服务器中执行此操作。使用预提交脚本检查文件中是否有以多个空格开头的行并阻止提交。

我同意制表符比空格更好。这是个人喜好,但团队的一致性非常重要。

于 2010-02-17T18:54:07.690 回答