php 缓存代码的坏味道

缓存在Web程序里必不可少，最常见的形式如下：

01 class Foo extends DAO
02 {
03     public function find_by_a()
04     {
05         $result = $this->cache->get('cache_a');
06
07         if (!$result) {
08             $result = $this->db->getAll('select ... from ... where a ...');
09
10             $this->cache->set('cache_a', $result);
11         }
12
13         return $result;
14     }
15
16     public function find_by_b()
17     {
18         $result = $this->cache->get('cache_b');
19
20         if (!$result) {
21             $result = $this->db->getAll('select ... from ... where b ...');
22
23             $this->cache->set('cache_b', $result);
24         }
25
26         return $result;
27     }
28 }

这个代码很平常，实际情况中，多数人差不多都是这么写代码，先用某个键在缓存里取一下，如果没有就从数据库里实际查询一次，并且把结果缓存起来，这样的代码虽然不够健壮（没有捕捉可能存在的异常），不过本身并没有太大问题，但是若干个方法叠加起来，我们就能明显的感受到坏味道：重复！不说废话了哦，直接给出解决方案：

01 abstract class DAO
02 {
03     public function getCache($key, $closure)
04     {
05         $result = $this->cache->get($key);
06
07         if (!$result) {
08             $result = $closure();
09
10             $this->cache->set($key, $result);
11         }
12
13         return $result;
14     }
15 }
16
17 class Foo extends DAO
18 {
19     public function find_by_a()
20     {
21         return $this->getCache('cache_a', function() {
22             return $this->db->getAll('select ... from ... where a ...');
23         });
24     }
25
26     public function find_by_b()
27     {
28         return $this->getCache('cache_b', function() {
29             return $this->db->getAll('select ... from ... where b ...');
30         });
31     }
32 }

代码有点简陋，通过把非公共代码提取成一个closure，传递给getCache方法，从而消除了重复的坏味道。