Предложить лучший способ использования этого условного оператора

Question

Есть ли лучший способ проверить этот кусок кода,

if (is_string($test) && $test != NULL) { 
    $test = 'yes'; 
} else { 
    $test = null; 
} 

И

if (is_null($test) || !isset($this->_status[$test])) { 
    $icon = null; 
} else { 
    $icon = $this->getImage($this->_status[$test]); 
} 

Answer 1

Лучше путь вы имеете в виду за счет сокращения линии кодов? Как насчет использования тернарного оператора.

$test = is_string($test) ? "Yes" : NULL 

Answer 2

Вы можете использовать сокращенную если/другое, используя оператор троичного

$test = (is_string($test) and $test != NULL) ? 'yes' : null; 
$icon = (is_null($test) or ! isset($this->_status[$test])) ? null : $this->getImage($this->_status[$test]);