以更简洁的方式重写此代码，无需太多其他代码

Question

我想重写这段代码，而不需要那么多“其他”，但在不需要时不检查事物或运行查询方面仍然保持高效。

有人可以建议更好的方法来编写这个函数吗？

public static function fetch($content) {

    products_library::init();
    self::$cache = $cache = url::assetsPath() . '../cache/soldout_cache';

    //check the cache
    if (file_exists($cache)) {
        $cache_date = filectime($cache);
        db::select('date_modified');
        db::orderBy('date_modified DESC');
        db::limit(1);
        $mod_date = db::get('sc_module_products')->fetch(PDO::FETCH_ASSOC);
        if ($mod_date) {
            $mod_date = strtotime('date_modified');
            if ($cache_date >= $mod_date) { //serve the cache
                try {
                    $soldout = filewriter::read($cache);
                    $soldout = unserialize($soldout);
                } catch (Exception $e) {
                    $soldout = self::query();
                }
            }
            else
                $soldout = self::query();
        }
        else
            $soldout = self::query();
    }
    else
        $soldout = self::query();

    $data['items'] = $soldout; // print_r($items); exit;

    $html = view::load('Product_Display', $data, true);
    return $html;
}

谢谢

Answer 1

将其重构为返回而不是 else 语句的方法

private static function getSoldout() {
    self::$cache = $cache = url::assetsPath() . '../cache/soldout_cache';

    //check the cache
    if (!file_exists($cache)) {
      return self::query();
    }

    $cache_date = filectime($cache);
    db::select('date_modified');
    db::orderBy('date_modified DESC');
    db::limit(1);
    $mod_date = db::get('sc_module_products')->fetch(PDO::FETCH_ASSOC);
    if (!$mod_date) {
      return self::query();
    }

    $mod_date = strtotime('date_modified');
    if ($cache_date < $mod_date) {
      return self::query();
    }

    try {
      //serve the cache
      $soldout = filewriter::read($cache);
      $soldout = unserialize($soldout);
      return $soldout;
    } catch (Exception $e) {
      return self::query();
    }
}

public static function fetch($content) {

    products_library::init();

    $soldout = self::getSoldout();

    $data['items'] = $soldout; // print_r($items); exit;

    $html = view::load('Product_Display', $data, true);
    return $html;
}

我不明白这一行，那里有错误吗？

$mod_date = strtotime('date_modified');

Answer 2

将 $soldout 设置为 NULL。然后删除 else $soldout = self::query() 语句。

在 if 语句测试 $soldout 是否为 NULL 且结果正确后运行查询。

Answer 3

开关盒块在这里会产生奇迹。您只需一个指向默认情况的 break 语句。然而，如果我处于你的立场，我会尝试重构整个事情，这需要的不仅仅是快速修复。

Answer 4

像这样的东西可能会起作用。我不确定所有的 if 内部发生了什么以及为什么你需要这么多，它可能更紧凑。

public static function fetch($content) {

    products_library::init();
    self::$cache = $cache = url::assetsPath() . '../cache/soldout_cache';

    $soldout = self::fetchCache($cache);
    if ($soldout === false)
    {
        $soldout = self::query();
    }

    $data['items'] = $soldout; // print_r($items); exit;

    $html = view::load('Product_Display', $data, true);
    return $html;
}

public static function fetchCache($cache) {
    if (file_exists($cache)) {
        $cache_date = filectime($cache);
        db::select('date_modified');
        db::orderBy('date_modified DESC');
        db::limit(1);
        $mod_date = db::get('sc_module_products')->fetch(PDO::FETCH_ASSOC);
        if ($mod_date) {
            $mod_date = strtotime('date_modified');
            if ($cache_date >= $mod_date) { //serve the cache
                try {
                    $result = filewriter::read($cache);
                    $result = unserialize($soldout);
                    return $result;
                } catch (Exception $e) {
                    return false;
                }
            }
        }
    }
    return false;
}

Answer 5

在我看来，好像你可以通过在第一个 if 检查之前将其设置为 self::query() 来默认 $soldout 为 self::query() ，然后删除所有其他内容，因此如果条件不匹配，它仍然是 self::query ()。可能不起作用，具体取决于 self::query() 的作用。