Question

我在下面有一小段代码，我很好奇你会在最佳实践/代码维护等方面改变哪些类型的东西。

function _setAccountStatus($Username, $AccountStatus)
{
if ($Username == '' || ($AccountStatus != 'Active' || $AccountStatus != 'Banned' || $AccountStatus != 'Suspended')) {
    // TODO: throw error here.
}

$c1 = new Criteria();

$c1->add(UsersPeer::USERNAME,$Username);

$rs = UsersPeer::doSelect($c1);

if (count($rs) > 0) {
    $UserRow = array_pop($rs);

    $UserRow->setAccountStatus($AccountStatus);

    try {
        $UserRow->save();
    } catch ( PropelException $e ) {
        return false;
    }

    return true;
}

return false;
}

Answer 1

我会在你的if语句中使用empty()而不是$ Username ==''。我之前没有使用过推进器，但是我希望将这个方法放在我的User对象本身上，同时获取和保存由单独对象执行的用户对象。伪代码就是这样的。

$ user = userManager-＆gt; getUser（$ username）; $用户＆GT; setAccountStatus（$ accountStatus）; $ userManager-＆GT; saveUser（$用户）;

Answer 2

在最后一次返回false之前的else子句将被优先使用，只是为了使代码更具可读性。

您在我的代码中对最佳实践/维护有什么改变？

2 个答案: