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

http://twitter.com/#!/shinichiN/status/113883259829628928

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);
    }
}