天天看點

接口應該傳回統一格式,不能以文字作為調用方的判斷依據的,要以狀态碼作為依據

    /**

     *

     * 檢查手機号碼是否可用

     * @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

        }

    }