答案：代码审查中应避免嵌套三元运算符、确保类型安全使用??替代?:、注意操作符优先级加括号、禁止副作用操作，提升可读性与逻辑正确性。
在PHP开发中，三元运算符（?:）是一种简洁的条件表达式写法，常用于替代简单的if-else语句。虽然它能提升代码的紧凑性，但过度或不当使用可能导致可读性下降、逻辑错误甚至安全问题。因此，在代码审查过程中，需重点关注以下几点。
1. 避免嵌套三元运算符
嵌套的三元运算符会显著降低代码可读性，容易引发理解偏差和维护困难。
不推荐写法：$result = $a ? $b ? $c : $d : $e;
这种写法逻辑复杂，不易快速判断执行路径。
立即学习“PHP免费学习笔记（深入）”；
推荐做法：将嵌套三元运算符拆分为普通if-else语句或使用括号明确优先级。
$result = $a ? ($b ? $c : $d) : $e;（若必须使用）
更清晰的方式是使用多行if语句，便于调试和阅读。
2. 确保类型安全与默认值合理
三元运算符常用于设置默认值，例如：$name = $_GET['name'] ? $_GET['name'] : 'guest';。但这种写法存在隐患——当值为0、空字符串或false时，仍会被判定为“假”，从而触发默认值。
用户真实输入为"0"时，也可能被替换为默认值，造成数据误判。
推荐做法：使用isset()或??（null合并运算符）来判断变量是否存在或为null。
$name = $_GET['name'] ?? 'guest';
这样只在键不存在或值为null时使用默认值，避免误判其他“falsy”值。
3. 注意操作符优先级
PHP中三元运算符的优先级较低，与其他操作符混合使用时可能产生意外结果。
示例问题：echo 'Hello ' . $user ? $user : 'Guest';
由于点连接符（.）优先级高于三元运算符，实际执行顺序等价于：(echo 'Hello ' . $user) ? $user : 'Guest';，这会导致逻辑错误。
用括号明确分组：
echo 'Hello ' . ($user ? $user : 'Guest');
4. 保持语义清晰，避免副作用
三元运算符应仅用于返回值，不应包含函数调用或带有副作用的操作。
不良示例：$status = $valid ? doSave() : logError();
这种写法将业务逻辑隐藏在表达式中，不利于追踪执行流程，也违反了“表达式应无副作用”的原则。
推荐重构：使用标准控制结构：
if ($valid) {
doSave();
} else {
logError();
}
基本上就这些。三元运算符用得好可以提升代码简洁度，但审查时要特别关注可读性、类型安全和逻辑正确性。 
