1

好吧,如果状态为 1,则为“有效的保险事件”,如果为 2,则为“已完成的保险事件”。

if(!empty($ins_event))
            {
            echo "<tr><td>&nbsp;<img src='/check-icon.gif'> <a href='". matry::here(array('event_id'=>$ins_event['id'])) . "'>" . ( $ins_event['status'] == 2 ? "Completed Insurance Event": "Active Insurance Event") . "</a></td></tr>"; 
            }
else 
        {
        echo "<tr><td>" . cbox_return() . "<a href='". matry::here_to('new', array('tfilt'=>'IN', 'pfilt'=>$patient->code)) . "' style='color: #000; color:$rx_image_color'>**Ins Event Not Created**</td></tr>";
        }

我在这里有一个颜色变量:

<?php
$rx_event_colors = $rx_ev_status = '#009933';
?>

我如何使用此变量来获取 2 的状态并更改字体颜色。

我应该分解脚本并使用 if {} else {} 语句吗?


更新代码:

echo "<tr><td>&nbsp;<img src='/check-icon.gif'> <a href='". matry::here(array('event_id'=>$ins_event['id'])) . "'" . ( $ins_event['status'] == 2 ? ' style="color: ' . $rx_ev_status . '">Completed Insurance Event' : '>Active Insurance Event') . "</a></td></tr>"; 
4

3 回答 3

2

你基本上会做同样的三元运算:

($ins_event['status'] == 2 ? ' style="color: ' . $rx_ev_status . '"' : '')
于 2013-02-13T00:15:49.140 回答
1

你是这个意思吗?你的问题不是很清楚。。

if(!empty($ins_event))
            {
            echo "<tr><td>&nbsp;<img src='/check-icon.gif'> <a ".($ins_event['status'] == 2 ? 'style="color:'.$rx_ev_status.';"': '')." href='". matry::here(array('event_id'=>$ins_event['id'])) . "'>" . ( $ins_event['status'] == 2 ? "Completed Insurance Event": "Active Insurance Event") . "</a></td></tr>"; 
            }

这是一个编辑,为了清楚起见,将回声分成不同的行...

echo '<tr><td>&nbsp;<img src="/check-icon.gif">';
echo '<a '.($ins_event['status'] == 2 ? 'style="color:'.$rx_ev_status.';"': '')." ";
echo ' href="'. matry::here(array('event_id'=>$ins_event['id'])) . '">'; 
echo ($ins_event['status'] == 2 ? 'Completed Insurance Event': 'Active Insurance Event');
echo '</a></td></tr>'; 
于 2013-02-13T00:14:35.737 回答
-2

是的,出于性能和代码维护的原因,逻辑操作应尽可能少地进行。在单个if { … } else { … }块内设置文本状态和颜色变量。将状态和颜色存储到变量后,可以echo大大简化语句。

echo "<tr><td>&nbsp;<img src='/check-icon.gif'> <a href='". matry::here(array('event_id'=>$ins_event['id'])) . "'>$status</a></td></tr>";

我还会考虑将 的输出存储matry::here(…)在一个变量中,以便更轻松地阅读代码。

于 2013-02-13T00:16:30.387 回答