代码结构优化: 请问怎么判断数据不再当前表中就去另一个表中查询

现在的代码结构感觉过于耦合, 请问怎么优化下.


    if (Cache::has('api_get_city_name_' . $id)) {
        return Cache::get('api_get_city_name_' . $id);
    }

    $rows = \App\Models\CommodityTicket::select('province', 'city')->find($id);
    $rows = empty($rows) ? \App\Models\ScenicSpot::select('province', 'city')->find($id) : $rows;
    $rows = empty($rows) ? \App\Models\Advertorial::select('province', 'city')->find($id) : $rows;

    if (empty($rows)) return Cache::remember('api_get_city_name_' . $id, 600, function () {
        return '';
    });
    .
    .
    .
《L05 电商实战》
从零开发一个电商项目,功能包括电商后台、商品 & SKU 管理、购物车、订单管理、支付宝支付、微信支付、订单退款流程、优惠券等
《L04 微信小程序从零到发布》
从小程序个人账户申请开始,带你一步步进行开发一个微信小程序,直到提交微信控制台上线发布。
讨论数量: 1
qiuyuhome
use App\Models\CommodityTicket;
use App\Models\ScenicSpot;
use App\Models\Advertorial;

public function demo(int $id) 
{
    $cacheName = 'api_get_city_name_' . $id;
    $cacheTime = 600;
    return Cache::remember($cacheName, $cacheTime, function () use ($id) {
        /**
         * 基于现在的表结构, 肯定是需要每个表都查一遍的.
         * 可以考虑新建一张表, 字段为: id, table_name(对应的哪个表), refer_id(查询这个 3 个表使用的 id).
         * 下面是基于你的现有的逻辑, 我的优化. 没想到更高的办法.
         */

        $rows = CommodityTicket::find($id, ['province', 'city']);
        if ($rows) {
            return $rows;
        }

        $rows = ScenicSpot::find($id, ['province', 'city']);
        if ($rows) {
            return $rows;
        }

        return Advertorial::find($id, ['province', 'city']);
    });
}
4年前 评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容,与人为善,比聪明更重要!