Qual dos 2 vocês fariam "neste código"? Boa semântica e fácil leitura
if (in_array($id, array_keys(self::$colors))) {
return self::$colors[$id]['Name'];
} else {
throw new Exception("Color ID <b>$id</b> not exists");
}
Ou assim, em apenas uma linha:
if (in_array($id, array_keys(self::$colors))) return self::$colors[$id]['Name']; else throw new Exception("Color ID <b>$id</b> not exists");