一段網上找的代碼突然爆了,項目出現大Bug

點擊上方藍色字體,選擇“標星公衆號”

優質文章,第一時間送達

轉自:51CTO

www.toutiao.com/i6790598743222649355

本人是做遊戲服務器開發的,碰到一個需求,給符合某些要求的玩家發送道具獎勵,獎勵的數量根據離線的天數計算。

圖片來自 Pexels

這個需求實現起來很簡單,只需要在玩家上線的時候計算上次離線時間和當前時間間隔的天數,然後根據策劃的算法,計算出道具種類與數量,發一封郵件給玩家就可以了。

計算兩個時間間隔天數的函數沒有現成的,自己又懶得寫,就上谷歌搜了下,選了第一條結果,代碼如下:

/**
     * date2比date1多的天數
     * @param date1
     * @param date2
     * @return
     */
    public static int differentDays(Date date1,Date date2)
    {
        Calendar cal1 = Calendar.getInstance();
        cal1.setTime(date1);
        
        Calendar cal2 = Calendar.getInstance();
        cal2.setTime(date2);
       int day1= cal1.get(Calendar.DAY_OF_YEAR);
        int day2 = cal2.get(Calendar.DAY_OF_YEAR);
        
        int year1 = cal1.get(Calendar.YEAR);
        int year2 = cal2.get(Calendar.YEAR);
        if(year1 != year2) //同一年
        {
            int timeDistance = 0 ;
            for(int i = year1 ; i < year2 ; i ++)
            {
                if(i%4==0 && i%100!=0 || i%400==0) //閏年
                {
                    timeDistance += 366;
                }
                else    //不是閏年
                {
                    timeDistance += 365;
                }
            }
            
            return timeDistance + (day2-day1) ;
        }
        else    //不同年
        {
            System.out.println("判斷day2 - day1 : " + (day2-day1));
            return day2-day1;
        }
    }

代碼來源:https://www.cnblogs.com/0201zcr/p/5000977.html

把代碼複製到項目裏,調試下,發現沒問題就直接用了,畢竟谷歌結果第一名,放心。

這段代碼跑了幾個月一直沒問題,但是到了 2020-1-1 日那天,有玩家反饋收到了幾百封獎勵郵件,高興壞了,但是出於對遊戲的熱愛,還是通知了運營人員。

運營把 Bug 反饋到服務器這邊後我開始排查,百思不得其解的是最近幾天都沒有更新服務器, 而前幾天服務器都穩穩地,怎麼突然就出 Bug 了呢?

接下來就是分析玩家數據,結合代碼邏輯確定問題所在,最終根據 Bug 的表現排除了所有可能性後,發現唯一可能出問題的地方就是那個網上覆制過來的計算天數差的函數。

根據調試發現,這個函數在兩個日期參數是不同的年份並且第一個日期大於第二個日期的時候,會返回一個錯誤的結果,比如:

differentDays("2020-1-1","2019-12-25")

理論上這麼調用正確的結果是 -7,但是因爲函數有 Bug,調用結果是 358。

於是本來不用發獎勵,因爲這種特殊情況一下子發出去 358 份,嚴重影響了遊戲某類道具的平衡性。

最後,我改用 Java8 的日期庫修復了 Bug,代碼如下:

publicstaticintdifferentDays(Datedate1,Datedate2{

  if(date1==null||date2==null){
    thrownewRuntimeException("日期不能爲空");
  }
  LocalDatelocalDate1=date2LocalDate(date1);
  LocalDatelocalDate2=date2LocalDate(date2);
  return Generic.long2int(localDate1.until(localDate2,ChronoUnit.DAYS));
}
publicstaticLocalDatedate2LocalDate(Date date{
  Instantinstant=date.toInstant();
  ZoneIdzoneId=ZoneId.systemDefault();
  LocalDatelocalDate=instant.atZone(zoneId).toLocalDate();
  return localDate;
}

至於補救方式就是統計名單,把發出去但還沒有用掉的道具回收,用掉的就當福利,然後再發公告道歉,再送些其他物品彌補。

也幸好補救的及時,要是這些道具收不回來,遊戲運營的策略都要大變了,我特麼肯定沒好果子吃了。

所以千萬別在網上覆制來路不明的代碼亂用,如果真的要用,必須反覆測試,否則哪一天突然暴雷有你受的。



記得點擊「在看」,然後轉給你身邊的朋友們。


發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章