Discuz!官方免费开源建站系统

 找回密码
 立即注册

QQ登录

只需一步,快速开始

搜索

两个if判断语句

[复制链接]
harouse 发表于 2012-6-18 12:21:43 | 显示全部楼层 |阅读模式
菜鸟请教:install/include/install_function.php 32行
if($error_msg) {
    if(!empty($error_msg)) {

    }
}

$error_msg和 !empty($error_msg)不是一样的吗?听说好像!empty($error_msg)的写法可以防注入和安全,不懂为什么可以起到安全作用。菜鸟以为,只要if(!empty($error_msg))就可以了,不需要外层的if。请问,为什么要加上外层的if判断呢?
m.king 发表于 2012-6-18 15:40:01 | 显示全部楼层
{:soso_e103:}  这个确实是个问题,我看看他们这么写的含义,感谢您的反馈!
回复

使用道具 举报

mark35 发表于 2012-6-18 15:55:55 | 显示全部楼层
本帖最后由 mark35 于 2012-6-18 15:57 编辑
  1. if($error_msg) {
  2.     if(!empty($error_msg)) {

  3.     }
  4. }
复制代码
真没想到DZX还有这么憋足的代码。 真没审核机制和审核人员么?

用if (!empty($error_msg)) 判断而不是直接 if ($error_msg)的原因是如果变量未定义则会抛出notice级错误。所以判断变量值要么用isset(), 要么empty()而不要直接判断变量



回复

使用道具 举报

 楼主| harouse 发表于 2012-6-18 16:10:22 | 显示全部楼层
m.king 发表于 2012-6-18 15:40
这个确实是个问题,我看看他们这么写的含义,感谢您的反馈!

不用谢!
回复

使用道具 举报

 楼主| harouse 发表于 2012-6-18 16:29:41 | 显示全部楼层
本帖最后由 harouse 于 2012-10-22 17:30 编辑
mark35 发表于 2012-6-18 15:55
真没想到DZX还有这么憋足的代码。 真没审核机制和审核人员么?

用if (!empty($error_msg)) 判断而不是直 ...

谢谢您!!
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

手机版|小黑屋|Discuz! 官方站 ( 皖ICP备16010102号 )star

GMT+8, 2025-2-25 14:40 , Processed in 0.020808 second(s), 6 queries , Gzip On, Redis On.

Powered by Discuz! X3.4

Copyright © 2001-2023, Tencent Cloud.

快速回复 返回顶部 返回列表