接口应该返回统一格式,不能以文字作为调用方的判断依据的,要以状态码作为依据

    /**
     *
     * 检查手机号码是否可用
     * @param $cellphone 手机号码
     */
    public function checkPhone($cellphone) {
        $cellphone = trim($cellphone);
        if (InputCheck::CheckCellphone($cellphone) == false)
            throw new ApiException(4001023);
        $member = Member::findFirstByCellphone($cellphone);
        
        //todo by wangtao
        //这里的返回方式没有统一,可以注册或不可以注册都返回的一个文字,让调用方很难判断,一旦文字变化就会让调用方代码变得很麻烦。
        //应该以状态码的形式统一起来返回,哪怕是true or false都要更好
        //提示信息应该只能作为一个接口的辅助形式,不能作为调用方判断依据的。
        //不知道现在有哪些应用已经在调用这个接口,如果要修改,新开一个接口来保证统一。如果确定没有调用,可以删除掉这个接口,避免影响后续接手人员,造成风格的不统一
        if (!empty($member)) {
            $this->response->render('手机号已经被注册啦,换一个手机号码吧!');//返回false会更好,有利于调用方的代码判断
        } else {
            $this->response->render('手机号可以注册');//成功应该要返回一个成功状态码,比如true
        }
    }

posted @ 2015-06-15 18:30  王滔  阅读(563)  评论(0编辑  收藏  举报