删除“ button1”字段,并在相关方法中将其声明为局部变量

时间:2019-01-22 22:58:13

标签: c# visual-studio codacy

我以编码的形式审查我的代码,编码告诉我

import regex
str = '<div type="copy3" class="sc-bxivhb dHqnfT">756 W Peachtree St NW Atlanta GA 30308</div>'
match = regex.search(r'type="copy3"[^>]+>\K[^<]+', str)

编码的意思是 私人按钮button1;

在这种情况下为button1_Click

我的代码是(只是一个小例子,因为我的代码更大):

Remove the 'button1' field and declare it as a local variable in the relevant methods.

我不知道该怎么做,有人可以解释一下在这里做什么吗? 问候

2 个答案:

答案 0 :(得分:1)

在表单编辑选项卡中,选择此按钮并将其GenerateMember设置为false。但是请注意,您将无法在后面的代码中使用此按钮。

enter image description here

答案 1 :(得分:0)

  

删除“ button1”字段,并在相关方法中将其声明为局部变量。

不要。

您显示的代码是由Windows Forms Designer生成的。如果您对其进行修改,则可能会发生以下两种情况之一:

  1. 更糟糕的情况-您破坏了表格。
  2. 最佳情况-下次调用设计器时,它会覆盖您的更改。

两者都不是理想的,而且两者都是完全多余的。看来您拥有的是一个静态分析工具,该工具可用于分析所编写的代码,而对于您未编写且由该工具生成的代码本质上毫无意义。 (例如,您也不需要分析来自第三方库的代码。)

老实说,只需将您的静态分析工具设置为忽略生成的代码即可。 (这条建议同样适用于单元测试覆盖率度量标准。尤其是涉及到Visual Studio中SOAP服务客户端生成的数千行代码时。如果考虑到一项旧式Web服务集成,可能会扼杀您的分析度量标准,并且没有真正的这样做很有价值。)