Ewww. Это затрудняет чтение, я думаю, что подобное решение было бы лучше. (Но не лучше). Лучше всего, вероятно, иметь функции js, которые знают, как обрабатывать то, что скрывать и показывать, а также сами добавлять стили в элементы тега или не выводить идентификаторы явно.
В любом случае, это должно быть более читабельным, и будет проще добавлять дополнительные элементы;
<?php
switch ($re) {
case 'reject':
$styles = array(
'hide' => 'none',
'onreject' => 'block',
'onapprove' => 'none'
);
break;
case 'approve':
$styles = array(
'hide' => 'none',
'onreject' => 'none',
'onapprove' => 'block'
);
default:
$styles = array(
'hide' => 'block',
'onreject' => 'none',
'onapprove' => 'none'
);
}
foreach ($styles as $id => $value) {
echo 'document.getElementById("' , $id, '").style.display="', $value, '";', PHP_EOL;
}
?>