【发布时间】:2014-09-16 21:35:12
【问题描述】:
我正在使用 PHP 生成 HTML,数据显示框的主要可选部分取决于框中显示的数据类型。这导致我的主要代码行看起来像这样:
if ($sLockIconHTML == "") {
// Leave a space where the lock icon would have been.
$sLockIconHTML = "<div class='user-task-box-right-action-icon'></div>";
}
什么是替换字符串值的干净方法,但前提是该值是空字符串?
【问题讨论】:
-
代码长度和代码可读性可以成反比关系。那是一行,并不意味着代码是更干净的。
-
这听起来像是一个可疑的设计.. 忽略整个“使用模板”,可能只有“一个”输出变量(谁的名字不取决于“实际内容”)互斥的显示结果。
-
1.在每个上下文中(即通常在函数内部),您通常应该知道哪些变量已设置,哪些未设置。 2. 上面的代码如果
$sLockIconHTML没有设置会触发E_NOTICE错误(意思是:乱码)。 -
对 jason 详细代码的一百万票赞成更容易阅读和调试。
-
@lxg 这只是一个 sn-p,而不是整个代码,显然
$sLockIconHTML已在上面定义,并且在某些时候可能会重新定义。
标签: php coding-style code-formatting