Re: これ、どう考えてももっと短くなる気がしてならない

これ、どう考えてももっと短くなる気がしてならない。誰か短くして頂戴。。これをあと５個書くとかバカバカしい気はしてる。反省もしてる。 bit.ly/p9RWq8 #codetter—
西川 伸一 (@shinichiN) September 14, 2011

if ( $_POST['cawaii_desuka'] == 'yes' ) {
    if ( isset($_POST['bg_css']) ) {
        $bg_css = $_POST['bg_css'];
    }
    if ( $bg_css == 'bg_brown' || $bg_css == 'bg_check' || $bg_css == 'bg_none' ) {
        update_option('bg_css', $bg_css);
    }
}

短くするならこうかな？(未検証)

if (isset($_POST['cawaii_desuka']) && $_POST['cawaii_desuka'] === 'yes') {
    $bg_css = isset($_POST['bg_css']) ? $_POST['bg_css'] : '';
    preg_match('/^bg_(brown|check|none)$/', $bg_css) && update_option('bg_css', $bg_css);
}

わかりやすいのならこう

if (isset($_POST['cawaii_desuka']) && $_POST['cawaii_desuka'] === 'yes') {
    $bg_css = isset($_POST['bg_css']) ? $_POST['bg_css'] : '';
    switch ($bg_css) {
        case 'bg_brown':
        case 'bg_check':
        case 'bg_none':
            update_option('bg_css', $bg_css);
    }
}