Question

您如何看待这种构图？这是错的吗？怎么做得好？

$graph_url = 'https://graph.facebook.com/me?access_token=' . $result['access_token'];
$fb_user = json_decode( file_get_contents( $graph_url ) );
try {
    $user = new Model_User( $fb_user->username );
    $user_meta = new Model_User_Meta ( $user->get ( 'user_id' ) );
    $user_meta->set_user_meta( '_facebook_id', $fb_user->id );
    $user_meta->set_user_meta( '_last_logged_in', 'current_timestamp' );
    $user_meta->save();
} catch ( Exception $e ) {
    if ( $e->getCode() === 0 ){
        $password = Helper_Password::generate_password();
        $hash = Helper_Password::hash_string( $password );
        try {
            $user = new Model_User();
            $user->set( 'user_name', $fb_user->username );
            $user->set( 'user_pass', $hash );
            $user->set( 'user_email', $fb_user->email );
            $user->set( 'user_status',( $fb_user->verified ? 'active' : 'inactive' ) );
            $user->set( 'display_name', $fb_user->name );
            $status = $user->save();
            $user_meta = new Model_User_Meta ( $status->user_id );
            $user_meta->set_user_meta( '_facebook_id', $fb_user->id );
            $user_meta->set_user_meta( '_last_logged_in', 'current_timestamp' );
            $user_meta->save();
        } catch ( Exception $e ) {
            throw $e;
        }
    } else {
        throw $e;
    }
}

Answer 1

尝试在捕获期间恢复，然后在必要时抛出异常就可以了。

但是你的代码中存在一些含糊之处。你指的是哪个$e？原来的例外，还是新的例外？如果要抛出原始异常，则不应在第二个catch语句中覆盖$e：

} catch ( Exception $e ) {
    if ( $e->getCode() === 0 ){
        try {
            // try recovering here?
        } catch ( Exception $otherException ) {
            throw $e;  // throw the original exception instead of the new one
        }
    } else {
        throw $e;
    }
}

如果你想抛出新的异常，你根本不需要内部捕获......

} catch ( Exception $e ) {
    if ( $e->getCode() === 0 ){
        // try recovering here, and let the exception fly as they will
    } else {
        throw $e;
    }
}

您如何看待在catch中嵌套try / catch语句？

1 个答案: