使用 try/catch 时处理返回值的正确方法

Question

我正在开发一个处理外部 API 的 wordpress 插件。我想对 API 调用使用 try/catch 块，但我不确定我处理返回值的方式是否正常。

try {
            $response = wp_remote_post($url,$args);

            $communication_location = wp_remote_retrieve_header( $response, 'location' );
            $communication_location_arr = explode('/', $communication_location);

            $communication_id = end($communication_location_arr);
            $response_code = wp_remote_retrieve_response_code($response);

        }

        catch (Exception $e){
            throw new Exception('Something went wrong when trying to create the communication');
        }

        return array(0 => $response_code,1 => $communication_id);

try 块应该只包含 wp_remote_post 调用吗？

Answer 1

object 的方法被附加到 init 动作钩子上，并在触发 init 钩子时被抛出，而不是在创建对象时，也不是在它们被附加时。

class SomeClass {
    public function __construct() {
        // when the init action/event happens, call the wp_some_method
        add_action( 'init', array( $this, 'wp_some_method' ) );
    }
    function wp_some_method( $post_type ){
        throw new \Exception('error'); 
    }
}
try{
    // great, no exceptions where thrown while creating the object
    $o = new SomeClass();    
} catch (\Exception $ex) {
    echo $ex->getMessage();
}

// a small period of time later somewhere in WP Core...

do_action( 'init' ); // a method we attached to the init hook threw an exception, but nothing was there to catch it!

这些会更合适：

• 在类方法中添加try catch（最好）
• 不要在附加到钩子/事件的函数中抛出异常（更好）
• 在不是您附加的方法的新方法中抛出异常，以便您可以添加 try catch（好的，需要很好地分离关注点和抽象）
• 添加一个全局错误处理程序（hackish，强烈建议不要这样做，这将花费比其价值更多的时间，可能会捕获您从未打算捕获的其他异常）

否则，没有合理、合乎逻辑、常识的理由说明 throw new \Exception 的代码行应该像上面那样在 try catch 块内执行，而无需像在测试中那样有目的地手动调用它。