最近修改的几个小bug

2019-11-14 16:39:05

字体：大中小

来源：转载

供稿：网友

最近修改的几个 bug，问题不大，查找起来却几番周折，汇总起来如下。

1.诡异电话号码

客服邮件反馈，很多用户服务热线变成了“0371-45875487”。看到这问题的第一反映是可能因为程序某个地方有人不小心写死了“0371-45875487”，因为服务热线对应数据库为一个字段ServiceTelephone，自然的把“0371-45875487”想象成了一个整体，在web解决方案中查找关键字“0371-45875487”，没有结果，数据库存储过程查找一遍，也没有结果，思路中断了，就先把数据处理了下，原因待查。

没几天，客服又反馈用户服务热线好多又变成了“0371-45875487”，再次超找问题，方式和上次一样，只是搜索关键词变成了“45875487”，没想到很顺利的定位到了问题。出现问题是在个人资料修改页，截取部分代码如下：

                   <tr>                        <td  align="right" valign="middle" bgcolor="#F9F9F9">                            服务热线：</td>                        <td align="left" valign="middle">                            <input name="mendiantelephone1" type="text" id="mendiantelephone1" style="width: 30px;"                                value="0371" size="4" maxlength="5" runat="server" />                            -                            <input name="mendiantelephone2" type="text" id="mendiantelephone2" style="width: 50px;"                                value="45875487" maxlength="8" runat="server" /></td>                    </tr>

资料修改aspx代码

                if (!string.IsNullOrEmpty(AgentInfo.MendianTelephone))                {                    mendiantelephone1.Value = AgentInfo.MendianTelephone.IndexOf("-") < 0 ? "" : AgentInfo.MendianTelephone.Substring(0, AgentInfo.MendianTelephone.IndexOf("-"));                    mendiantelephone2.Value = AgentInfo.MendianTelephone.IndexOf("-") < 0 ? AgentInfo.MendianTelephone : AgentInfo.MendianTelephone.Substring(AgentInfo.MendianTelephone.IndexOf("-") + 1);                }

资料修改aspx.cs代码

如果用户原来没有填写服务热线，在修改页这两个控件的值默认成了“0371”和“45875487”，用户好不知情的情况下把自己的服务热线修改错了。问题修改也很容易，直接把默认值去掉即可。

这问题告诉我们看似诡异的问题肯定也有其原因的，查找问题时可以缩小搜索关键词。

2.同名缓存带来的问题

问题描述：惠州房源信息，同一楼盘对应的区县、商圈竟然出现了不同值，如小区三千俊林对应的区县商圈出现了两组“惠州惠阳区”和“惠阳淡水"。查找原因原来是用到了同名缓存造成的，代码如下：

        /// <summary>        /// 根据newcode 获取楼盘字典信息        /// </summary>        public DataTable GetBuildingDicByNewcode(CityBase citySite, long newcode, string PRojtype)        {            DataTable dt = new DataTable();            string oewname = "districtinfo_bus_around" + newcode.ToString() + "_" + projtype + "new";            string cachename = CacheType.housedic + "_" + oewname;            //先取数据缓存            if (this.MemCache.KeyExists(cachename))            {                dt = this.MemCache.Get<DataTable>(cachename);            }            else            {                dt = buildingErrorData.GetBuildingDicByNewcode(citySite, newcode, projtype);                this.MemCache.Set(cachename, dt, DateTime.Now.AddDays(1));            }            return dt;        }

区县商圈都是根据楼盘编号获取的，而楼盘编号在全国都是唯一的，不会重复，这段代码看似没什么问题，那问题在哪呢？原来还有一个异地楼盘业务，就是深圳可以调用惠州的楼盘，调用的时候惠州就成了深圳的区县了，相当于深圳和惠州同一楼盘的区县商圈属性不同。

这问题出现是因为新业务的出现造成了老代码出现问题，新做业务时方方面面得考虑周全才行。

3.重复提交问题

问题描述：在用户进行放心房操作时，有个计数的字段，每设置一条计数字段+1，每取消一次计算字段-1，但这计数字段却出现了负数的情况。查找问题，最终出现在了用户重复提交上，而且从数据上看是有用户发现了这一漏洞，恶意提交了数据。

        public bool CancelRealHouse(CityBase citySite, HouSEOptEntity optEntity, HouseType htype = HouseType.Sale)        {            //刷新房源并设置标签            rowCount = houseDAO.UpdateRealHouseStatus(citySite, htype, optEntity.AgentId, optEntity.HouseIds, string.Empty, false);            //调整操作数,只有设置上了放心房标签才会去调整            if (rowCount > 0)            {                this.houseBizMember.AgentPowerUsingBiz.HouseAction(citySite, htype, HouseAction.UnRealHouse, optEntity.AgentId, rowCount, rowCount, 0);            }        }        public int UpdateRealHouseStatus(CityBase citySite, HouseType houseType, int agentId, List<int> houseIds, string infoCode, bool isSet)        {            int retVal = 0;            int isRealHouse = isSet ? 1 : 0;            DBHelper db = DBHelper.GetDBHelper(BusinessType.AgentHouseWrite, citySite, agentId);            string tableName = GetTableName(citySite, houseType, true);            string refTableName = GetRefreshTableName(citySite, houseType);            string houseIdsStr = string.Join<int>(",", houseIds);            string sql = sql = string.Format("update {0} set isRealHouse = {1} where agentid={2} and status = 1 and isRealHouse != {1} and houseid in ({3});", tableName, isRealHouse, agentId, houseIdsStr);            retVal = db.ExecuteNonQuery(CommandType.Text, sql);            sql = string.Format("update s set s.registdate = getdate(),s.dtimestamp=getdate() from {0} s inner join {1} h with(nolock) on s.houseid = h.houseid where h.agentid={2} and h.[status]=1 and h.houseid in ({3})", refTableName, tableName, agentId, houseIdsStr);            retVal = db.ExecuteNonQuery(CommandType.Text, sql);            return retVal;        }