【教训】null == ''，改造ThinkSNS 系统里面的一个缓存管理函数S()后，留下一个大bug

本来想简化 ThinkSNS 系统里面的一个缓存管理函数：

<?php
/**
 * 用来对应用缓存信息的读、写、删除
 * $expire = null/0 表示永久缓存，否则为缓存有效期
 */
function S($name, $value = '', $expire = null)
{
	static $_cache = array();   //减少缓存读取
	$cache = model('Cache');
	//$name = C('DATA_CACHE_PREFIX').$name;
	
	if ('' !== $value)
	{
		if (is_null($value))
		{
			// 删除缓存
			$result = $cache->rm($name);
			if ($result)
			{
				unset($_cache[$name]);
			}

			return $result;
		}
		else
		{
			// 缓存数据
			$cache->set($name, $value, $expire);
			$_cache[$name] = $value;
		}

		return true;
	}
	
	if (isset($_cache[$name]))
	{
		return $_cache[$name];
	}
	
	// 获取缓存数据
	$value = $cache->get($name);
	$_cache[$name] = $value;

	return $value;
}

 

简化如下：

/**
 * 内存缓存
 *
 * @param string $name 缓存键
 * @param string $value 缓存值（为''时，表示获取; 为null表示删除）
 * @param mixed $expire = null/0 表示永久缓存，否则为缓存有效期
 */
function S($name, $value = '', $expire = null)
{
	$cache = model('Cache');

	if ($value == '') // 获取缓存
	{
		return $cache->get($name);
	}
	elseif (is_null($value)) // 删除缓存
	{
		return $cache->rm($name);
	}
	else
	{
		return $cache->set($name, $value, $expire);
	}
}

 

结果呢，留下了一个大bug，想通过 S('键名', null); 来删除某个缓存，死活不成功！

通过跟踪，原来 当 $value = null 时，if ($value == '') 匹配成功，弱类型转换，即 null == ''，所以需要把“等号”改为“恒等号”。即，把 if ($value == '') 改为 if ($value === '')，问题解决。

/**
 * 内存缓存
 *
 * @param string $name 缓存键
 * @param string $value 缓存值（为''时，表示获取; 为null表示删除）
 * @param mixed $expire = null/0 表示永久缓存，否则为缓存有效期
 */
function S($name, $value = '', $expire = null)
{
	$cache = model('Cache');

	if ($value === '') // 获取缓存
	{
		return $cache->get($name);
	}
	elseif (is_null($value)) // 删除缓存
	{
		return $cache->rm($name);
	}
	else
	{
		return $cache->set($name, $value, $expire);
	}
}