zoukankan      html  css  js  c++  java
  • Class does not Implement Equals——Code Correctness(代码正确性)

        系列文章目录:

        使用Fortify进行代码静态分析(系列文章)

    class does not implement equals(类未能实现Equals方法)

        示例:    

     1 protected void Check_Clicked(Object sender, EventArgs e)
     2 {
     3    for (int i = 0; i < dgList.Items.Count; i++)
     4    {
     5       if (sender.Equals(dgList.Items[i].FindControl("cbxItem")))
     6       {
     7          
     8       }
     9     }
    10 }

       

        Fortify提示:

        Equals() is called on an object that does not implement Equals()。 

      在未实现Equals的类上调用Equals()方法。

     

      详细解释: 

        When comparing objects, developers usually want to compare properties of objects. However, calling Equals() on a class (or any super class/interface) that does not explicitly implement Equals() results in a call to the Equals() method inherited from System.Object. Instead of comparing object member fields or other properties, Object.Equals() compares two object instances to see if they are the same. Although there are legitimate uses of Object.Equals(), it is often an indication of buggy code. 

      当比较对象时,开发人员通常想比较的是对象的属性或字段。但是,调用未显式实现Equals()方法的类、超类或者接口,会导致调用从System.Object的继承而来的Equals()方法。Objects.Equals()方法比较是为了比较两个对象是否相同,而不是比较它们的字段或者属性。虽然这种写法是合法的,但通常这也意味着代码Bug 

     

        Fortify错误示例:    

     1 public class AccountGroup
     2 {
     3     private int gid;
     4 
     5     public int Gid
     6     {
     7         get { return gid; }
     8         set { gid = value; }
     9     }
    10 }
    11 ...
    12 public class CompareGroup
    13 {
    14     public bool compareGroups(AccountGroup group1, AccountGroup group2)
    15     {
    16         return group1.Equals(group2);//Equals() is not implemented in AccountGroup
    17     }
    18 }

     

    Fortify建议:

         Verify that the use of Object.Equals() is really the method you intend to call. If not, implement an Equals() method or use a different method for comparing objects.

         确保调用Ojbect.Equals()方法确实是你需要调用的,否则,实现Equals()方法来进行对象的比较。

     

     Fortify推荐示例: 

     1 public class AccountGroup
     2 {
     3     private int gid;
     4 
     5     public int Gid
     6     {
     7         get { return gid; }
     8         set { gid = value; }
     9     }
    10 
    11     public override Boolean Equals(Object obj)
    12     {
    13         if (obj == null)
    14             return false;
    15         if (this.GetType() != obj.GetType())
    16             return false;
    17         AccountGroup other = (AccountGroup)obj;
    18         return (gid == other.Gid);
    19     }
    20 }
    21 22 public class CompareGroup
    23 {
    24     public static bool compareGroups(AccountGroup group1, AccountGroup group2)
    25     {
    26         return group1.Equals(group2);
    27     }
    28 }
  • 相关阅读:
    正则表达式
    文件上传例子
    如何做好数字化体验管理,了解一下?
    云原生背景下故障演练体系建设的思考与实践—云原生混沌工程系列之指南篇
    OpenKruise v1.0:云原生应用自动化达到新的高峰
    Spring Boot Serverless 实战系列“部署篇” | Mall 应用
    阿里云实时数仓Hologres年度发布,解读数仓新趋势
    基于 ASK + EB 构建容器事件驱动服务
    各位 PHPer,Serverless 正当时
    如何在零停机的情况下迁移 Kubernetes 集群
  • 原文地址:https://www.cnblogs.com/gudi/p/6625830.html
Copyright © 2011-2022 走看看