_checklowerlimit() 函数设置无用全局变量.可能导致异常

upload/source/function/function_credit.php line22

1. 
   
2. function _checklowerlimit($action, $uid = 0, $coef = 1, $fid = 0, $returnonly = 0) {
   
3.     global $_G;
   
4. 
   
5.     include_once libfile('class/credit');
   
6.     $credit = & credit::instance();
   
7.     $limit = $credit->lowerlimit($action, $uid, $coef, $fid);
   
8.     if($returnonly) return $limit;
   
9.     if($limit !== true) {
   
10.         $GLOBALS['id'] = $limit;     // <--------- 设定了一个全局变量
    
11.         $lowerlimit = is_array($action) && $action['extcredits'.$limit] ? abs($action['extcredits'.$limit]) + $_G['setting']['creditspolicy']['lowerlimit'][$limit] : $_G['setting']['creditspolicy']['lowerlimit'][$limit];
    

复制代码

设定$GLOBALS['id']这个全局变量有何用处？ 在其他代码中没有找到。查看dz72代码
include/global.func.php line82

1. 
   
2. function checklowerlimit($creditsarray, $coef = 1) {
   
3.     if(is_array($creditsarray)) {
   
4.         global $extcredits, $id;  // <----这儿搞出的全局变量
   
5.         foreach($creditsarray as $id => $addcredits) {
   
6.             $addcredits = $addcredits * $coef;
   
7.             if($addcredits < 0 && ($GLOBALS['extcredits'.$id] < $extcredits[$id]['lowerlimit'] || (($GLOBALS['extcredits'.$id] + $addcredits) < $extcredits[$id]['lowerlimit']))) {
   
8.                 showmessage('credits_policy_lowerlimit');
   
9.             }
   
10.         }
    
11.     }
    
12. }
    

复制代码

估计是程序员是在老版本代码上改写新版程序，看到 global $extcredits, $id; 这一行不知道也没仔细搜索$id这个全局变量在哪儿用到、有什么用。直接照搬的global显得偷懒，干脆就直接赋值给$GLOBLS吧
这样随意赋值全局变量并且使用的是最常见的键名，如果在其他文件中发生id冲突那么是极难排查的

楼主有心了 不过DZ目前基本没人管了吧

让故事继续 发表于 2014-3-25 12:11
楼主有心了 不过DZ目前基本没人管了吧

在微薄微信冲击下论坛模式将会回归价值取向。而高大上的X系列将会被无情淘汰，倒是快速小巧7.2的存在价值还大些

Discuz! X3.1??

枯心树 发表于 2014-3-25 19:48
Discuz! X3.1??

没下X3.1，看的X3的代码。不会有那位程序员这么有精力发现了此处吧～

mark35 发表于 2014-3-25 21:23
没下X3.1，看的X3的代码。不会有那位程序员这么有精力发现了此处吧～

果然细心。。。

mark35 发表于 2014-3-25 18:52
在微薄微信冲击下论坛模式将会回归价值取向。而高大上的X系列将会被无情淘汰，倒是快速小巧7.2的存在价值 ...

戴志康已经从腾讯离职 discuz创始人

让故事继续 发表于 2014-3-27 00:48
戴志康已经从腾讯离职 discuz创始人

dz对tx已经没啥大价值了。前两天不是说tx连微薄都要放弃么