Go基于go-cache搭建基础缓存系统时if else语句触发报错

核心报错原因 #

• 违反Go语法格式强制规则：Go语言规定else/else if关键字必须与上一层if代码块的闭合大括号}处于同一行，不允许单独换行、空行后书写。原代码中第一个if块闭合后空了一行才写else if，会直接触发编译器语法解析错误，这是if else块报错的核心来源。
• 变量作用域越界：原代码在else if _, ok := n.data[key]; ok分支内定义的局部变量ok，作用域仅覆盖当前else if代码块，后续else块访问该变量时已超出作用域范围，会触发“未定义变量”类编译错误。
• 命名冲突隐患：全局缓存实例变量命名为cache，与导入的patrickmn/go-cache包名完全重名，会导致编译器将后续所有cache.开头的调用识别为对全局变量的访问，而非对缓存包方法的调用，触发类型不匹配错误。

对应修正方案 #

• 调整if-else书写格式：所有分支的连接符（else/else if）必须紧跟在上一层代码块的闭合大括号后，不换行、不空档。
• 重构分支逻辑：减少多层嵌套的else if，在缓存未命中的分支内先获取数据源的值与存在状态，按“存在则写入缓存返回、不存在则返回错误”的逻辑拆分判断，从根源上避免作用域越界问题。
• 重命名全局缓存实例：将与包名重名的全局缓存变量修改为memoryCache，消除命名冲突。

修正后可运行的代码实现 #

package datasource

import (
	"fmt"
	"github.com/patrickmn/go-cache"
	"time"
)

type DataSource interface {
	Value(key string) (interface{}, error)
}

// DataSourceStr 类型，实现DataSource接口
type DataSourceStr struct {
	data map[string]string
}

// 重命名缓存实例避免与包名冲突
var memoryCache = cache.New(5*time.Minute, 5*time.Minute)

func (n *DataSourceStr) Value(key string) (interface{}, error) {
	// 检索对应缓存键
	cached, found := memoryCache.Value(key)
	// 缓存命中则直接返回值
	if found {
		return cached, nil
	}

	// 缓存未命中则查询数据源
	val, ok := n.data[key]
	if !ok {
		return nil, fmt.Errorf("key not found %q", key)
	}

	// 将查询结果写入缓存后返回
	memoryCache.Set(key, val, cache.DefaultExpiration)
	return val, nil
}

func getFromDS(datasource DataSource, key string) (string, error) {
	v, err := datasource.Value(key)
	if err != nil {
		return "", err
	}
	return fmt.Sprint(v), nil
}

注：原getFromDS函数中错误返回时丢失了原始错误信息，修正后将原始错误正常向上传递，符合接口设计的错误传递逻辑。

内容的提问来源于stack exchange，提问作者LeCoda